Skip to content

Commit 5cb5437

Browse files
ianlancetaylorgopherbot
authored andcommitted
cmd/go/internal/vcsweb/vcstest: skip bzr test if deps not found
The linux-arm64 trybots are consistently failing with vcstest_test.go:155: 2025/01/30 21:50:41 hello.txt: > handle bzr > env BZR_EMAIL='Russ Cox <[email protected]>' > env EMAIL='Russ Cox <[email protected]>' > bzr init-repo . [stderr] brz: ERROR: Couldn't import breezy and dependencies. Please check the directory containing breezy is on your PYTHONPATH. Error: PyErr { type: <class 'ModuleNotFoundError'>, value: ModuleNotFoundError("No module named 'breezy'"), traceback: None } vcstest_test.go:161: hello.txt:6: bzr init-repo .: exit status 1 This seems to be a problem with the builder. For now, skip the test if we see that error message, just as we already skip the test if the bzr executable is not found. For #71504 Change-Id: If8b6d4dea02dc16198ba6067595dff3340a81299 Reviewed-on: https://go-review.googlesource.com/c/go/+/646635 Reviewed-by: Dmitri Shuralyov <[email protected]> Auto-Submit: Ian Lance Taylor <[email protected]> Reviewed-by: Carlos Amedee <[email protected]> Reviewed-by: Ian Lance Taylor <[email protected]> LUCI-TryBot-Result: Go LUCI <[email protected]> Reviewed-by: Dmitri Shuralyov <[email protected]>
1 parent 7472b4c commit 5cb5437

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

src/cmd/go/internal/vcweb/vcstest/vcstest_test.go

+7
Original file line numberDiff line numberDiff line change
@@ -158,6 +158,13 @@ func TestScripts(t *testing.T) {
158158
if notInstalled := (vcweb.ServerNotInstalledError{}); errors.As(err, &notInstalled) || errors.Is(err, exec.ErrNotFound) {
159159
t.Skip(err)
160160
}
161+
162+
// For issue #71504 ignore an error about
163+
// bzr not being able to find dependencies.
164+
if strings.Contains(buf.String(), "brz: ERROR: Couldn't import breezy and dependencies.") {
165+
t.Skip("skipping test due to bzr installation problem")
166+
}
167+
161168
t.Error(err)
162169
}
163170
})

0 commit comments

Comments
 (0)