Skip to content

Commit 4b636c9

Browse files
committed
Merge pull request #11722 from stepancheg:fix-test-bundle-validate
PiperOrigin-RevId: 759259753
2 parents a98032f + 334c56f commit 4b636c9

File tree

2 files changed

+7
-4
lines changed

2 files changed

+7
-4
lines changed

Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -232,7 +232,7 @@ nogo-tests:
232232
#
233233
# FIXME(gvisor.dev/issue/10045): Need to fix broken tests.
234234
unit-tests: ## Local package unit tests in pkg/..., tools/.., etc.
235-
@$(call test,--test_tag_filters=-nogo$(COMMA)-requires-kvm --build_tag_filters=-network_plugins -- //:all pkg/... tools/... runsc/... vdso/... test/trace/... -//pkg/metric:metric_test -//pkg/coretag:coretag_test -//runsc/config:config_test -//tools/tracereplay:tracereplay_test -//test/trace:trace_test)
235+
@$(call test,--test_tag_filters=-nogo$(COMMA)-requires-kvm --build_tag_filters=-network_plugins -- //:all pkg/... tools/... runsc/... vdso/... test/trace/... -//pkg/metric:metric_test -//pkg/coretag:coretag_test -//tools/tracereplay:tracereplay_test -//test/trace:trace_test)
236236
.PHONY: unit-tests
237237

238238
# See unit-tests: this includes runsc/container.

runsc/config/config_test.go

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -695,9 +695,12 @@ func TestBundleValidate(t *testing.T) {
695695
name: "invalid value",
696696
bundle: Bundle(map[string]string{"debug": "invalid"}),
697697
verify: func(err error) error {
698-
want := `parsing "invalid": invalid syntax`
699-
if !strings.Contains(err.Error(), want) {
700-
return fmt.Errorf("mismatch error: got: %q want: %q", err.Error(), want)
698+
// Error differs in open-source version, and internally at Google
699+
// https://github.com/google/gvisor/pull/11722#issuecomment-2877847616
700+
wantOss := "parse error"
701+
wantInternal := `parsing "invalid": invalid syntax`
702+
if !strings.Contains(err.Error(), wantOss) && !strings.Contains(err.Error(), wantInternal) {
703+
return fmt.Errorf("mismatch error: got: %q want: %q or %s", err.Error(), wantOss, wantInternal)
701704
}
702705
return nil
703706
},

0 commit comments

Comments
 (0)