Fix Version/s: None
Team:Data Release Production
The nightly build failed because of a photodiode test failure. Looking at the code again it seems that the test is assuming that other tests are run first. This can not be relied upon since in some test environments the order can be randomized or with multi-process testing the test can be allocated to a subprocess that does not have any other relevant tests in it.
The reason it was working at all is that the ingest tests set up a per-testcase butler repo for efficiency reasons and just change the output run per test. Since exposure records are written for any ingest those records then exist for the followup photodiode test.
You can reproduce the error using pytest -k testPhotodiode tests/test_ingest.py.
In particular the error message is not with the command-line execution but is in the butler get that runs afterwards. This suggests that the command-line is returning good exit status even though it should be complaining that it was not able to ingest the file.
- Rewrite the test to not inherit from ingest test base but to ingest the relevant raw file explicitly.
- Add a second test where that raw file is not ingested that should fail because there is no matching exposure record.
Note that raw-ingest collates all the failure reasons and reports them at the end and does not fail immediately. it tries to ingest what it can first.
I'm sorry I didn't spot this issue in my review.
|Status||To Do [ 10001 ]||In Progress [ 3 ]|
|Reviewers||Tim Jenness [ tjenness ]|
|Status||In Progress [ 3 ]||In Review [ 10004 ]|
|Status||In Review [ 10004 ]||Reviewed [ 10101 ]|
|Resolution||Done [ 10000 ]|
|Status||Reviewed [ 10101 ]||Done [ 10002 ]|