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

taxcalc/tests/test_policy.py

@@ -54,12 +54,12 @@ 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
@@ -90,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
@@ -192,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)
@@ -218,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)
@@ -258,14 +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},
-        '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
@@ -289,7 +289,7 @@ def check_eitc_c(ppo, reform, ifactor):
     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]
@@ -299,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)
@@ -321,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)
@@ -349,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)
@@ -384,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)
@@ -395,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)
 
@@ -405,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
@@ -418,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]),
@@ -519,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
@@ -535,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
@@ -572,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
 
 
@@ -596,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
@@ -711,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] = {}
@@ -726,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
@@ -750,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
 
 
@@ -769,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)
@@ -784,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)
 
@@ -805,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
 
@@ -815,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
@@ -838,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)
 
@@ -867,20 +867,20 @@ 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 = {'ODC_c-indexed': {2020: True}}
+    reform1 = {"ODC_c-indexed": {2020: True}}
     policy1 = Policy()
     policy1.implement_reform(reform1)
     offset = -0.005
-    reform2 = {'ODC_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)
     # extract from policy1 and policy2 the parameter values of ODC_c
@@ -911,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)
@@ -940,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()
@@ -1221,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(
@@ -1499,8 +1499,8 @@ def test_ext_plus_odc1_reform(tests_path):
     """
     # 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
@@ -1509,13 +1509,13 @@ def test_ext_plus_odc1_reform(tests_path):
     ref.implement_reform(Policy.read_json_reform(ext_text))
     odc1_reform = {
         # one possible other-dependent-credit revision to extend-TCJA reform
-        'ODC_c': {
+        "ODC_c": {
             2026: 1000.00,
             2027: 1000.00,
             2028: 1000.00,
             2029: 1000.00,
         },
-        'ODC_c-indexed': {2029: True},
+        "ODC_c-indexed": {2029: True},
     }
     ref.implement_reform(odc1_reform)
     assert not ref.parameter_errors
@@ -1559,8 +1559,8 @@ def test_ext_plus_odc2_reform(tests_path):
     """
     # 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
@@ -1569,15 +1569,15 @@ def test_ext_plus_odc2_reform(tests_path):
     ref.implement_reform(Policy.read_json_reform(ext_text))
     odc2_reform = {
         # one possible other-dependent-credit revision to extend-TCJA reform
-        'ODC_c': {
+        "ODC_c": {
             2026: 600.00,
             2027: 600.00,
             2028: 600.00,
             2029: 800.00,
         },
-        'ODC_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(odc2_reform)
     assert not ref.parameter_errors

taxcalc/tests/test_puf_var_stats.py

@@ -66,7 +66,7 @@ def create_base_table(test_path):
     # create table_dict with sorted read vars followed by sorted calc vars
     table_dict = {}
     for var in sorted(read_vars):
-        if "taxdata_puf" in read_var_dict['read'][var]['availability']:
+        if 'taxdata_puf' in read_var_dict['read'][var]['availability']:
             table_dict[var] = read_var_dict['read'][var]['desc']
         else:
             pass