Skip to content

Commit 0bf2a8f

Browse files
committed
Removed integration test marker from tests/unit/test_query_resolver.TestClickCLI.test_main_function_with_contingency_table and updated tests README
1 parent cd9acbd commit 0bf2a8f

2 files changed

Lines changed: 10 additions & 4 deletions

File tree

tests/README.md

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,14 +6,18 @@ uv run pytest tests/unit
66
```
77

88
## Integration tests
9-
Require Docker and a reachable Postgres instance. Disabled by default.
9+
The `tests/integration/test_docker_runner.py` require Docker and a reachable Postgres instance.
10+
11+
The `tests/integration/test_submission_api_session_integration.py` require a deployed 5STES instance.
12+
13+
All integration tests are disabled by default. Run via:
1014

1115
```bash
1216
uv run pytest tests/integration -m integration
1317
```
1418

1519
## Docker smoke tests
16-
Compose-based smoke tests for the analytics and bunny-wrapper containers.
20+
Compose-based entrypoint smoke tests for the analytics and bunny-wrapper containers.
1721
```bash
1822
docker compose -f docker/analytics-dev/test.compose.yml up -d
1923
docker compose -f docker/bunny-wrapper/test.compose.yml up -d

tests/unit/test_query_resolver.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,10 @@
1111
- The Click CLI interface: expected options, their names, and defaults.
1212
1313
All tests are unit tests — external dependencies (SQLAlchemy engine, database
14-
connections) are mocked. No real database or container is required.
14+
connections) are mocked.
15+
16+
There is one integration style test: TestClickCLI.test_main_function_with_contingency_table
17+
but we classify as unit test for our purposes as it requires no external infrastacture (uses SQlLite).
1518
"""
1619
import json
1720
import os
@@ -450,7 +453,6 @@ def test_main_function_with_percentile_sketch(self):
450453
if os.path.exists(temp_filename):
451454
os.remove(temp_filename)
452455

453-
@pytest.mark.integration
454456
def test_main_function_with_contingency_table(self):
455457
"""Test main function with contingency table analysis."""
456458
user_query = "SELECT gender, race FROM patients"

0 commit comments

Comments
 (0)