Skip to content

Switch from nose2 to pytest, add local unit tests#116

Merged
ad-m merged 2 commits intomasterfrom
worktree-quirky-gathering-treasure
Mar 7, 2026
Merged

Switch from nose2 to pytest, add local unit tests#116
ad-m merged 2 commits intomasterfrom
worktree-quirky-gathering-treasure

Conversation

@ad-m
Copy link
Owner

@ad-m ad-m commented Mar 7, 2026

Summary

  • Replace nose2 with pytest as test runner, configure via pyproject.toml
  • Tag all 8 existing E2E tests with @pytest.mark.e2e; excluded by default so pytest runs only fast unit tests
  • Add 50 local unit tests covering task serialization, exceptions, and client/job logic (no API keys, no network)
  • Separate CI: make test (unit tests on every push), make test_e2e (E2E tests in dedicated workflow)
  • Delete unittest.cfg (nose2 multiprocess config), remove nose2 references from setup.py/setup.cfg/tox.ini

Test plan

  • pytest runs 50 unit tests, 8 E2E deselected (verified locally)
  • pytest -m e2e --override-ini="addopts=" collects all 8 E2E tests
  • make test runs unit tests only
  • CI pythonpackage.yml runs unit tests across Python 3.9–3.14
  • CI e2e.yml runs E2E tests via make test_e2e

🤖 Generated with Claude Code

Adam Dobrawy and others added 2 commits March 7, 2026 12:44
- Replace nose2 with pytest as test runner
- Tag all existing E2E tests with @pytest.mark.e2e
- Add 50 fast unit tests for tasks, exceptions, and client/job logic
- Separate CI: unit tests on every push, E2E tests via make test_e2e
- Delete unittest.cfg, add pyproject.toml with pytest config

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@ad-m ad-m merged commit a4c14dd into master Mar 7, 2026
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant