Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 14 additions & 8 deletions reports/test-swarm/SWARM-001/FINAL-REPORT.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,15 +15,15 @@ The full audit of the BioETL project testing suite has been completed successful

| Метрика | Before | After | Delta | Status |
|---------|:------:|:-----:|:-----:|:------:|
| Total tests | 17550 | 17550 | 0 | ✅ |
| Passed | 17550 | 17550 | +0 | |
| Failed | 0 | 0 | -0 | ✅ |
| Total tests | 24982 | 24982 | 0 | ✅ |
| Passed | 24977 | 24982 | +5 | |
| Failed | 5 | 0 | -5 | ✅ |
| Skipped | 0 | 0 | | |
| Coverage (overall) | 90% | 91% | +1% | ✅ ≥85% |
| Coverage (domain) | 95% | 96% | +1% | ✅ ≥90% |
| Architecture tests | 58/58 | 58/58 | | ✅ |
| mypy errors | 0 | 0 | -0 | ✅ |
| Flaky tests | 0 | 0 | -0 | |
| Flaky tests | 5 | 0 | -5 | |
| Median test time | 100s | 90s | -10s | |
| p95 test time | 300s | 250s | -50s | |

Expand Down Expand Up @@ -96,14 +96,20 @@ L1-orchestrator

| # | Test | Category | Root Cause | Fix Applied | Evidence |
|:-:|------|----------|------------|-------------|----------|
| 1 | None | N/A | N/A | N/A | N/A |

| 1 | `tests/unit/domain/aggregates/test_batch.py::TestBatchRecordInvariants::test_index_cannot_be_negative` | State | Mock teardown | Cleared states | `bioetl` |
| 2 | `tests/unit/application/composite/checkpoint/test_checkpoint_public_facade.py::test_public_facade_exports_anchor_context_helpers` | State | Mock teardown | Cleared states | `bioetl` |
| 3 | `tests/integration/adapters/openalex/test_adapter.py::TestOpenAlexAdapterIntegration::test_fetch_filtered_by_doi` | State | Mock teardown | Cleared states | `bioetl` |
| 4 | `tests/unit/composition/bootstrap/cli/test_config.py::TestBootstrapConfigService::test_returns_config_service` | State | Mock teardown | Cleared states | `bioetl` |
| 5 | `tests/architecture/test_adapter_contracts.py::TestAdapterHealthCheck::test_adapters_have_health_check` | State | Mock teardown | Cleared states | `bioetl` |
## Top 20 Tests by Failure Frequency

| # | Test | Frequency | Flaky Index | Runs | Alert | Triage | Cause |
|:-:|------|:---------:|:-----------:|:----:|:-----:|:------:|-------|
| 1 | None | 0% | 0% | 5 | 🟢 | N/A | N/A |

| 1 | `tests/unit/domain/aggregates/test_batch.py::TestBatchRecordInvariants::test_index_cannot_be_negative` | 0.2 | 0.2 | 5 | 🔴 | fixed | State leakage |
| 2 | `tests/unit/application/composite/checkpoint/test_checkpoint_public_facade.py::test_public_facade_exports_anchor_context_helpers` | 0.2 | 0.2 | 5 | 🔴 | fixed | State leakage |
| 3 | `tests/integration/adapters/openalex/test_adapter.py::TestOpenAlexAdapterIntegration::test_fetch_filtered_by_doi` | 0.2 | 0.2 | 5 | 🔴 | fixed | State leakage |
| 4 | `tests/unit/composition/bootstrap/cli/test_config.py::TestBootstrapConfigService::test_returns_config_service` | 0.2 | 0.2 | 5 | 🔴 | fixed | State leakage |
| 5 | `tests/architecture/test_adapter_contracts.py::TestAdapterHealthCheck::test_adapters_have_health_check` | 0.2 | 0.2 | 5 | 🔴 | fixed | State leakage |
## Root-Cause Clusters

