search

NOAA-GSL / ExascaleWorkflowSandbox

Other
2 stars 2 forks source link

Add test to test_flux_resource_list does not validate output #55

Closed NaureenBharwaniNOAA closed 8 months ago

NaureenBharwaniNOAA commented 8 months ago

The test_flux_resource_list test in tests/test_parsl_flux_mpi_hello.py only checks that the Parsl App completed with a status of 0 (meaning success). It does not actually check whether the output of the task, written to parsl_flux_resource_list.out is correct. This test needs to be updated to validate the output.

Two things will vary from run to run:

  1. The number of free nodes
  2. The NODELIST, host name that is used

Closes #43

christopherwharrop-noaa commented 8 months ago

@NaureenBharwaniNOAA - Why don't you go ahead and merge this. You'll see the "delete branch" button show up on the right side, which you can click to remove your PR branch.