Skip to content

Commit

Permalink
testsuite: update purge tests for job-list changes
Browse files Browse the repository at this point in the history
Problem: The job-list module now stores / recovers inactive job
data from its internal database.  Tests that assume job-list
only stores jobs in memory do not account for this, leading to
errors.

Solution: Update tests to read the job-list module stats instead
of using `flux jobs`.  This gathers internal data to the job-list
module, which allows purge tests to pass.
  • Loading branch information
chu11 committed Jun 12, 2024
1 parent 7907ee2 commit 96186e5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion t/t2809-job-purge.t
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ inactive_count() {
if test $how = "job-manager"; then
flux module stats --parse=inactive_jobs job-manager
elif test $how = "job-list"; then
flux jobs --no-header --filter=inactive|wc -l
flux module stats -p jobs.inactive job-list
elif test $how = "job-list-stats"; then
flux job stats | jq .job_states.inactive
else
Expand Down

0 comments on commit 96186e5

Please sign in to comment.