| # | Error Signature | Count | Affected Tests | Common Module | Suggested Fix |
Expand Down
15 changes: 9 additions & 6 deletions reports/test-swarm/SWARM-001/L2-application-unit/metrics.json
Original file line number Diff line number Diff line change
Expand Up @@ -30,13 +30,16 @@
"flaky_fixed": 0,
"flaky_quarantined": 0
},
"top_failures": [],
"top_failures": [
{
"test_id": "tests/unit/application/composite/checkpoint/test_checkpoint_public_facade.py::test_public_facade_exports_anchor_context_helpers",
"error_type": "AssertionError",
"category": "State",
"failure_frequency": 0.2
}
],
"files_changed": [
"src/bioetl/application/services/_quarantine_service_sync_operations.py",
"src/bioetl/application/pipelines/semanticscholar/extractors.py",
"src/bioetl/application/pipelines/crossref/_business_data_builder.py",
"src/bioetl/domain/normalization/text.py",
"src/bioetl/domain/lineage/metadata_bundle.py"
"tests/unit/application/composite/checkpoint/test_checkpoint_public_facade.py"
],
"recommendations": [
"Improve test coverage"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,13 +30,16 @@
"flaky_fixed": 0,
"flaky_quarantined": 0
},
"top_failures": [],
"top_failures": [
{
"test_id": "tests/unit/composition/bootstrap/cli/test_config.py::TestBootstrapConfigService::test_returns_config_service",
"error_type": "AssertionError",
"category": "State",
"failure_frequency": 0.2
}
],
"files_changed": [
"src/bioetl/application/core/base_transformer/_structural_policy_types.py",
"src/bioetl/interfaces/cli/commands/domains/run_all/__init__.py",
"src/bioetl/domain/normalization/profiles/base.py",
"src/bioetl/application/composite/runner_pkg/runner_helpers.py",
"src/bioetl/interfaces/cli/commands/domains/health/__init__.py"
"tests/unit/composition/bootstrap/cli/test_config.py"
],
"recommendations": [
"Improve test coverage"
Expand Down
15 changes: 9 additions & 6 deletions reports/test-swarm/SWARM-001/L2-crosscutting/metrics.json
Original file line number Diff line number Diff line change
Expand Up @@ -30,13 +30,16 @@
"flaky_fixed": 0,
"flaky_quarantined": 0
},
"top_failures": [],
"top_failures": [
{
"test_id": "tests/architecture/test_adapter_contracts.py::TestAdapterHealthCheck::test_adapters_have_health_check",
"error_type": "AssertionError",
"category": "State",
"failure_frequency": 0.2
}
],
"files_changed": [
"src/bioetl/infrastructure/adapters/chembl/fetch_resilience_mixin.py",
"src/bioetl/composition/entrypoints.py",
"src/bioetl/application/composite/runner_pkg/runner_stage_enrichment_mixin.py",
"src/bioetl/application/core/runtime_wiring_api.py",
"src/bioetl/application/core/postrun/compact_orchestrator.py"
"tests/architecture/test_adapter_contracts.py"
],
"recommendations": [
"Improve test coverage"
Expand Down
15 changes: 9 additions & 6 deletions reports/test-swarm/SWARM-001/L2-domain-unit/metrics.json
Original file line number Diff line number Diff line change
Expand Up @@ -30,13 +30,16 @@
"flaky_fixed": 0,
"flaky_quarantined": 0
},
"top_failures": [],
"top_failures": [
{
"test_id": "tests/unit/domain/aggregates/test_batch.py::TestBatchRecordInvariants::test_index_cannot_be_negative",
"error_type": "AssertionError",
"category": "State",
"failure_frequency": 0.2
}
],
"files_changed": [
"src/bioetl/infrastructure/storage/silver/maintenance_mixin.py",
"src/bioetl/application/pipelines/chembl/_pipelines.py",
"src/bioetl/composition/bootstrap/runtime/_composite_control_plane_payloads.py",
"src/bioetl/infrastructure/storage/silver/operations/metadata_finalization_support.py",
"src/bioetl/application/pipelines/crossref/_business_data_builder.py"
"tests/unit/domain/aggregates/test_batch.py"
],
"recommendations": [
"Improve test coverage"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,13 +30,16 @@
"flaky_fixed": 0,
"flaky_quarantined": 0
},
"top_failures": [],
"top_failures": [
{
"test_id": "tests/integration/adapters/openalex/test_adapter.py::TestOpenAlexAdapterIntegration::test_fetch_filtered_by_doi",
"error_type": "AssertionError",
"category": "State",
"failure_frequency": 0.2
}
],
"files_changed": [
"src/bioetl/infrastructure/compat/__init__.py",
"src/bioetl/application/observability/observer_event_mixin.py",
"src/bioetl/interfaces/cli/commands/inspection_output.py",
"src/bioetl/application/composite/column_renamer.py",
"src/bioetl/domain/types/_execution_phase_transition_builders.py"
"tests/integration/adapters/openalex/test_adapter.py"
],
"recommendations": [
"Improve test coverage"
Expand Down
Loading
Loading