Result of testing.T respect strict option (#539)

* test: testing.T.Failed should be false when strict is false and has pending steps

* fix: testing.T respect strict option

run_test.go

@@ -100,6 +100,7 @@ func Test_FailsOrPassesBasedOnStrictModeWhenHasPendingSteps(t *testing.T) {
 			ctx.Step(`^one$`, func() error { return nil })
 			ctx.Step(`^two$`, func() error { return ErrPending })
 		},
+		testingT: t,
 	}
 
 	r.storage = storage.NewStorage()
@@ -109,10 +110,13 @@ func Test_FailsOrPassesBasedOnStrictModeWhenHasPendingSteps(t *testing.T) {
 	}
 
 	failed := r.concurrent(1)
+	require.False(t, r.testingT.Failed())
 	require.False(t, failed)
 	assert.Equal(t, 1, beforeScenarioFired)
 	assert.Equal(t, 1, afterScenarioFired)
 
+	// avoid t is Failed because this testcase Failed
+	r.testingT = nil
 	r.strict = true
 	failed = r.concurrent(1)
 	require.True(t, failed)

suite.go

@@ -468,7 +468,7 @@ func (s *suite) runPickle(pickle *messages.Pickle) (err error) {
 		// Running scenario as a subtest.
 		s.testingT.Run(pickle.Name, func(t *testing.T) {
 			ctx, err = s.runSteps(ctx, pickle, pickle.Steps)
-			if err != nil {
+			if s.shouldFail(err) {
 				t.Error(err)
 			}
 		})