taxcalc 5.1.0__py3-none-any.whl → 5.3.0__py3-none-any.whl

taxcalc/tests/test_policy.py

@@ -54,14 +54,15 @@ def test_incorrect_implement_reform_usage():
     with pytest.raises(paramtools.ValidationError):
         pol.implement_reform([])
     with pytest.raises(paramtools.ValidationError):
-        pol.implement_reform({2099: {'II_em': 99000}})
+        pol.implement_reform({2099: {"II_em": 99000}})
     pol.set_year(2019)
     with pytest.raises(paramtools.ValidationError):
-        pol.implement_reform({2018: {'II_em': 99000}})
+        pol.implement_reform({2018: {"II_em": 99000}})
     with pytest.raises(paramtools.ValidationError):
-        pol.implement_reform({2020: {'II_em': -1000}})
+        pol.implement_reform({2020: {"II_em": -1000}})
 
 
+@pytest.mark.local
 def test_json_reform_url():
     """
     Test reading a JSON reform from a URL. Results from the URL are expected
@@ -89,8 +90,8 @@ def test_json_reform_url():
         }
     }
     """
-    reform_url = ('https://raw.githubusercontent.com/PSLmodels/'
-                  'Tax-Calculator/master/taxcalc/reforms/ptaxes0.json')
+    reform_url = ("https://raw.githubusercontent.com/PSLmodels/"
+                  "Tax-Calculator/master/taxcalc/reforms/ptaxes0.json")
     params_str = Policy.read_json_reform(reform_str)
     params_url = Policy.read_json_reform(reform_url)
     assert params_str == params_url
