Skip to content

Commit e85f69f

Browse files
committed
[FAB-13369] gossip and etcdraft tests run first
Help drive out test flakes early. Change-Id: I907446d055dc1bb37078c3e9089c5b3d78a11772 Signed-off-by: Matthew Sykes <sykesmat@us.ibm.com>
1 parent ec0ce6e commit e85f69f

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

unit-test/run.sh

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,10 @@ excluded_packages=(
1313
)
1414

1515
# regexes for packages that must be run serially
16+
# NOTE: these packages are now tested first as they tend to fail more frequently
1617
serial_packages=(
1718
"github.com/hyperledger/fabric/gossip"
19+
"github.com/hyperledger/fabric/orderer/consensus/etcdraft"
1820
)
1921

2022
# packages which need to be tested with build tag pluginsenabled
@@ -82,15 +84,15 @@ run_tests() {
8284
echo ${GO_TAGS}
8385

8486
time {
85-
local parallel=$(parallel_test_packages "$@")
86-
if [ -n "${parallel}" ]; then
87-
go test ${flags} ${race_flags} -tags "$GO_TAGS" ${parallel[@]} -short -timeout=20m
88-
fi
89-
9087
local serial=$(serial_test_packages "$@") # race is disabled as well
9188
if [ -n "${serial}" ]; then
9289
go test ${flags} -tags "$GO_TAGS" ${serial[@]} -short -p 1 -timeout=20m
9390
fi
91+
92+
local parallel=$(parallel_test_packages "$@")
93+
if [ -n "${parallel}" ]; then
94+
go test ${flags} ${race_flags} -tags "$GO_TAGS" ${parallel[@]} -short -timeout=20m
95+
fi
9496
}
9597
}
9698

0 commit comments

Comments
 (0)