suite: fix TestSubtestPanic failure (#1501)

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.

suite/suite_test.go

@@ -669,9 +669,14 @@ func (s *subtestPanicSuite) TearDownSubTest() {
 }
 
 func (s *subtestPanicSuite) TestSubtestPanic() {
-	s.Run("subtest", func() {
+	ok := s.Run("subtest", func() {
 		panic("panic")
 	})
+	// The panic must have the test reported as failed
+	if s.False(ok, "TestSubtestPanic/subtest failure is expected") {
+		// But we still want to success here, so just skip.
+		s.T().Skip("Subtest failure is expected")
+	}
 }
 
 func TestSubtestPanic(t *testing.T) {