From ded462ea6b320974ecb6979aa500d1bcbf4f7dd3 Mon Sep 17 00:00:00 2001 From: Stijnvandenbroek Date: Wed, 11 Mar 2026 14:13:15 +0000 Subject: [PATCH] docs: simplify comment styles --- tests/test_dagster.py | 8 -------- tests/test_ml.py | 16 ---------------- tests/test_ops.py | 8 -------- 3 files changed, 32 deletions(-) diff --git a/tests/test_dagster.py b/tests/test_dagster.py index 2b805f2..b8bf57b 100644 --- a/tests/test_dagster.py +++ b/tests/test_dagster.py @@ -27,9 +27,7 @@ ALL_JOBS = [ ] -# --------------------------------------------------------------------------- # Generic schedule tests -# --------------------------------------------------------------------------- class TestSchedulesGeneric: @@ -53,9 +51,7 @@ class TestSchedulesGeneric: assert schedule.default_status == DefaultScheduleStatus.RUNNING -# --------------------------------------------------------------------------- # Generic job tests -# --------------------------------------------------------------------------- class TestJobsGeneric: @@ -70,9 +66,7 @@ class TestJobsGeneric: assert job.description -# --------------------------------------------------------------------------- # Schedule-specific tests -# --------------------------------------------------------------------------- class TestScheduleSpecific: @@ -89,9 +83,7 @@ class TestScheduleSpecific: assert funda_ingestion_schedule._run_config_fn is not None -# --------------------------------------------------------------------------- # Definitions integration test -# --------------------------------------------------------------------------- class TestDefinitions: diff --git a/tests/test_ml.py b/tests/test_ml.py index 955b980..ef826cf 100644 --- a/tests/test_ml.py +++ b/tests/test_ml.py @@ -21,9 +21,7 @@ from data_platform.assets.ml.discord_alerts import ( from data_platform.assets.ml.elo_inference import EloInferenceConfig, _best_run -# --------------------------------------------------------------------------- # preprocess -# --------------------------------------------------------------------------- class TestPreprocess: @@ -101,9 +99,7 @@ class TestPreprocess: assert feat in df.columns, f"Missing feature: {feat}" -# --------------------------------------------------------------------------- # _best_run -# --------------------------------------------------------------------------- class TestBestRun: @@ -152,9 +148,7 @@ class TestBestRun: ) -# --------------------------------------------------------------------------- # _build_embed -# --------------------------------------------------------------------------- class TestBuildEmbed: @@ -209,9 +203,7 @@ class TestBuildEmbed: assert embed["color"] == 0x00B894 -# --------------------------------------------------------------------------- # _cleanup_old_runs -# --------------------------------------------------------------------------- class TestCleanupOldRuns: @@ -257,9 +249,7 @@ class TestCleanupOldRuns: client.delete_run.assert_not_called() -# --------------------------------------------------------------------------- # elo_prediction_model asset -# --------------------------------------------------------------------------- def _make_training_df(n=20): @@ -354,9 +344,7 @@ class TestEloPredictionModelAsset: elo_prediction_model(context, config, postgres, mlflow_resource) -# --------------------------------------------------------------------------- # elo_inference asset -# --------------------------------------------------------------------------- class TestEloInferenceAsset: @@ -457,9 +445,7 @@ class TestEloInferenceAsset: assert result.metadata["status"].value == "No new listings to score." -# --------------------------------------------------------------------------- # listing_alert asset -# --------------------------------------------------------------------------- class TestListingAlertAsset: @@ -570,9 +556,7 @@ class TestListingAlertAsset: assert mock_post.call_count == 2 -# --------------------------------------------------------------------------- # Config defaults -# --------------------------------------------------------------------------- class TestMLConfigs: diff --git a/tests/test_ops.py b/tests/test_ops.py index 138bf4a..0d01391 100644 --- a/tests/test_ops.py +++ b/tests/test_ops.py @@ -16,9 +16,7 @@ from data_platform.ops.elementary import ( ) -# --------------------------------------------------------------------------- # SourceFreshnessConfig -# --------------------------------------------------------------------------- class TestSourceFreshnessConfig: @@ -27,9 +25,7 @@ class TestSourceFreshnessConfig: assert cfg.source_name == "raw_funda" -# --------------------------------------------------------------------------- # _elementary_schema_exists -# --------------------------------------------------------------------------- class TestElementarySchemaExists: @@ -69,9 +65,7 @@ class TestElementarySchemaExists: assert result is False -# --------------------------------------------------------------------------- # elementary_run_models -# --------------------------------------------------------------------------- class TestElementaryRunModels: @@ -110,9 +104,7 @@ class TestElementaryRunModels: elementary_run_models(context) -# --------------------------------------------------------------------------- # elementary_generate_report -# --------------------------------------------------------------------------- class TestElementaryGenerateReport: