From 2972a7b1e528c9ba66c764e71464a678440a1e89 Mon Sep 17 00:00:00 2001
From: baarkerlounger <db@slothlife.xyz>
Date: Thu, 10 Feb 2022 14:08:31 +0000
Subject: [PATCH] Also remove reasonable preference reason

---
 db/migrate/20220210132254_remove_checkbox_parent_fields.rb | 2 ++
 db/schema.rb                                               | 1 -
 spec/factories/case_log.rb                                 | 3 ---
 spec/fixtures/complete_case_log.json                       | 2 --
 spec/fixtures/exports/case_logs.xml                        | 3 ---
 5 files changed, 2 insertions(+), 9 deletions(-)

diff --git a/db/migrate/20220210132254_remove_checkbox_parent_fields.rb b/db/migrate/20220210132254_remove_checkbox_parent_fields.rb
index 4a4a7925e..61ccec22d 100644
--- a/db/migrate/20220210132254_remove_checkbox_parent_fields.rb
+++ b/db/migrate/20220210132254_remove_checkbox_parent_fields.rb
@@ -3,6 +3,7 @@ class RemoveCheckboxParentFields < ActiveRecord::Migration[7.0]
     change_table :case_logs, bulk: true do |t|
       t.remove :accessibility_requirements
       t.remove :condition_effects
+      t.remove :reasonable_preference_reason
     end
   end
 
@@ -10,6 +11,7 @@ class RemoveCheckboxParentFields < ActiveRecord::Migration[7.0]
     change_table :case_logs, bulk: true do |t|
       t.column :accessibility_requirements, :string
       t.column :condition_effects, :string
+      t.remove :reasonable_preference_reason, :string
     end
   end
 end
diff --git a/db/schema.rb b/db/schema.rb
index ef946def8..bd49638f9 100644
--- a/db/schema.rb
+++ b/db/schema.rb
@@ -98,7 +98,6 @@ ActiveRecord::Schema[7.0].define(version: 202202071123100) do
     t.integer "lawaitlist"
     t.string "property_postcode"
     t.integer "reasonpref"
-    t.string "reasonable_preference_reason"
     t.integer "cbl"
     t.integer "chr"
     t.integer "cap"
diff --git a/spec/factories/case_log.rb b/spec/factories/case_log.rb
index 4e59c1656..0f0367f84 100644
--- a/spec/factories/case_log.rb
+++ b/spec/factories/case_log.rb
@@ -53,8 +53,6 @@ FactoryBot.define do
       reservist { "No" }
       illness { "Yes" }
       preg_occ { "Yes" }
-      accessibility_requirements { "No" }
-      condition_effects { "dummy" }
       tenancy_code { "BZ757" }
       startertenancy { "No" }
       tenancylength { 5 }
@@ -81,7 +79,6 @@ FactoryBot.define do
       lawaitlist { "Less than 1 year" }
       property_postcode { "NW1 5TY" }
       reasonpref { "Yes" }
-      reasonable_preference_reason { "dummy" }
       cbl { "Yes" }
       chr { "Yes" }
       cap { "No" }
diff --git a/spec/fixtures/complete_case_log.json b/spec/fixtures/complete_case_log.json
index a172080c3..f98c80d70 100644
--- a/spec/fixtures/complete_case_log.json
+++ b/spec/fixtures/complete_case_log.json
@@ -47,8 +47,6 @@
     "reservist": "No",
     "illness": "Yes",
     "preg_occ": "No",
-    "accessibility_requirements": "No",
-    "condition_effects": "dummy",
     "tenancy_code": "BZ757",
     "startdate": "12/12/2021",
     "day": 12,
diff --git a/spec/fixtures/exports/case_logs.xml b/spec/fixtures/exports/case_logs.xml
index 6653465bb..fb09c1619 100644
--- a/spec/fixtures/exports/case_logs.xml
+++ b/spec/fixtures/exports/case_logs.xml
@@ -47,8 +47,6 @@
     <reservist>No</reservist>
     <illness>Yes</illness>
     <preg_occ>Yes</preg_occ>
-    <accessibility_requirements>No</accessibility_requirements>
-    <condition_effects>dummy</condition_effects>
     <tenancy_code>BZ757</tenancy_code>
     <startertenancy>No</startertenancy>
     <tenancylength>5</tenancylength>
@@ -68,7 +66,6 @@
     <lawaitlist>Less than 1 year</lawaitlist>
     <property_postcode>NW1 5TY</property_postcode>
     <reasonpref>Yes</reasonpref>
-    <reasonable_preference_reason>dummy</reasonable_preference_reason>
     <cbl>Yes</cbl>
     <chr>Yes</chr>
     <cap>No</cap>