Skip to content

test: remove redundant import and streaming-stub smoke checks#214

Merged
ttuff merged 1 commit intomainfrom
codex/audit-tests-for-redundancy-and-utility
Mar 28, 2026
Merged

test: remove redundant import and streaming-stub smoke checks#214
ttuff merged 1 commit intomainfrom
codex/audit-tests-for-redundancy-and-utility

Conversation

@ttuff
Copy link
Copy Markdown
Contributor

@ttuff ttuff commented Mar 28, 2026

Motivation

  • Remove overlapping, low-value smoke assertions to reduce test churn while keeping focused contract coverage intact.

Description

  • Deleted a duplicate __version__ presence check from src/cubedynamics/tests/test_import_and_pipe.py and removed redundant streaming-stub NotImplementedError assertions from src/cubedynamics/tests/test_imports.py, keeping the explicit stub and download behavior tests in src/cubedynamics/tests/test_streaming_contracts.py.

Testing

  • Ran pytest src/cubedynamics/tests/test_imports.py src/cubedynamics/tests/test_import_and_pipe.py src/cubedynamics/tests/test_streaming_contracts.py -q which passed (7 passed, 1 warning) and ran pytest --collect-only -q which reported 231 collected tests.

Codex Task

@ttuff ttuff merged commit 7d75eb7 into main Mar 28, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant