flux-framework / flux-sched

Fluxion Graph-based Scheduler
GNU Lesser General Public License v3.0
84 stars 39 forks source link

testsuite: prepare for flux_job_wait(3) change #1147

Closed garlick closed 2 months ago

garlick commented 2 months ago

Problem: some tests presume that only jobs submitted with the 'waitable' flag will contribute to the return value of flux job wait --all, but flux-framework/flux-core#8357 proposes to make all jobs waitable, regardless of the flag.

Ensure that t1023-multiqueue-constraints.t consumes residual wait state before starting the array of waitable jobs.

codecov[bot] commented 2 months ago

Codecov Report

Merging #1147 (0067432) into master (991d370) will not change coverage. The diff coverage is n/a.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1147 +/- ## ====================================== Coverage 70.6% 70.6% ====================================== Files 95 95 Lines 12744 12744 ====================================== Hits 9010 9010 Misses 3734 3734 ```
garlick commented 2 months ago

Yeah it's a bit of a concern.... Thanks, I'll set MWP.