Fixed some formatting with for respec

rubyforgood · May 27, 2023 · 1aa1f0b · 1aa1f0b
1 parent 3f42ae2
commit 1aa1f0b
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 7 deletions.
diff --git a/app/controllers/partners/families_controller.rb b/app/controllers/partners/families_controller.rb
@@ -10,7 +10,7 @@ def index
         current_partner.families
                        .order(sort_order),
         params[:filterrific],
-        default_filter_params: {"include_archived"=>0},
+        default_filter_params: {"include_archived" => 0}
       ) || return
 
       @families = @filterrific.find

diff --git a/app/models/partners/family.rb b/app/models/partners/family.rb
@@ -49,7 +49,6 @@ class Family < Base
     scope :search_agency_guardians, ->(query) { where('case_manager ilike ?', "%#{query}%") }
     scope :include_archived, ->(query) { query.zero? ? where(archived: false) : all }
 
-
     INCOME_TYPES = ['SSI', 'SNAP/FOOD Stamps', 'TANF', 'WIC', 'Housing/subsidized', 'Housing/unsubsidized', 'N/A'].freeze
     INSURANCE_TYPES = ['Private insurance', 'Medicaid', 'Uninsured'].freeze
     EMPLOYMENT_TYPES = %w[Full-time Part-time N/A].freeze
@@ -64,7 +63,7 @@ def create_authorized
     end
 
     def archive_children
-      children.update_all(archived: true)
+      children.each { |c| c.update(archived: true) }
     end
 
     def guardian_display_name
@@ -106,7 +105,7 @@ def csv_export_attributes
         updated_at,
         partner_id,
         military,
-        archived,
+        archived
       ]
     end
   end

diff --git a/spec/requests/partners/family_requests_spec.rb b/spec/requests/partners/family_requests_spec.rb
@@ -43,7 +43,7 @@
       comments: "Some comment 2",
       military: true,
       partner: partner,
-      archived: true,)
+      archived: true)
   end
 
   describe "GET #index" do
@@ -59,14 +59,14 @@
     end
 
     it "should render without any issues and present all families" do
-      get partners_families_path, params: {"filterrific"=>{"include_archived"=>"1"}}
+      get partners_families_path, params: {"filterrific" => {"include_archived" => "1"}}
       expect(response).to render_template(:index)
       expect(assigns[:families].count).to eq(2)
     end
 
     it "should export CSV" do
       headers = {"Accept" => "text/csv", "Content-Type" => "text/csv"}
-      params = {"filterrific"=>{"search_guardian_names"=>"", "search_agency_guardians"=>"", "include_archived"=>"1"}}
+      params = {"filterrific" => {"search_guardian_names" => "", "search_agency_guardians" => "", "include_archived" => "1"}}
       get partners_families_path, headers: headers, params: params
       csv = <<~CSV
         id,guardian_first_name,guardian_last_name,guardian_zip_code,guardian_county,guardian_phone,case_manager,home_adult_count,home_child_count,home_young_child_count,sources_of_income,guardian_employed,guardian_employment_type,guardian_monthly_pay,guardian_health_insurance,comments,created_at,updated_at,partner_id,military,archived