WRSAT-185 create disability benefits screen (#72)

jenny-heath · web-flow · commit d4232d119fa9 · 2026-01-14T13:21:44.000-08:00
diff --git a/app/assets/stylesheets/application.scss b/app/assets/stylesheets/application.scss
@@ -69,8 +69,6 @@ i.no-button-cross {
 }
 
 .checkbox-group {
-  margin-bottom: 6rem;
-
   .input-group {
     margin-bottom: 0;
   }
diff --git a/app/controllers/disability_benefits_controller.rb b/app/controllers/disability_benefits_controller.rb
@@ -0,0 +1,17 @@
+class DisabilityBenefitsController < QuestionController
+  include WrExemptionsConcern
+
+  def self.attributes_edited
+    [
+      :receiving_benefits_ssdi,
+      :receiving_benefits_ssi,
+      :receiving_benefits_veterans_disability,
+      :receiving_benefits_disability_pension,
+      :receiving_benefits_workers_compensation,
+      :receiving_benefits_insurance_payments,
+      :receiving_benefits_other,
+      :receiving_benefits_none,
+      :receiving_benefits_write_in
+    ]
+  end
+end
diff --git a/app/controllers/navigation/screener_navigation.rb b/app/controllers/navigation/screener_navigation.rb
@@ -10,6 +10,7 @@ class ScreenerNavigation < ControllerNavigation::ControllerNavigation
         ControllerNavigation::NavigationStep.new(CaringForSomeoneController),
         ControllerNavigation::NavigationStep.new(IsPregnantController),
         ControllerNavigation::NavigationStep.new(HasUnemploymentBenefitsController),
+        ControllerNavigation::NavigationStep.new(DisabilityBenefitsController),
         ControllerNavigation::NavigationStep.new(BasicInfoMilestoneController),
         ControllerNavigation::NavigationStep.new(PersonalInformationController),
         ControllerNavigation::NavigationStep.new(TempEndController)
diff --git a/app/models/screener.rb b/app/models/screener.rb
@@ -9,6 +9,14 @@ class Screener < ApplicationRecord
   enum :caring_for_disabled_or_ill_person, {unfilled: 0, yes: 1, no: 2}, prefix: true
   enum :caring_for_no_one, {unfilled: 0, yes: 1, no: 2}, prefix: true
   enum :has_unemployment_benefits, {unfilled: 0, yes: 1, no: 2}, prefix: true
+  enum :receiving_benefits_ssdi, {unfilled: 0, yes: 1, no: 2}, prefix: true
+  enum :receiving_benefits_ssi, {unfilled: 0, yes: 1, no: 2}, prefix: true
+  enum :receiving_benefits_veterans_disability, {unfilled: 0, yes: 1, no: 2}, prefix: true
+  enum :receiving_benefits_disability_pension, {unfilled: 0, yes: 1, no: 2}, prefix: true
+  enum :receiving_benefits_workers_compensation, {unfilled: 0, yes: 1, no: 2}, prefix: true
+  enum :receiving_benefits_insurance_payments, {unfilled: 0, yes: 1, no: 2}, prefix: true
+  enum :receiving_benefits_other, {unfilled: 0, yes: 1, no: 2}, prefix: true
+  enum :receiving_benefits_none, {unfilled: 0, yes: 1, no: 2}, prefix: true
   attr_writer :birth_date_year, :birth_date_month, :birth_date_day
   attr_writer :pregnancy_due_date_year, :pregnancy_due_date_month, :pregnancy_due_date_day
   normalizes :phone_number, with: ->(value) { Phonelib.parse(value, "US").national }
@@ -48,6 +56,20 @@ class Screener < ApplicationRecord
     validates :has_unemployment_benefits, inclusion: {in: %w[yes no], message: I18n.t("validations.must_answer_yes_or_no")}
   end
 
+  with_context :disability_benefits do
+    validates :receiving_benefits_none, inclusion: {in: %w[unfilled no]},
+      if: -> {
+        receiving_benefits_ssdi_yes? ||
+          receiving_benefits_ssi_yes? ||
+          receiving_benefits_veterans_disability_yes? ||
+          receiving_benefits_disability_pension_yes? ||
+          receiving_benefits_workers_compensation_yes? ||
+          receiving_benefits_insurance_payments_yes? ||
+          receiving_benefits_other_yes?
+      }
+    validates :receiving_benefits_write_in, absence: true, if: -> { receiving_benefits_other_no? }
+  end
+
   before_save do
     if is_pregnant_no?
       self.pregnancy_due_date = nil
diff --git a/app/views/caring_for_someone/edit.html.erb b/app/views/caring_for_someone/edit.html.erb
@@ -16,7 +16,7 @@
 </div>
 
 <%= form_with model: @model, url: { action: :update }, data: { turbo: false }, local: true, method: "put", builder: Cfa::Styleguide::CfaFormBuilder do |f| %>
-  <div class="checkbox-group">
+  <div class="checkbox-group spacing-below-60">
     <%= f.cfa_checkbox(:caring_for_child_under_6, t("views.caring_for_someone.edit.child_under_6"), options: { checked_value: "yes", unchecked_value: "no" }) %>
     <%= f.cfa_checkbox(:caring_for_disabled_or_ill_person, t("views.caring_for_someone.edit.disabled_or_ill_person"), options: { checked_value: "yes", unchecked_value: "no" }) %>
     <%= f.cfa_checkbox(:caring_for_no_one, t("general.none_of_the_above"), options: { checked_value: "yes", unchecked_value: "no", id: "none_of_the_above"}) %>
diff --git a/app/views/disability_benefits/edit.html.erb b/app/views/disability_benefits/edit.html.erb
@@ -0,0 +1,27 @@
+<% content_for :question_title, t("views.disability_benefits.edit.title") %>
+
+<h1 class="h2"><%= content_for(:question_title) %></h1>
+
+<%= form_with model: @model, url: { action: :update }, data: { turbo: false }, local: true, method: "put", builder: Cfa::Styleguide::CfaFormBuilder do |f| %>
+  <%= f.hidden_field :receiving_benefits_write_in, value: "" %>
+  <div class="checkbox-group">
+    <div class="question-with-follow-up">
+      <div class="question-with-follow-up__question">
+        <%= f.cfa_checkbox(:receiving_benefits_ssdi, t("views.disability_benefits.edit.receiving_benefits_ssdi"), options: { checked_value: "yes", unchecked_value: "no" }) %>
+        <%= f.cfa_checkbox(:receiving_benefits_ssi, t("views.disability_benefits.edit.receiving_benefits_ssi"), options: { checked_value: "yes", unchecked_value: "no" }) %>
+        <%= f.cfa_checkbox(:receiving_benefits_veterans_disability, t("views.disability_benefits.edit.receiving_benefits_veterans_disability"), options: { checked_value: "yes", unchecked_value: "no" }) %>
+        <%= f.cfa_checkbox(:receiving_benefits_disability_pension, t("views.disability_benefits.edit.receiving_benefits_disability_pension"), options: { checked_value: "yes", unchecked_value: "no" }) %>
+        <%= f.cfa_checkbox(:receiving_benefits_workers_compensation, t("views.disability_benefits.edit.receiving_benefits_workers_compensation"), options: { checked_value: "yes", unchecked_value: "no" }) %>
+        <%= f.cfa_checkbox(:receiving_benefits_insurance_payments, t("views.disability_benefits.edit.receiving_benefits_insurance_payments"), options: { checked_value: "yes", unchecked_value: "no" }) %>
+        <%= f.cfa_checkbox(:receiving_benefits_other, t("general.other"), options: { checked_value: "yes", unchecked_value: "no", "data-follow-up": "#write-in" }) %>
+        <%= f.cfa_checkbox(:receiving_benefits_none, t("general.none_of_the_above"), options: { checked_value: "yes", unchecked_value: "no", id: "none_of_the_above" }) %>
+      </div>
+
+      <div class="question-with-follow-up__follow-up" id="write-in">
+        <%= f.cfa_input_field(:receiving_benefits_write_in, t("views.disability_benefits.edit.receiving_benefits_write_in")) %>
+      </div>
+    </div>
+  </div>
+
+  <%= f.submit t("general.continue"), class: "button button--primary" %>
+<% end %>
diff --git a/config/locales/en.yml b/config/locales/en.yml
@@ -10,6 +10,7 @@ en:
       content_html: This benefits application is a service built by <a target="_blank" rel="noopener nofollow" href="https://www.codeforamerica.org">Code for America</a> in partnership on behalf of the people of the United States.
     negative: 'No'
     none_of_the_above: None of the above
+    other: Other
     privacy_policy: Privacy Policy
     progress_bar_text_html: Section %{step} of 3 <strong>%{section_name}</strong>
     work_rules: Work rules
@@ -32,6 +33,16 @@ en:
         help_text_html: The person <strong>does not</strong> need to live with you.
         notice_text: Caring for someone means you regularly do things like make food, help with daily tasks like bathing and personal care, run errands for them, and monitor their health and wellbeing.
         title: Do you care for someone who cannot care for themselves?
+    disability_benefits:
+      edit:
+        receiving_benefits_disability_pension: Disability pension
+        receiving_benefits_insurance_payments: Disability insurance payments
+        receiving_benefits_ssdi: Social Security Disability Benefits (SSDI)
+        receiving_benefits_ssi: SSI (Supplemental Security Income)
+        receiving_benefits_veterans_disability: Veteran’s disability
+        receiving_benefits_workers_compensation: Worker’s compensation
+        receiving_benefits_write_in: If you don’t see your benefit above, please write it here.
+        title: Are you receiving any of these disability benefits?
     has_child:
       edit:
         help_text: You do not have to be the child’s parent, be responsible for the child, or be their legal guardian. If you live with the child, select “Yes,” even if they aren’t included in your SNAP case.
diff --git a/config/locales/es.yml b/config/locales/es.yml
@@ -10,6 +10,7 @@ es:
       content_html: This benefits application is a service built by <a target="_blank" rel="noopener nofollow" href="https://www.codeforamerica.org">Code for America</a> in partnership on behalf of the people of the United States.
     negative: 'No'
     none_of_the_above: Ninguna de las anteriores
+    other: Otra
     privacy_policy: Privacy Policy
     progress_bar_text_html: Sección %{step} de 3 <strong>%{section_name}</strong>
     work_rules: Reglas de trabajo
@@ -32,6 +33,16 @@ es:
         help_text_html: "<strong>No es necesario</strong> que la persona viva con usted."
         notice_text: Cuidar a alguien significa que usted regularmente hace cosas como preparar comida, ayudar con tareas diarias como bañarse o asearse, hacer mandados para esa persona y estar pendiente de su salud y bienestar.
         title: "¿Cuida usted a alguien que no puede cuidar de sí mismo?"
+    disability_benefits:
+      edit:
+        receiving_benefits_disability_pension: Pensión por discapacidad
+        receiving_benefits_insurance_payments: Pagos de seguro por discapacidad
+        receiving_benefits_ssdi: Seguro Social
+        receiving_benefits_ssi: SSI (Ingreso Suplementario del Seguro Social)
+        receiving_benefits_veterans_disability: Beneficios de discapacidad para veteranos
+        receiving_benefits_workers_compensation: Compensación para trabajadores
+        receiving_benefits_write_in: Si el beneficio que recibe no está en la lista, escríbalo aquí.
+        title: "¿Recibe alguno de estos beneficios por discapacidad?"
     has_child:
       edit:
         help_text: You do not have to be the child’s parent, be responsible for the child, or be their legal guardian. If you live with the child, select “Yes,” even if they aren’t included in your SNAP case.
diff --git a/db/migrate/20260109221241_add_disability_benefits_columns_to_screeners.rb b/db/migrate/20260109221241_add_disability_benefits_columns_to_screeners.rb
@@ -0,0 +1,13 @@
+class AddDisabilityBenefitsColumnsToScreeners < ActiveRecord::Migration[8.1]
+  def change
+    add_column :screeners, :receiving_benefits_ssdi, :integer, null: false, default: 0
+    add_column :screeners, :receiving_benefits_ssi, :integer, null: false, default: 0
+    add_column :screeners, :receiving_benefits_veterans_disability, :integer, null: false, default: 0
+    add_column :screeners, :receiving_benefits_disability_pension, :integer, null: false, default: 0
+    add_column :screeners, :receiving_benefits_workers_compensation, :integer, null: false, default: 0
+    add_column :screeners, :receiving_benefits_insurance_payments, :integer, null: false, default: 0
+    add_column :screeners, :receiving_benefits_other, :integer, null: false, default: 0
+    add_column :screeners, :receiving_benefits_none, :integer, null: false, default: 0
+    add_column :screeners, :receiving_benefits_write_in, :string
+  end
+end
diff --git a/db/schema.rb b/db/schema.rb
diff --git a/spec/controllers/disability_benefits_controller_spec.rb b/spec/controllers/disability_benefits_controller_spec.rb
@@ -0,0 +1,33 @@
+require "rails_helper"
+
+RSpec.describe DisabilityBenefitsController, type: :controller do
+  describe "#update" do
+    it "persists the values to the current screener" do
+      screener = create(:screener)
+
+      params = {
+        receiving_benefits_ssdi: "no",
+        receiving_benefits_ssi: "yes",
+        receiving_benefits_veterans_disability: "no",
+        receiving_benefits_disability_pension: "no",
+        receiving_benefits_workers_compensation: "no",
+        receiving_benefits_insurance_payments: "no",
+        receiving_benefits_other: "yes",
+        receiving_benefits_none: "no",
+        receiving_benefits_write_in: "boop"
+      }
+
+      post :update, params: {screener: params}
+      screener.reload
+      expect(screener.receiving_benefits_ssdi).to eq "no"
+      expect(screener.receiving_benefits_ssi).to eq "yes"
+      expect(screener.receiving_benefits_veterans_disability).to eq "no"
+      expect(screener.receiving_benefits_disability_pension).to eq "no"
+      expect(screener.receiving_benefits_workers_compensation).to eq "no"
+      expect(screener.receiving_benefits_insurance_payments).to eq "no"
+      expect(screener.receiving_benefits_other).to eq "yes"
+      expect(screener.receiving_benefits_none).to eq "no"
+      expect(screener.receiving_benefits_write_in).to eq "boop"
+    end
+  end
+end
diff --git a/spec/features/flow_spec.rb b/spec/features/flow_spec.rb
@@ -34,6 +34,10 @@
     expect(page).to have_selector("h1", text: I18n.t("views.has_unemployment_benefits.edit.title"))
     click_on I18n.t("general.negative")
 
+    expect(page).to have_selector("h1", text: I18n.t("views.disability_benefits.edit.title"))
+    check I18n.t("general.none_of_the_above")
+    click_on I18n.t("general.continue")
+
     expect(page).to have_selector("h1", text: I18n.t("views.basic_info_milestone.edit.title"))
     click_on I18n.t("general.continue")
 
diff --git a/spec/models/screener_spec.rb b/spec/models/screener_spec.rb
@@ -92,6 +92,52 @@
         expect(screener.errors[:caring_for_no_one]).to be_present
       end
     end
+
+    context "with_context :disability_benefits" do
+      it "cannot choose a benefit and 'none of the above'" do
+        screener = Screener.new(
+          receiving_benefits_ssdi: "no",
+          receiving_benefits_ssi: "no",
+          receiving_benefits_veterans_disability: "yes",
+          receiving_benefits_disability_pension: "no",
+          receiving_benefits_workers_compensation: "no",
+          receiving_benefits_insurance_payments: "yes",
+          receiving_benefits_other: "no",
+          receiving_benefits_none: "yes"
+        )
+
+        screener.valid?(:disability_benefits)
+        expect(screener.errors[:receiving_benefits_none]).to be_present
+
+        # valid if receiving_benefits_none is "no"
+        screener.assign_attributes(receiving_benefits_none: "no")
+        expect(screener.valid?(:disability_benefits)).to eq true
+
+        # valid if everything but receiving_benefits_none is "no"
+        screener.assign_attributes(
+          receiving_benefits_veterans_disability: "no",
+          receiving_benefits_insurance_payments: "no",
+          receiving_benefits_none: "yes"
+        )
+        expect(screener.valid?(:disability_benefits)).to eq true
+      end
+
+      it "can only have a write-in answer if 'other' is checked" do
+        screener = Screener.new(
+          receiving_benefits_other: "no",
+          receiving_benefits_write_in: "a different benefit"
+        )
+
+        screener.valid?(:disability_benefits)
+        expect(screener.errors[:receiving_benefits_write_in]).to be_present
+
+        screener.assign_attributes(
+          receiving_benefits_other: "yes",
+          receiving_benefits_write_in: "a different benefit"
+        )
+        expect(screener.valid?(:disability_benefits)).to eq true
+      end
+    end
   end
 
   describe "before_save" do

Original file line number	Diff line number	Diff line change
`@@ -69,8 +69,6 @@ i.no-button-cross {`
`69`	`69`	`}`
`70`	`70`
`71`	`71`	`.checkbox-group {`
`72`		`- margin-bottom: 6rem;`
`73`		`-`
`74`	`72`	`.input-group {`
`75`	`73`	`margin-bottom: 0;`
`76`	`74`	`}`