@@ -191,7 +192,7 @@ def test_constant_inflation_rate_with_reform():
     fyr = 2034
     ryr = fyr - 1
     reform = {
-        'II_em': {(ryr - 3): 1000,  # to avoid divide-by-zero under TCJA
+        "II_em": {(ryr - 3): 1000,  # to avoid divide-by-zero under TCJA
                   ryr: 20000}
     }
     pol.implement_reform(reform)
@@ -217,7 +218,7 @@ def test_variable_inflation_rate_with_reform():
     assert pol._II_em[2013 - syr] == 3900
     # implement reform in 2020 which is two years before the last year, 2022
     reform = {
-        'II_em': {2018: 1000,  # to avoid divide-by-zero under TCJA
+        "II_em": {2018: 1000,  # to avoid divide-by-zero under TCJA
                   2020: 20000}
     }
     pol.implement_reform(reform)
@@ -257,15 +258,14 @@ def test_multi_year_reform():
         wfactor[syr + i] = 1.0 + wratelist[i]
     # specify multi-year reform using a param:year:value-fomatted dictionary
     reform = {
-        'SS_Earnings_c': {2016: 300000,
+        "SS_Earnings_c": {2016: 300000,
                           2017: 500000,
                           2019: 700000},
-        'SS_Earnings_c-indexed': {2017: False,
+        "SS_Earnings_c-indexed": {2017: False,
                                   2019: True},
-        'CTC_c': {2015: 2000},
-        'EITC_c': {2016: [900, 5000, 8000, 9000],
+        "EITC_c": {2016: [900, 5000, 8000, 9000],
                    2019: [1200, 7000, 10000, 12000]},
-        'II_em': {2016: 7000,
+        "II_em": {2016: 7000,
                   2019: 9000}
     }
     # implement multi-year reform
@@ -280,42 +280,16 @@ def test_multi_year_reform():
     check_eitc_c(pol, reform, ifactor)
     check_ii_em(pol, reform, ifactor)
     check_ss_earnings_c(pol, reform, wfactor)
-    check_ctc_c(pol, reform)
     # end of test_multi_year_reform with the check_* functions below:
 
 
-def check_ctc_c(ppo, reform):
-    """
-    Compare actual and expected _CTC_c parameter values
-    generated by the test_multi_year_reform() function above.
-    Ensure that future-year values in policy_current_law.json
-    are overwritten by reform.
-    """
-    actual = {}
-    arr = getattr(ppo, '_CTC_c')
-    for i in range(0, ppo.num_years):
-        actual[ppo.start_year + i] = arr[i]
-    assert actual[2013] == 1000
-    assert actual[2014] == 1000
-    e2015 = reform['CTC_c'][2015]
-    assert actual[2015] == e2015
-    e2016 = actual[2015]
-    assert actual[2016] == e2016
-    e2017 = actual[2016]
-    assert actual[2017] == e2017
-    e2018 = actual[2017]
-    assert actual[2018] == e2018
-    e2019 = actual[2018]
-    assert actual[2019] == e2019
-
-
 def check_eitc_c(ppo, reform, ifactor):
     """
     Compare actual and expected _EITC_c parameter values
     generated by the test_multi_year_reform() function above.
     """
     actual = {}
-    arr = getattr(ppo, '_EITC_c')
+    arr = getattr(ppo, "_EITC_c")
     alen = len(arr[0])
     for i in range(0, ppo.num_years):
         actual[ppo.start_year + i] = arr[i]
@@ -325,13 +299,13 @@ def check_eitc_c(ppo, reform, ifactor):
                        atol=0.01, rtol=0.0)
     assert np.allclose(actual[2015], [503, 3359, 5548, 6242],
                        atol=0.01, rtol=0.0)
-    e2016 = reform['EITC_c'][2016]
+    e2016 = reform["EITC_c"][2016]
     assert np.allclose(actual[2016], e2016, atol=0.01, rtol=0.0)
     e2017 = [ifactor[2016] * actual[2016][j] for j in range(0, alen)]
     assert np.allclose(actual[2017], e2017, atol=0.01, rtol=0.0)
     e2018 = [ifactor[2017] * actual[2017][j] for j in range(0, alen)]
     assert np.allclose(actual[2018], e2018, atol=0.01, rtol=0.0)
-    e2019 = reform['EITC_c'][2019]
+    e2019 = reform["EITC_c"][2019]
     assert np.allclose(actual[2019], e2019, atol=0.01, rtol=0.0)
     e2020 = [ifactor[2019] * actual[2019][j] for j in range(0, alen)]
     assert np.allclose(actual[2020], e2020, atol=0.01, rtol=0.0)
@@ -347,19 +321,19 @@ def check_ii_em(ppo, reform, ifactor):
     generated by the test_multi_year_reform() function above.
     """
     actual = {}
-    arr = getattr(ppo, '_II_em')
+    arr = getattr(ppo, "_II_em")
     for i in range(0, ppo.num_years):
         actual[ppo.start_year + i] = arr[i]
     assert actual[2013] == 3900
     assert actual[2014] == 3950
     assert actual[2015] == 4000
-    e2016 = reform['II_em'][2016]
+    e2016 = reform["II_em"][2016]
     assert actual[2016] == e2016
     e2017 = ifactor[2016] * actual[2016]
     assert np.allclose([actual[2017]], [e2017], atol=0.01, rtol=0.0)
     e2018 = ifactor[2017] * actual[2017]
     assert np.allclose([actual[2018]], [e2018], atol=0.01, rtol=0.0)
-    e2019 = reform['II_em'][2019]
+    e2019 = reform["II_em"][2019]
     assert actual[2019] == e2019
     e2020 = ifactor[2019] * actual[2019]
     assert np.allclose([actual[2020]], [e2020], atol=0.01, rtol=0.0)
@@ -375,19 +349,19 @@ def check_ss_earnings_c(ppo, reform, wfactor):
     generated by the test_multi_year_reform() function above.
     """
     actual = {}
-    arr = getattr(ppo, '_SS_Earnings_c')
+    arr = getattr(ppo, "_SS_Earnings_c")
     for i in range(0, ppo.num_years):
         actual[ppo.start_year + i] = arr[i]
     assert actual[2013] == 113700
     assert actual[2014] == 117000
     assert actual[2015] == 118500
-    e2016 = reform['SS_Earnings_c'][2016]
+    e2016 = reform["SS_Earnings_c"][2016]
     assert actual[2016] == e2016
-    e2017 = reform['SS_Earnings_c'][2017]
+    e2017 = reform["SS_Earnings_c"][2017]
     assert actual[2017] == e2017
     e2018 = actual[2017]  # no indexing after 2017
     assert actual[2018] == e2018
-    e2019 = reform['SS_Earnings_c'][2019]
+    e2019 = reform["SS_Earnings_c"][2019]
     assert actual[2019] == e2019
     e2020 = wfactor[2019] * actual[2019]  # indexing after 2019
     assert np.allclose([actual[2020]], [e2020], atol=0.01, rtol=0.0)
@@ -410,7 +384,7 @@ def test_implement_reform_raises_on_no_year():
     """
     Test that implement_reform raises error for missing year.
     """
-    reform = {'STD_Aged': [1400, 1200, 1400, 1400, 1400]}
+    reform = {"STD_Aged": [1400, 1200, 1400, 1400, 1400]}
     ppo = Policy()
     with pytest.raises(paramtools.ValidationError):
         ppo.implement_reform(reform)
@@ -421,7 +395,7 @@ def test_implement_reform_raises_on_early_year():
     Test that implement_reform raises error for early year.
     """
     ppo = Policy()
-    reform = {'STD_Aged': {2010: [1400, 1100, 1100, 1400, 1400]}}
+    reform = {"STD_Aged": {2010: [1400, 1100, 1100, 1400, 1400]}}
     with pytest.raises(paramtools.ValidationError):
         ppo.implement_reform(reform)
 
@@ -431,7 +405,7 @@ def test_reform_with_default_indexed():
     Test that implement_reform indexes after first reform year.
     """
     ppo = Policy()
-    reform = {'II_em': {2015: 4300}}
+    reform = {"II_em": {2015: 4300}}
     ppo.implement_reform(reform)
     # II_em has a default indexing status of true, so
     # in 2016 its value should be greater than 4300
@@ -444,7 +418,7 @@ def test_reform_makes_no_changes_before_year():
     Test that implement_reform makes no changes before first reform year.
     """
     ppo = Policy()
-    reform = {'II_em': {2015: 4400}, 'II_em-indexed': {2015: True}}
+    reform = {"II_em": {2015: 4400}, "II_em-indexed": {2015: True}}
     ppo.implement_reform(reform)
     ppo.set_year(2015)
     assert np.allclose(ppo._II_em[:3], np.array([3900, 3950, 4400]),
@@ -545,7 +519,7 @@ def test_pop_the_cap_reform():
     assert mte[2015 - syr] == 118500
     assert mte[2016 - syr] == 118500
     # specify a "pop the cap" reform that eliminates MTE cap in 2016
-    reform = {'SS_Earnings_c': {2016: 9e99}}
+    reform = {"SS_Earnings_c": {2016: 9e99}}
     ppo.implement_reform(reform)
     mte = ppo._SS_Earnings_c
     assert mte[2015 - syr] == 118500
@@ -561,13 +535,13 @@ def test_order_of_indexing_and_level_reforms():
     # specify two reforms that raises the MTE and stops its indexing in 2015
     reforms = [
         {
-            'SS_Earnings_c': {2015: 500000},
-            'SS_Earnings_c-indexed': {2015: False}
+            "SS_Earnings_c": {2015: 500000},
+            "SS_Earnings_c-indexed": {2015: False}
         },
         # now reverse the order of the two reform provisions
         {
-            'SS_Earnings_c-indexed': {2015: False},
-            'SS_Earnings_c': {2015: 500000}
+            "SS_Earnings_c-indexed": {2015: False},
+            "SS_Earnings_c": {2015: 500000}
         }
     ]
     # specify two Policy objects
@@ -598,12 +572,12 @@ def test_misspecified_reform_dictionary():
     """
     # specify apparently the same reform in two different ways, forgetting
     # that Python dictionaries have unique keys
-    reform1 = {'II_em': {2019: 1000, 2020: 2000}}
+    reform1 = {"II_em": {2019: 1000, 2020: 2000}}
     # pylint: disable=duplicate-key
-    reform2 = {'II_em': {2019: 1000}, 'II_em': {2020: 2000}}
+    reform2 = {"II_em": {2019: 1000}, "II_em": {2020: 2000}}
     # these two reform dictionaries are not the same: the second
-    # 'II_em' key value for 2020 in reform2 OVERWRITES and REPLACES
-    # the first 'II_em' key value for 2019 in reform2
+    # "II_em" key value for 2020 in reform2 OVERWRITES and REPLACES
+    # the first "II_em" key value for 2019 in reform2
     assert reform1 != reform2
 
 
@@ -622,114 +596,114 @@ def test_section_titles(tests_path):
         for line in md_text.splitlines():
             # This is shown as an empty case in current law policy and
             # validation.
-            if line.startswith('## Other Parameters (not in Tax-Brain webapp'):
-                sdict[''] = {}
-                sdict[''][''] = 0
+            if line.startswith("## Other Parameters (not in Tax-Brain webapp"):
+                sdict[""] = {}
+                sdict[""][""] = 0
                 continue
-            sec2line = line.startswith('### ')
-            sec1line = line.startswith('## ')
+            sec2line = line.startswith("### ")
+            sec1line = line.startswith("## ")
             # Create outer-layer dictionary entry for sec1.
             if sec1line:
-                sec1 = line.replace('##', '', 1).strip()
+                sec1 = line.replace("##", "", 1).strip()
                 sdict[sec1] = {}
             # Create inner dictionary entry for sec1-sec2.
             # Note that sec1 will have been defined from a previous loop.
             if sec2line:
-                sec2 = line.replace('###', '', 1).strip()
+                sec2 = line.replace("###", "", 1).strip()
                 sdict[sec1][sec2] = 0
         return sdict
     # begin main logic of test_section_titles
     # specify expected section titles ordered as on the Tax-Brain webapp
-    cgqd_tax_same = ('Tax All Capital Gains And Dividends The Same '
-                     'As Regular Taxable Income')
+    cgqd_tax_same = ("Tax All Capital Gains And Dividends The Same "
+                     "As Regular Taxable Income")
     valid_dict = {
-        '': {  # empty section_1 implies parameter not displayed in Tax-Brain
-            '': 0
+        "": {  # empty section_1 implies parameter not displayed in Tax-Brain
+            "": 0
         },
-        'Parameter Indexing': {
-            'Offsets': 0
+        "Parameter Indexing": {
+            "Offsets": 0
         },
-        'Payroll Taxes': {
-            'Social Security FICA': 0,
-            'Medicare FICA': 0,
-            'Additional Medicare FICA': 0
+        "Payroll Taxes": {
+            "Social Security FICA": 0,
+            "Medicare FICA": 0,
+            "Additional Medicare FICA": 0
         },
-        'Social Security Taxability': {
-            'Social Security Benefit Taxability': 0,
+        "Social Security Taxability": {
+            "Social Security Benefit Taxability": 0,
         },
-        'Above The Line Deductions': {
-            'Misc. Adjustment Haircuts': 0,
-            'Misc. Exclusions': 0,
-            'Child And Elderly Care': 0
+        "Above The Line Deductions": {
+            "Misc. Adjustment Haircuts": 0,
+            "Misc. Exclusions": 0,
+            "Child And Elderly Care": 0
         },
-        'Personal Exemptions': {
-            'Personal And Dependent Exemption Amount': 0,
-            # 'Personal Exemption Phaseout Starting Income': 0,
-            'Personal Exemption Phaseout Rate': 0,
-            'Repeal for Dependents Under Age 18': 0
+        "Personal Exemptions": {
+            "Personal And Dependent Exemption Amount": 0,
+            # "Personal Exemption Phaseout Starting Income": 0,
+            "Personal Exemption Phaseout Rate": 0,
+            "Repeal for Dependents Under Age 18": 0
         },
-        'Standard Deduction': {
-            'Standard Deduction Amount': 0,
-            'Additional Standard Deduction For Blind And Aged': 0
-            # 'Standard Deduction For Dependents': 0
+        "Standard Deduction": {
+            "Standard Deduction Amount": 0,
+            "Additional Standard Deduction For Blind And Aged": 0
+            # "Standard Deduction For Dependents": 0
         },
-        'Nonrefundable Credits': {
-            'Misc. Credit Limits': 0,
-            'Child And Dependent Care': 0,
-            'Personal Nonrefundable Credit': 0
+        "Nonrefundable Credits": {
+            "Misc. Credit Limits": 0,
+            "Child And Dependent Care": 0,
+            "Personal Nonrefundable Credit": 0
         },
-        'Child/Dependent Credits': {
-            'Child Tax Credit': 0,
-            'Additional Child Tax Credit': 0,
-            'Other Dependent Tax Credit': 0
+        "Child/Dependent Credits": {
+            "Child Tax Credit": 0,
+            "Additional Child Tax Credit": 0,
+            "Other Dependent Tax Credit": 0
         },
-        'Itemized Deductions': {
-            'Medical Expenses': 0,
-            'State And Local Income And Sales Taxes': 0,
-            'State, Local, And Foreign Real Estate Taxes': 0,
-            'State And Local Taxes And Real Estate Taxes': 0,
-            'Interest Paid': 0,
-            'Charity': 0,
-            'Casualty': 0,
-            'Miscellaneous': 0,
-            'Itemized Deduction Limitation': 0,  # Pease
-            'Ceiling On The Amount Of Itemized Deductions Allowed': 0  # ID_c
+        "Itemized Deductions": {
+            "Medical Expenses": 0,
+            "State And Local Income And Sales Taxes": 0,
+            "State, Local, And Foreign Real Estate Taxes": 0,
+            "State And Local Taxes And Real Estate Taxes": 0,
+            "Interest Paid": 0,
+            "Charity": 0,
+            "Casualty": 0,
+            "Miscellaneous": 0,
+            "Itemized Deduction Limitation": 0,  # Pease
+            "Ceiling On The Amount Of Itemized Deductions Allowed": 0  # ID_c
         },
-        'Capital Gains And Dividends': {
-            'Regular - Long Term Capital Gains And Qualified Dividends': 0,
-            'AMT - Long Term Capital Gains And Qualified Dividends': 0,
+        "Capital Gains And Dividends": {
+            "Regular - Long Term Capital Gains And Qualified Dividends": 0,
+            "AMT - Long Term Capital Gains And Qualified Dividends": 0,
             cgqd_tax_same: 0
         },
-        'Personal Income': {
-            'Regular: Non-AMT': 0,
-            'Pass-Through': 0,
-            'Alternative Minimum Tax': 0
+        "Personal Income": {
+            "Regular: Non-AMT": 0,
+            "Pass-Through": 0,
+            "Alternative Minimum Tax": 0
         },
-        'Other Taxes': {
-            'Net Investment Income Tax': 0
+        "Other Taxes": {
+            "Net Investment Income Tax": 0
         },
-        'Refundable Credits': {
-            'Earned Income Tax Credit': 0,
-            'New Refundable Child Tax Credit': 0,
-            'Personal Refundable Credit': 0,
-            'Refundable Payroll Tax Credit': 0
+        "Refundable Credits": {
+            "Earned Income Tax Credit": 0,
+            "New Refundable Child Tax Credit": 0,
+            "Personal Refundable Credit": 0,
+            "Refundable Payroll Tax Credit": 0
         },
-        'Surtaxes': {
-            'New Minimum Tax': 0,
-            'New AGI Surtax': 0,
-            'Lump-Sum Tax': 0
+        "Surtaxes": {
+            "New Minimum Tax": 0,
+            "New AGI Surtax": 0,
+            "Lump-Sum Tax": 0
         },
-        'Universal Basic Income': {
-            'UBI Benefits': 0,
-            'UBI Taxability': 0
+        "Universal Basic Income": {
+            "UBI Benefits": 0,
+            "UBI Taxability": 0
         },
-        'Benefits': {
-            'Benefit Repeal': 0,
+        "Benefits": {
+            "Benefit Repeal": 0,
         }
     }
     # check validity of parameter section titles in policy_current_law.json
-    path = os.path.join(tests_path, '..', 'policy_current_law.json')
-    with open(path, 'r', encoding='utf-8') as clpfile:
+    path = os.path.join(tests_path, "..", "policy_current_law.json")
+    with open(path, "r", encoding="utf-8") as clpfile:
         clpdict = json.load(clpfile)
         clpdict.pop("schema", None)
     # ... make sure ever clpdict section title is in valid_dict
@@ -737,9 +711,9 @@ def test_section_titles(tests_path):
     for pname in clpdict:
         param = clpdict[pname]
         assert isinstance(param, dict)
-        sec1title = param['section_1']
+        sec1title = param["section_1"]
         assert sec1title in valid_dict
-        sec2title = param['section_2']
+        sec2title = param["section_2"]
         assert sec2title in valid_dict[sec1title]
         if sec1title not in clp_dict:
             clp_dict[sec1title] = {}
@@ -752,9 +726,9 @@ def test_section_titles(tests_path):
         for sec2title in secdict:
             assert sec2title in clp_dict[sec1title]
     # check validity of parameter section titles in docs/uguide.htmx skeleton
-    path = os.path.join(tests_path, '..', '..', 'docs', 'guide',
-                        'policy_params.md')
-    with open(path, 'r', encoding='utf-8') as md_file:
+    path = os.path.join(tests_path, "..", "..", "docs", "guide",
+                        "policy_params.md")
+    with open(path, "r", encoding="utf-8") as md_file:
         md_text = md_file.read()
     md_dict = generate_section_dictionary(md_text)
     # ... make sure every md_dict section title is in valid_dict
@@ -776,17 +750,17 @@ def test_description_punctuation(tests_path):
     Check that each description ends in a period.
     """
     # read JSON file into a dictionary
-    path = os.path.join(tests_path, '..', 'policy_current_law.json')
-    with open(path, 'r', encoding='utf-8') as jsonfile:
+    path = os.path.join(tests_path, "..", "policy_current_law.json")
+    with open(path, "r", encoding="utf-8") as jsonfile:
         dct = json.load(jsonfile)
         dct.pop("schema", None)
     all_desc_ok = True
     for param in dct.keys():
-        if not dct[param]['description'].endswith('.'):
+        if not dct[param]["description"].endswith("."):
             all_desc_ok = False
-            print('param,description=',
+            print("param,description=",
                   str(param),
-                  dct[param]['description'])
+                  dct[param]["description"])
     assert all_desc_ok
 
 
@@ -795,8 +769,8 @@ def test_get_index_rate():
     Test Parameters.get_index_rate.
     """
     pol = Policy()
-    wgrates = pol.get_index_rate('SS_Earnings_c', 2017)
-    pirates = pol.get_index_rate('II_em', 2017)
+    wgrates = pol.get_index_rate("SS_Earnings_c", 2017)
+    pirates = pol.get_index_rate("II_em", 2017)
     assert isinstance(wgrates, np.float64)
     assert wgrates == pol.wage_growth_rates(2017)
     assert pirates == pol.inflation_rates(2017)
@@ -810,11 +784,11 @@ def test_reform_with_removed_parameter(monkeypatch):
     Try to use removed parameter in a reform.
     """
     policy1 = Policy()
-    reform1 = {'FilerCredit_c': {2020: 1000}}
+    reform1 = {"FilerCredit_c": {2020: 1000}}
     with pytest.raises(paramtools.ValidationError):
         policy1.implement_reform(reform1)
     policy2 = Policy()
-    reform2 = {'FilerCredit_c-indexed': {2020: True}}
+    reform2 = {"FilerCredit_c-indexed": {2020: True}}
     with pytest.raises(paramtools.ValidationError):
         policy2.implement_reform(reform2)
 
@@ -831,7 +805,7 @@ def test_reform_with_out_of_range_error():
     Try to use out-of-range values versus other parameter values in a reform.
     """
     pol = Policy()
-    reform = {'SS_thd2': {2020: [20000, 20000, 20000, 20000, 20000]}}
+    reform = {"SS_thd2": {2020: [20000, 20000, 20000, 20000, 20000]}}
     pol.implement_reform(reform, raise_errors=False)
     assert pol.parameter_errors
 
@@ -841,12 +815,12 @@ def test_reform_with_warning():
     Try to use warned out-of-range parameter value in reform.
     """
     exp_warnings = {
-        'ID_Medical_frt': [
-            'ID_Medical_frt[year=2020] 0.05 < min 0.075 '
+        "ID_Medical_frt": [
+            "ID_Medical_frt[year=2020] 0.05 < min 0.075 "
         ]
     }
     pol = Policy()
-    reform = {'ID_Medical_frt': {2020: 0.05}}
+    reform = {"ID_Medical_frt": {2020: 0.05}}
 
     pol.implement_reform(reform, print_warnings=True)
     assert pol.warnings == exp_warnings
@@ -864,17 +838,17 @@ def test_reform_with_scalar_vector_errors():
     Test catching scalar-vector confusion.
     """
     policy1 = Policy()
-    reform1 = {'SS_thd2': {2020: 30000}}
+    reform1 = {"SS_thd2": {2020: 30000}}
     with pytest.raises(paramtools.ValidationError):
         policy1.implement_reform(reform1)
 
     policy2 = Policy()
-    reform2 = {'ID_Medical_frt': {2020: [0.08]}}
+    reform2 = {"ID_Medical_frt": {2020: [0.08]}}
     with pytest.raises(paramtools.ValidationError):
         policy2.implement_reform(reform2)
 
     policy3 = Policy()
-    reform3 = {'ID_Medical_frt': [{"year": 2020, "value": [0.08]}]}
+    reform3 = {"ID_Medical_frt": [{"year": 2020, "value": [0.08]}]}
     with pytest.raises(paramtools.ValidationError):
         policy3.adjust(reform3)
 
@@ -893,35 +867,35 @@ def test_reform_with_scalar_vector_errors():
 def test_index_offset_reform():
     """
     Test a reform that includes both a change in parameter_indexing_CPI_offset
-    and a change in a variable's indexed status in the same year.
+    and a change in a variable"s indexed status in the same year.
     """
     # create policy0 to extract inflation rates before any
     # parameter_indexing_CPI_offset
     policy0 = Policy()
-    policy0.implement_reform({'parameter_indexing_CPI_offset': {2017: 0}})
+    policy0.implement_reform({"parameter_indexing_CPI_offset": {2017: 0}})
     cpiu_rates = policy0.inflation_rates()
 
-    reform1 = {'CTC_c-indexed': {2020: True}}
+    reform1 = {"ODC_c-indexed": {2020: True}}
     policy1 = Policy()
     policy1.implement_reform(reform1)
     offset = -0.005
-    reform2 = {'CTC_c-indexed': {2020: True},
-               'parameter_indexing_CPI_offset': {2020: offset}}
+    reform2 = {"ODC_c-indexed": {2020: True},
+               "parameter_indexing_CPI_offset": {2020: offset}}
     policy2 = Policy()
-    policy2.implement_reform(reform2)  # caused T-C crash before PR#2364
-    # extract from policy1 and policy2 the parameter values of CTC_c
+    policy2.implement_reform(reform2)
+    # extract from policy1 and policy2 the parameter values of ODC_c
     pvalue1 = {}
     pvalue2 = {}
     for cyr in [2019, 2020, 2021]:
         policy1.set_year(cyr)
-        pvalue1[cyr] = policy1.CTC_c[0]
+        pvalue1[cyr] = policy1.ODC_c[0]
         policy2.set_year(cyr)
-        pvalue2[cyr] = policy2.CTC_c[0]
+        pvalue2[cyr] = policy2.ODC_c[0]
     # check that pvalue1 and pvalue2 dictionaries contain the expected values
     assert pvalue2[2019] == pvalue1[2019]
     assert pvalue2[2020] == pvalue1[2020]
     assert pvalue2[2020] == pvalue2[2019]
-    # ... indexing of CTC_c begins shows up first in 2021 parameter values
+    # ... indexing of ODC_c begins shows up first in 2021 parameter values
     assert pvalue1[2021] > pvalue1[2020]
     assert pvalue2[2021] > pvalue2[2020]
     # ... calculate expected pvalue2[2021] from inflation rates and offset
@@ -937,8 +911,8 @@ def test_cpi_offset_affect_on_prior_years():
     Test that parameter_indexing_CPI_offset does not have affect
     on inflation rates in earlier years.
     """
-    reform1 = {'parameter_indexing_CPI_offset': {2022: 0}}
-    reform2 = {'parameter_indexing_CPI_offset': {2022: -0.005}}
+    reform1 = {"parameter_indexing_CPI_offset": {2022: 0}}
+    reform2 = {"parameter_indexing_CPI_offset": {2022: -0.005}}
     p1 = Policy()
     p2 = Policy()
     p1.implement_reform(reform1)
@@ -966,11 +940,11 @@ def test_cpi_offset_on_reverting_params():
     Test that params that revert to their pre-TCJA values
     in 2026 revert if a parameter_indexing_CPI_offset is specified.
     """
-    reform0 = {'parameter_indexing_CPI_offset': {2020: -0.001}}
-    reform1 = {'STD': {2017: [6350, 12700, 6350, 9350, 12700]},
-               'parameter_indexing_CPI_offset': {2020: -0.001}}
-    reform2 = {'STD': {2020: [10000, 20000, 10000, 10000, 20000]},
-               'parameter_indexing_CPI_offset': {2020: -0.001}}
+    reform0 = {"parameter_indexing_CPI_offset": {2020: -0.001}}
+    reform1 = {"STD": {2017: [6350, 12700, 6350, 9350, 12700]},
+               "parameter_indexing_CPI_offset": {2020: -0.001}}
+    reform2 = {"STD": {2020: [10000, 20000, 10000, 10000, 20000]},
+               "parameter_indexing_CPI_offset": {2020: -0.001}}
 
     p0 = Policy()
     p1 = Policy()
@@ -998,11 +972,15 @@ def test_raise_errors_regression():
     """
     This tests that raise_errors prevents the error from being thrown. The
     correct behavior is to exit the `adjust` function and store the errors.
+    ** From the parameters.py docstring for the adjust method:
+        raise_errors: Boolean
+          Raise errors as a ValidationError. If False, they will be stored
+          in the errors attribute.
     """
     ref = {
-        "II_brk7-indexed": [{"value": True}],
+        "II_brk7-indexed": [{"value": False}],
         "II_brk6": [{"value": 316700, "MARS": "single", "year": 2020}],
-        "II_brk7": [{"value": 445400, "MARS": "single", "year": 2020}],
+        "II_brk7": [{"value": 320700, "MARS": "single", "year": 2020}],
 
     }
     pol = Policy()
@@ -1177,14 +1155,14 @@ def test_activate_index():
     """
     pol1 = Policy()
     pol1.implement_reform({
-        "CTC_c": {2022: 2000},
-        "CTC_c-indexed": {2022: True}
+        "ODC_c": {2022: 1000},
+        "ODC_c-indexed": {2022: True}
     })
     pol2 = Policy()
     pol2.adjust(
         {
-            "CTC_c": [{"year": 2022, "value": 2000}],
-            "CTC_c-indexed": [{"year": 2022, "value": True}],
+            "ODC_c": [{"year": 2022, "value": 1000}],
+            "ODC_c-indexed": [{"year": 2022, "value": True}],
         }
     )
     cmp_policy_objs(pol1, pol2)
@@ -1193,12 +1171,12 @@ def test_activate_index():
     pol0.set_year(year=2021)
     pol2.set_state(year=[2021, 2022, 2023])
     exp = np.array([
-        pol0.CTC_c[0],
-        2000,
-        2000 * (1 + pol2.inflation_rates(year=2022))
+        pol0.ODC_c[0],
+        1000,
+        1000 * (1 + pol2.inflation_rates(year=2022))
     ]).round(2)
 
-    np.testing.assert_allclose(pol2.CTC_c, exp)
+    np.testing.assert_allclose(pol2.ODC_c, exp)
 
 
 def test_apply_cpi_offset():
@@ -1243,7 +1221,7 @@ def test_apply_cpi_offset():
 
 def test_multiple_cpi_swaps():
     """
-    Test changing a parameter's indexed status multiple times.
+    Test changing a parameter"s indexed status multiple times.
     """
     pol1 = Policy()
     pol1.implement_reform(
@@ -1404,7 +1382,7 @@ def test_adj_cpi_offset_and_index_status():
     """
     pol1 = Policy()
     pol1.implement_reform({
-        "CTC_c-indexed": {2020: True},
+        "ODC_c-indexed": {2020: True},
         "parameter_indexing_CPI_offset": {2020: -0.005}},
     )
     pol2 = Policy()
@@ -1412,7 +1390,7 @@ def test_adj_cpi_offset_and_index_status():
         {
             "parameter_indexing_CPI_offset":
             [{"year": 2020, "value": -0.005}],
-            "CTC_c-indexed": [{"year": 2020, "value": True}],
+            "ODC_c-indexed": [{"year": 2020, "value": True}],
         }
     )
     cmp_policy_objs(pol1, pol2)
@@ -1429,7 +1407,7 @@ def test_adj_cpi_offset_and_index_status():
 
     pol2.set_state(year=[2021, 2022])
     np.testing.assert_equal(
-        (pol2.CTC_c[1] / pol2.CTC_c[0] - 1).round(4),
+        (pol2.ODC_c[1] / pol2.ODC_c[0] - 1).round(4),
         round(pol0.inflation_rates(year=2021) + (-0.005), 4),
     )
 
@@ -1515,48 +1493,48 @@ def test_cpi_offset_does_not_affect_wage_indexed_params():
     np.testing.assert_equal(act_before_2025, exp_before_2025)
 
 
-def test_ext_plus_ctc1_reform(tests_path):
+def test_ext_plus_odc1_reform(tests_path):
     """
-    Test ext.json plus ctc1 compound reform relative to ext.json baseline.
+    Test ext.json plus odc1 compound reform relative to ext.json baseline.
     """
     # specify baseline policy, bas, as the extend-TCJA reform
     bas = Policy()
-    filename = os.path.join(tests_path, '..', 'reforms', 'ext.json')
-    with open(filename, 'r', encoding='utf-8') as rfile:
+    filename = os.path.join(tests_path, "..", "reforms", "ext.json")
+    with open(filename, "r", encoding="utf-8") as rfile:
         ext_text = rfile.read()
     bas.implement_reform(Policy.read_json_reform(ext_text))
     assert not bas.parameter_errors
-    # specify reform policy, ref, as extend-TCJA plus liberalization of CTC
+    # specify reform policy, ref, as extend-TCJA plus liberalization of ODC
     ref = Policy()
     ref.implement_reform(Policy.read_json_reform(ext_text))
-    ctc1_reform = {
-        # one possible child-tax-credit revision to the extend-TCJA reform
-        'CTC_c': {
-            2026: 2500.00,
-            2027: 2500.00,
-            2028: 2500.00,
-            2029: 2000.00,
+    odc1_reform = {
+        # one possible other-dependent-credit revision to extend-TCJA reform
+        "ODC_c": {
+            2026: 1000.00,
+            2027: 1000.00,
+            2028: 1000.00,
+            2029: 1000.00,
         },
-        'CTC_c-indexed': {2029: True},
+        "ODC_c-indexed": {2029: True},
     }
-    ref.implement_reform(ctc1_reform)
+    ref.implement_reform(odc1_reform)
     assert not ref.parameter_errors
     # check bas and ref parameter values against expected parameter values
-    exp_ctc_c_bas = {
-        2025: 2000,
-        2026: 2000,
-        2027: 2000,
-        2028: 2000,
-        2029: 2000,
-        2030: 2000,
+    exp_odc_c_bas = {
+        2025: 500,
+        2026: 500,
+        2027: 500,
+        2028: 500,
+        2029: 500,
+        2030: 500,
     }
-    exp_ctc_c_ref = {
-        2025: 2000,
-        2026: 2500,
-        2027: 2500,
-        2028: 2500,
-        2029: 2000,
-        2030: 2044.80,
+    exp_odc_c_ref = {
+        2025: 500,
+        2026: 1000,
+        2027: 1000,
+        2028: 1000,
+        2029: 1000,
+        2030: 1022.4,
     }
     exp_actc_c = {
         2025: 1700,
@@ -1568,57 +1546,57 @@ def test_ext_plus_ctc1_reform(tests_path):
     }
     for year in range(2025, 2031):
         bas.set_year(year)
-        assert np.allclose([bas.CTC_c], [exp_ctc_c_bas[year]])
+        assert np.allclose([bas.ODC_c], [exp_odc_c_bas[year]])
         assert np.allclose([bas.ACTC_c], [exp_actc_c[year]])
         ref.set_year(year)
-        assert np.allclose([ref.CTC_c], [exp_ctc_c_ref[year]])
+        assert np.allclose([ref.ODC_c], [exp_odc_c_ref[year]])
         assert np.allclose([ref.ACTC_c], [exp_actc_c[year]])
 
 
-def test_ext_plus_ctc2_reform(tests_path):
+def test_ext_plus_odc2_reform(tests_path):
     """
-    Test ext.json plus ctc2 compound reform relative to ext.json baseline.
+    Test ext.json plus odc2 compound reform relative to ext.json baseline.
     """
     # specify baseline policy, bas, as the extend-TCJA reform
     bas = Policy()
-    filename = os.path.join(tests_path, '..', 'reforms', 'ext.json')
-    with open(filename, 'r', encoding='utf-8') as rfile:
+    filename = os.path.join(tests_path, "..", "reforms", "ext.json")
+    with open(filename, "r", encoding="utf-8") as rfile:
         ext_text = rfile.read()
     bas.implement_reform(Policy.read_json_reform(ext_text))
     assert not bas.parameter_errors
-    # specify reform policy, ref, as extend-TCJA plus liberalization of CTC
+    # specify reform policy, ref, as extend-TCJA plus liberalization of ODC
     ref = Policy()
     ref.implement_reform(Policy.read_json_reform(ext_text))
-    ctc2_reform = {
-        # one possible child-tax-credit revision to the extend-TCJA reform
-        'CTC_c': {
-            2026: 2500.00,
-            2027: 2500.00,
-            2028: 2500.00,
-            2029: 2000.00,
+    odc2_reform = {
+        # one possible other-dependent-credit revision to extend-TCJA reform
+        "ODC_c": {
+            2026: 600.00,
+            2027: 600.00,
+            2028: 600.00,
+            2029: 800.00,
         },
-        'CTC_c-indexed': {2029: True},
-        'ACTC_c': {2029: 1750},
-        'ACTC_c-indexed': {2029: False},
+        "ODC_c-indexed": {2029: True},
+        "ACTC_c": {2029: 1750},
+        "ACTC_c-indexed": {2029: False},
     }
-    ref.implement_reform(ctc2_reform)
+    ref.implement_reform(odc2_reform)
     assert not ref.parameter_errors
     # check bas and ref parameter values against expected parameter values
-    exp_ctc_c_bas = {
-        2025: 2000,
-        2026: 2000,
-        2027: 2000,
-        2028: 2000,
-        2029: 2000,
-        2030: 2000,
+    exp_odc_c_bas = {
+        2025: 500,
+        2026: 500,
+        2027: 500,
+        2028: 500,
+        2029: 500,
+        2030: 500,
     }
-    exp_ctc_c_ref = {
-        2025: 2000,
-        2026: 2500,
-        2027: 2500,
-        2028: 2500,
-        2029: 2000,
-        2030: 2044.80,
+    exp_odc_c_ref = {
+        2025: 500,
+        2026: 600,
+        2027: 600,
+        2028: 600,
+        2029: 800,
+        2030: 817.92,
     }
     exp_actc_c_bas = {
         2025: 1700,
@@ -1638,8 +1616,8 @@ def test_ext_plus_ctc2_reform(tests_path):
     }
     for year in range(2025, 2031):
         bas.set_year(year)
-        assert np.allclose([bas.CTC_c], [exp_ctc_c_bas[year]])
+        assert np.allclose([bas.ODC_c], [exp_odc_c_bas[year]])
         assert np.allclose([bas.ACTC_c], [exp_actc_c_bas[year]])
         ref.set_year(year)
-        assert np.allclose([ref.CTC_c], [exp_ctc_c_ref[year]])
+        assert np.allclose([ref.ODC_c], [exp_odc_c_ref[year]])
         assert np.allclose([ref.ACTC_c], [exp_actc_c_ref[year]])