From 943c6e8f435977087605a80f3dc7fd0022e4824c Mon Sep 17 00:00:00 2001
From: Tigran Saluev <tigran@saluev.com>
Date: Sun, 10 Feb 2019 23:11:55 +0300
Subject: [PATCH] Add test on suite setup skipping

---
 suite/suite_test.go | 29 ++++++++++++++++++++++++++++-
 1 file changed, 28 insertions(+), 1 deletion(-)

diff --git a/suite/suite_test.go b/suite/suite_test.go
index 210dec2..26dddbd 100644
--- a/suite/suite_test.go
+++ b/suite/suite_test.go
@@ -265,7 +265,7 @@ func (suite *SuiteSkipTester) SetupSuite() {
 }
 
 func (suite *SuiteSkipTester) TestNothing() {
-	// SetupSuite() is only called when at least one test satisfies
+	// SetupSuite is only called when at least one test satisfies
 	// test filter. For this suite to be set up (and then tore down)
 	// it is necessary to add at least one test method.
 }
@@ -346,6 +346,33 @@ func TestRunSuite(t *testing.T) {
 
 }
 
+// This suite has no Test... methods. It's setup and teardown must be skipped.
+type SuiteSetupSkipTester struct {
+	Suite
+
+	setUp bool
+	toreDown bool
+}
+
+func (s *SuiteSetupSkipTester) SetupSuite() {
+	s.setUp = true
+}
+
+func (s *SuiteSetupSkipTester) NonTestMethod() {
+
+}
+
+func (s *SuiteSetupSkipTester) TearDownSuite() {
+	s.toreDown = true
+}
+
+func TestSkippingSuiteSetup(t *testing.T) {
+	suiteTester := new(SuiteSetupSkipTester)
+	Run(t, suiteTester)
+	assert.False(t, suiteTester.setUp)
+	assert.False(t, suiteTester.toreDown)
+}
+
 func TestSuiteGetters(t *testing.T) {
 	suite := new(SuiteTester)
 	suite.SetT(t)