Skip to content

Run Slurm script unit tests in CI#2744

Merged
fabrizio2210 merged 3 commits into
mainfrom
agent/slurm-script-unit-tests-ci
Jul 16, 2026
Merged

Run Slurm script unit tests in CI#2744
fabrizio2210 merged 3 commits into
mainfrom
agent/slurm-script-unit-tests-ci

Conversation

@fabrizio2210

@fabrizio2210 fabrizio2210 commented Jul 15, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add a test-python Makefile target using Python unittest discovery
  • discover *_test.py suites in all the repo
  • add a dedicated, change-gated GitHub Actions job that runs the target
  • include the job in the existing ci-success dependency set
  • skip cleanly only when neither test root contains tests

Why

This extracts the reusable unit-test runner from #2719 into a standalone change based on main. Keeping the runner independent lets Python tests in #2719, #2742, #2743, and future pull requests use the same CI job without depending on a health-check behavior change.

The discovery root cover all the Python scripts. In particular, this ensures #2743 runs its check_runner_test.py and write_soperator_metadata_test.py suites, as well as the existing worker tests.

PR #2743 is stacked on this branch so its checked-out head contains the Make target; #2742 remains stacked on #2743. Because the repository uses pull_request_target, the new job becomes available after this PR merges into main. Synchronizing #2743 after that merge will trigger the 109-test suite.

Validation

@fabrizio2210
fabrizio2210 merged commit a24659f into main Jul 16, 2026
22 checks passed
@fabrizio2210
fabrizio2210 deleted the agent/slurm-script-unit-tests-ci branch July 16, 2026 13:29
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.

2 participants