Skip to content

suite: fix TestSubtestPanic failure (#1501)#1503

Closed
dolmen wants to merge 1 commit intomasterfrom
suite-1501-fix-TestSubtestPanic
Closed

suite: fix TestSubtestPanic failure (#1501)#1503
dolmen wants to merge 1 commit intomasterfrom
suite-1501-fix-TestSubtestPanic

Conversation

@dolmen
Copy link
Collaborator

@dolmen dolmen commented Nov 1, 2023

Summary

The subtest of TestSubtestPanic is expected to fail, but that must not make the testuite of package 'suite' to fail. So call Skip to make go test to ignore that expected failure.

Changes

Motivation

suite testsuite failure (#1501).

Related issues

Cc: @linusbarth @xwjdsh

The subtest of TestSubtestPanic is expected to fail, but that must not
make the testuite of package 'suite' to fail. So call Skip to make 'go
test' to ignore that expected failure.
@dolmen dolmen marked this pull request as draft November 2, 2023 00:16
@dolmen
Copy link
Collaborator Author

dolmen commented Nov 2, 2023

Replaced by #1504.

@dolmen dolmen closed this Nov 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant