f39d46885edb7bfab12a10158d13895a5010d2ff
markd
  Sun Mar 13 11:21:26 2022 -0700
fixed shell check for command that should have failed

diff --git src/hg/utils/pslToBigPsl/tests/makefile src/hg/utils/pslToBigPsl/tests/makefile
index 27f6c6c..f817b95 100644
--- src/hg/utils/pslToBigPsl/tests/makefile
+++ src/hg/utils/pslToBigPsl/tests/makefile
@@ -1,37 +1,37 @@
 kentSrc = ../../../..
 include ../../../../inc/common.mk
 
 pslToBigPsl = ${DESTBINDIR}/pslToBigPsl
 bigPslToPsl = ${DESTBINDIR}/bigPslToPsl
 bedToBigBed = ${DESTBINDIR}/bedToBigBed
 
 # sh commands to deal with test that should have failed and didn't 
 cmdShouldFail = echo "Error: command should have failed" >&2; false
 
 all:: test
 
 test:: singleStrand doubleStrand badQSize
 	${MAKE} clean
 
 singleStrand: mkout
 	${pslToBigPsl} input/$@.psl output/$@.tmp
 	sort -k1,1 -k2,2n output/$@.tmp > output/$@.bigPslInput
 	${bedToBigBed} -type=bed12+12 -tab -as=${kentSrc}/hg/lib/bigPsl.as output/$@.bigPslInput input/chrom.sizes output/$@.bb 2> /dev/null
 	${bigPslToPsl} -collapseStrand output/$@.bb output/$@.psl
 	cmp input/$@.psl output/$@.psl
 
 doubleStrand: mkout
 	${pslToBigPsl} input/$@.psl output/$@.tmp
 	sort -k1,1 -k2,2n output/$@.tmp > output/$@.bigPslInput
 	${bedToBigBed} -type=bed12+12 -tab -as=${kentSrc}/hg/lib/bigPsl.as output/$@.bigPslInput input/chrom.sizes output/$@.bb 2> /dev/null
 	${bigPslToPsl} output/$@.bb output/$@.psl
 	cmp input/$@.psl output/$@.psl
 
 badQSize: mkout
-	${pslToBigPsl} -fa=input/MamRep605b.fa input/MamRep605b.psl output/$@.tmp && ${cmdShouldFail}
+	if ! ${pslToBigPsl} -fa=input/MamRep605b.fa input/MamRep605b.psl output/$@.tmp >& output/$@.err ; then true ; else ${cmdShouldFail} ; fi
 
 mkout:
 	@mkdir -p output
 
 clean::
 	rm -rf output