Run the test suite without -v so a failure shows in the build log (next side of #414)
check / check (push) Successful in 3m23s
check / check (push) Successful in 3m23s
script/test now runs go test without -v. The Docker build cuts each step's log at 2 MiB, and the verbose output of the whole suite passed that limit before any failure was printed, so a red script/cibuild showed only passing packages. Without -v, go test prints one result line per package and, for a package that fails, everything its tests wrote, the application's log lines included. -race, -p 4 -parallel 8 and the 90-second per-package timeout are unchanged. The cause of the red builds was fixed by the cheaper test hashing already on next; the same two changes go to main separately. Model: opus-5-5
This commit was merged in pull request #415.
This commit is contained in:
+6
-1
@@ -27,6 +27,11 @@
|
|||||||
# binaries build or run at once, each with at most eight parallel tests. Under
|
# binaries build or run at once, each with at most eight parallel tests. Under
|
||||||
# -race every test binary and every link costs a few hundred MB, so the
|
# -race every test binary and every link costs a few hundred MB, so the
|
||||||
# defaults (one per core) add up to several GB on a many-core host.
|
# defaults (one per core) add up to several GB on a many-core host.
|
||||||
|
#
|
||||||
|
# No -v: the Docker build cuts each step's log off at 2 MiB, and verbose output
|
||||||
|
# from the whole suite passes that before a failure is printed. Without it, go
|
||||||
|
# test prints one result line per package and, for a package that fails,
|
||||||
|
# everything its tests wrote, application log lines included.
|
||||||
set -eu
|
set -eu
|
||||||
|
|
||||||
ROOT="$(cd "$(dirname "$0")/.." && pwd -P)"
|
ROOT="$(cd "$(dirname "$0")/.." && pwd -P)"
|
||||||
@@ -34,7 +39,7 @@ ROOT="$(cd "$(dirname "$0")/.." && pwd -P)"
|
|||||||
main() {
|
main() {
|
||||||
cd "$ROOT"
|
cd "$ROOT"
|
||||||
"$ROOT/script/assets"
|
"$ROOT/script/assets"
|
||||||
go test -v -race -p 4 -parallel 8 -timeout 90s ./...
|
go test -race -p 4 -parallel 8 -timeout 90s ./...
|
||||||
}
|
}
|
||||||
|
|
||||||
main "$@"
|
main "$@"
|
||||||
|
|||||||
Reference in New Issue
Block a user