Fix Ruleset module namespacing

Author: baonguyenNava

app/models/strata/medicaid_ruleset.rb …/models/strata/rules/medicaid_ruleset.rbapp/models/strata/medicaid_ruleset.rb renamed to app/models/strata/rules/medicaid_ruleset.rb app/models/strata/medicaid_ruleset.rb renamed to app/models/strata/rules/medicaid_ruleset.rb

@@ -1,7 +1,8 @@
 module Strata
-  # Implements eligibility rules for Medicaid benefits.
-  # Handles age calculations, residency verification, and income-based qualification.
-  class MedicaidRuleset
+  module Rules
+    # Implements eligibility rules for Medicaid benefits.
+    # Handles age calculations, residency verification, and income-based qualification.
+    class MedicaidRuleset
       def medicaid_eligibility(state_of_residence, age_over_65, modified_adjusted_gross_income)
         age_over_65 && modified_adjusted_gross_income < 50000
       end
@@ -29,5 +30,6 @@ def modified_adjusted_gross_income(annual_income, deductions)
         return 0 if annual_income < deductions
         annual_income - deductions
       end
+    end
   end
 end

app/models/strata/paid_leave_ruleset.rb …odels/strata/rules/paid_leave_ruleset.rbapp/models/strata/paid_leave_ruleset.rb renamed to app/models/strata/rules/paid_leave_ruleset.rb app/models/strata/paid_leave_ruleset.rb renamed to app/models/strata/rules/paid_leave_ruleset.rb

@@ -1,7 +1,8 @@
 module Strata
-  # Implements eligibility rules for paid leave benefits based on Massachusetts PFML guidelines.
-  # Includes checks for submission timing, earnings requirements, and benefit calculations.
-  class PaidLeaveRuleset
+  module Rules
+    # Implements eligibility rules for paid leave benefits based on Massachusetts PFML guidelines.
+    # Includes checks for submission timing, earnings requirements, and benefit calculations.
+    class PaidLeaveRuleset
       def submitted_within_60_days_of_leave_start(submitted_at, leave_starts_on)
         return nil if submitted_at.nil? || leave_starts_on.nil?
 
@@ -44,5 +45,6 @@ def base_period
 
       def individual_average_weekly_wage
       end
+    end
   end
 end

spec/models/strata/medicaid_ruleset_spec.rb

@@ -0,0 +1,82 @@
+require 'rails_helper'
+
+RSpec.describe Strata::Rules::MedicaidRuleset do
+    evaluated_on = Date.new(2025, 5, 28) # Freeze time for testing
+    let(:rules) { described_class.new }
+
+    describe '#medicaid_eligibility' do
+      [
+        [ "when applicant is over 65 with qualifying income", "AK", true, 35000, true ],
+        [ "when applicant is over 65 but income is too high", "AK", true, 55000, false ],
+        [ "when applicant is under 65", "AK", false, 35000, false ],
+        [ "when state of residence is missing", nil, true, 35000, true ]
+      ].each do |context_description, state, over_65, income, expected|
+        context context_description do
+          it 'determines correct medicaid eligibility' do
+            expect(rules.medicaid_eligibility(state, over_65, income)).to be expected
+          end
+        end
+      end
+    end
+
+    describe '#age' do
+      [
+        [ "calculates age correctly", Date.new(1954, 1, 1), evaluated_on, 71 ], # 71 years old in 2025
+        [ "calculates age on birthday", Date.new(1954, 1, 1), Date.new(2004, 1, 1), 50 ],
+        [ "calculates age for age on leap year birthday", Date.new(2000, 2, 29), Date.new(2004, 2, 29), 4 ],
+        [ "calculates age for age on leap year birthday", Date.new(2000, 2, 29), Date.new(2005, 3, 1), 5 ],
+        [ "handles leap years", Date.new(2000, 5, 29), Date.new(2025, 5, 28), 24 ],
+        [ "returns nil when date of birth is nil", nil, evaluated_on, nil ],
+        [ "handles birth dates near evaluation date", Date.new(evaluated_on.year - 65, evaluated_on.month, evaluated_on.day + 1), evaluated_on, 64 ],
+        [ "returns nil when date of birth is after evaluation date", evaluated_on + 1.day, evaluated_on, nil ],
+        [ "returns nil when evaluated_on is nil", Date.new(1954, 1, 1), nil, nil ]
+      ].each do |description, birth_date, evaluated_on, expected|
+        it description do
+          expect(rules.age(birth_date, evaluated_on)).to expected.nil? ? be_nil : eq(expected)
+        end
+      end
+    end
+
+    describe '#age_over_65' do
+      [
+        [ "returns true when age is over 65", 71, true ],
+        [ "returns true when age is exactly 65", 65, true ],
+        [ "returns false when age is under 65", 64, false ],
+        [ "returns nil when age is nil", nil, nil ]
+      ].each do |description, age, expected|
+        it description do
+          expect(rules.age_over_65(age)).to be expected
+        end
+      end
+    end
+
+    describe '#state_of_residence' do
+      [
+        [ "returns state from residential address", Strata::Address.new({
+            street_line_1: "123 A St",
+            city: "Anchorage",
+            state: "AK",
+            zip_code: "12345"
+          }), "AK" ],
+        [ "returns nil when address is nil", nil, nil ]
+      ].each do |description, addr, expected|
+        it description do
+          expect(rules.state_of_residence(addr)).to expected.nil? ? be_nil : eq(expected)
+        end
+      end
+    end
+
+    describe '#modified_adjusted_gross_income' do
+      [
+        [ "subtracts deductions from annual income", 40000, 5000, 35000 ],
+        [ "calculates income without deductions", 40000, 0, 40000 ],
+        [ "calculates income where deductions exceeds income", 40000, 45000, 0 ],
+        [ "returns nil if income nil", nil, 5000, nil ],
+        [ "returns nil if deductions nil", 40000, nil, nil ]
+      ].each do |description, income, deductions, expected|
+        it description do
+          expect(rules.modified_adjusted_gross_income(income, deductions)).to eq(expected)
+        end
+      end
+    end
+end

spec/models/strata/paid_leave_ruleset_spec.rb

@@ -0,0 +1,24 @@
+require 'rails_helper'
+
+
+RSpec.describe Strata::Rules::PaidLeaveRuleset do
+  base_date = Date.new(2025, 7, 1)
+  let(:rules) { described_class.new }
+
+  describe '#submitted_within_60_days_of_leave_start' do
+    [
+      [ 'submitted exactly 60 days before leave start', base_date, (base_date - 60.days).beginning_of_day, true ],
+      [ 'submitted 30 days before leave start', base_date, (base_date - 30.days).beginning_of_day, true ],
+      [ 'submitted 61 days before leave start', base_date, (base_date - 61.days).beginning_of_day, false ],
+      [ 'submitted after leave start', base_date, base_date.to_time + 1.day, true ],
+      [ 'submitted_at is nil', base_date, nil, nil ],
+      [ 'leave_starts_on is nil', nil, base_date, nil ]
+    ].each do |description, leave_starts_on, submitted_at, expected|
+      context "when #{description}" do
+        it "returns #{expected}" do
+          expect(rules.submitted_within_60_days_of_leave_start(submitted_at, leave_starts_on)).to eq(expected)
+        end
+      end
+    end
+  end
+end