Skip to content

Seperate out Testing Suite into Unit and Regression Testing - #59

Merged
harryswift01 merged 15 commits into
mainfrom
58-general-seperate-out-unit-and-regression-test-suites
Sep 28, 2026
Merged

harryswift01 merged 15 commits into
mainfrom
58-general-seperate-out-unit-and-regression-test-suites

Conversation

@harryswift01

@harryswift01 harryswift01 commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

Summary

This PR updates SIST's test suite and CI, splitting tests into fast unit tests and regression tests, adding a set of scheduled CI workflows, and reporting combined Python + C++ coverage through Coveralls.

Changes

Split the test suite into tests/unit and tests/regression:

  • Moved test_energetics.py/test_ir_finder.py (pure Python, no build needed) into tests/unit
  • Moved test_cli.py/test_regression.py/test_legacy_entrypoints.py (need the compiled qsidd binaries and the real CLI) into tests/regression, along with tests/data/tests/reference
  • Split conftest.py accordingly, tests/unit needs no fixtures at all, so tests/regression/conftest.py now holds everything that builds/runs the CLI
  • tests/unit now runs standalone in under a second, with zero build or subprocess activity

Add scheduled CI workflows and README badges:

  • daily.yaml (unit tests across Python 3.12/3.13/3.14), weekly-docs.yaml (docs build), update-copyright-years-in-license-file.yaml, and renamed the existing conda-build regression job to weekly-regression.yaml
  • Updated README.md badges to match

Impact

  • No behavioural change to SIST itself, this is entirely test/CI structure
  • Coverage is now tracked for both the Python package and the C++ core

@harryswift01 harryswift01 self-assigned this Sep 25, 2026
@harryswift01 harryswift01 added the enhancement New feature or request label Sep 25, 2026
@harryswift01 harryswift01 linked an issue Sep 25, 2026 that may be closed by this pull request
@harryswift01 harryswift01 changed the title 58 general seperate out unit and regression test suites Seperate out Testing Suite into Unit and Regression Testing Sep 25, 2026

@jimboid jimboid left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code tests and tests that test the science should be for sure separate processes. Happy to see this merge.

@harryswift01
harryswift01 merged commit 3a4723e into main Sep 28, 2026
15 checks passed
@harryswift01
harryswift01 deleted the 58-general-seperate-out-unit-and-regression-test-suites branch September 28, 2026 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[General]: Seperate out unit and regression test suites

2 participants