Skip to content

Commit

Permalink
Set pipefail for Cram tests
Browse files Browse the repository at this point in the history
This was added in places where it's been necessary. Setting it globally
simplifies tests and allows future tests to use pipes without the need
to set pipefail explicitly.
  • Loading branch information
victorlin committed Sep 23, 2024
1 parent 1995748 commit 8d27d76
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion .cramrc
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
[cram]
shell = /bin/bash
shell = /bin/bash -o pipefail
indent = 2
1 change: 0 additions & 1 deletion tests/functional/curate/cram/validate-records.t
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ error when it encounters the record with mismatched fields.
Passing the records through multiple augur curate commands should raise the
same error when it encounters the record with mismatched fields.

$ set -o pipefail
$ cat records.ndjson \
> | ${AUGUR} curate passthru \
> | ${AUGUR} curate passthru \
Expand Down
1 change: 0 additions & 1 deletion tests/functional/merge/cram/merge.t
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
SETUP

$ export AUGUR="${AUGUR:-$TESTDIR/../../../../bin/augur}"
$ set -o pipefail


BASIC USAGE
Expand Down

0 comments on commit 8d27d76

Please sign in to comment.