Merge lp://staging/~larry-e-works/uci-engine/write-exitcode-to-file into lp://staging/uci-engine
Status: | Needs review |
---|---|
Proposed branch: | lp://staging/~larry-e-works/uci-engine/write-exitcode-to-file |
Merge into: | lp://staging/uci-engine |
Diff against target: |
32 lines (+15/-0) 1 file modified
test_runner/tstrun/run_test.py (+15/-0) |
To merge this branch: | bzr merge lp://staging/~larry-e-works/uci-engine/write-exitcode-to-file |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Vincent Ladeuil (community) | Approve | ||
Evan (community) | Needs Fixing | ||
Paul Larson | Approve | ||
Francis Ginther | Approve | ||
Review via email: mp+246721@code.staging.launchpad.net |
Commit message
Write adt-run exitcode into a file in the results directory.
Description of the change
Added bits to write the exit code to the results. Relevant trello task is: https:/
Unmerged revisions
- 924. By Larry E Works
-
Added bits to ensure results directory exists per Evan's comment to previous MP. Used os.makedirs instead of os.mkdir as it will ensure all intermediate-level directories are created. While this is not currently needed it may be when the test runner is modified to copy all results to swift.
- 923. By Larry E Works
-
Incorporated Pauls recommendation to user context manager instead of the explicit open/close.
- 922. By Larry E Works
-
Added the bit to put the exit code in the results directory.
- 921. By Larry E Works
-
Added bits to write the test exitcode to a file.
A couple of small comments