change where some default parameters are defined

Author: Hussain Jafari

tests/integration/release/conftest.py

@@ -63,11 +63,13 @@ def pytest_addoption(parser: pytest.Parser) -> None:
     parser.addoption(
         "--output-dir",
         action="store",
+        default=CLI_DEFAULT_OUTPUT_DIR,
         help=f"The output directory to write to. Defaults to {CLI_DEFAULT_OUTPUT_DIR}.",
     )
     parser.addoption(
         "--dataset",
         action="store",
+        default=CLI_DEFAULT_DATASET,
         help="The dataset to generate. Options are 'census', 'acs', 'cps', 'ssa', 'tax_w2_1099', 'wic', and 'tax_1040'. No argument will default to acs.",
     )
     parser.addoption(
@@ -96,8 +98,8 @@ def pytest_addoption(parser: pytest.Parser) -> None:
 # Fixtures #
 ############
 @pytest.fixture(scope="session")
-def release_output_dir(request: pytest.FixtureRequest) -> Path | None:
-    output_dir_name = request.config.getoption("--output-dir", default=CLI_DEFAULT_OUTPUT_DIR)
+def release_output_dir(request: pytest.FixtureRequest) -> Path:
+    output_dir_name = request.config.getoption("--output-dir")
     output_dir = Path(output_dir_name) / f"{time.strftime('%Y%m%d_%H%M%S')}"
     output_dir.mkdir(parents=True, exist_ok=False)
     return output_dir.resolve()
@@ -107,7 +109,7 @@ def release_output_dir(request: pytest.FixtureRequest) -> Path | None:
 def dataset_params(
     request: pytest.FixtureRequest,
 ) -> tuple[str | int | Callable[..., pd.DataFrame] | None, ...]:
-    dataset_name = request.config.getoption("--dataset", default=CLI_DEFAULT_DATASET)
+    dataset_name = request.config.getoption("--dataset")
     try:
         dataset_func = DATASET_GENERATION_FUNCS[dataset_name]
     except KeyError: