Merge pull request #52 from scr-oath/suite-Run-returns-count

Make suite.Run return the number of tests found and executed

Author: scr-oath

tests/README.md

@@ -82,7 +82,7 @@ end
 
 function TestSuite(t)
     -- Same mechanism for test discovery is used, but then the suite is run as sub tests via suite.Run
-    suite.Run(t, MySuite)
+    assert(suite.Run(t, MySuite), "No tests were run by this Suite")
 
     -- Called for every test: two tests so should be 2
     assert(MySuite.setupCount == 2, tostring(MySuite.setupCount))

tests/lua_const.go

@@ -1,7 +1,7 @@
 // GENERATED BY textFileToGoConst
 // GitHub:     github.com/logrusorgru/textFileToGoConst
 // input file: suite.lua
-// generated:  Sat Nov  5 20:27:23 PDT 2022
+// generated:  Thu Nov 10 11:29:24 PST 2022
 
 package tests
 
@@ -13,6 +13,8 @@ local suite = {
 }
 
 function suite.Run(t, testSuite)
+    local testCount = 0
+
     -- testSuite must be subclass of suite.Suite, so will have this method
     testSuite:SetT(t)
 
@@ -21,6 +23,7 @@ function suite.Run(t, testSuite)
     end
     for k, v in pairs(testSuite) do
         if strings.has_prefix(k, "Test") then
+            testCount = testCount + 1
             if testSuite.SetupTest then
                 testSuite:SetupTest()
             end
@@ -37,6 +40,8 @@ function suite.Run(t, testSuite)
     if testSuite.TearDownSuite then
         testSuite:TearDownSuite()
     end
+
+    return testCount
 end
 
 function suite.Suite:new(o)

tests/suite.lua

@@ -6,6 +6,8 @@ local suite = {
 }
 
 function suite.Run(t, testSuite)
+    local testCount = 0
+
     -- testSuite must be subclass of suite.Suite, so will have this method
     testSuite:SetT(t)
 
@@ -14,6 +16,7 @@ function suite.Run(t, testSuite)
     end
     for k, v in pairs(testSuite) do
         if strings.has_prefix(k, "Test") then
+            testCount = testCount + 1
             if testSuite.SetupTest then
                 testSuite:SetupTest()
             end
@@ -30,6 +33,8 @@ function suite.Run(t, testSuite)
     if testSuite.TearDownSuite then
         testSuite:TearDownSuite()
     end
+
+    return testCount
 end
 
 function suite.Suite:new(o)

tests/testdata/test_suite.lua

@@ -42,7 +42,7 @@ end
 
 function TestSuite(t)
     -- Same mechanism for test discovery is used, but then the suite is run as sub tests via suite.Run
-    suite.Run(t, MySuite)
+    assert(suite.Run(t, MySuite), "No tests were run by this Suite")
 
     -- Called for every test: two tests so should be 2
     assert(MySuite.setupCount == 2, tostring(MySuite.setupCount))
@@ -52,3 +52,10 @@ function TestSuite(t)
     assert(MySuite.setupSuiteCount == 1, tostring(MySuite.setupSuiteCount))
     assert(MySuite.tearDownSuiteCount == 1, tostring(MySuite.tearDownSuiteCount))
 end
+
+local EmptySuite = suite.Suite:new()
+
+function TestEmptySuiteReturnsZero(t)
+    local testCount = suite.Run(t, EmptySuite)
+    assert(testCount == 0, string.format('EmptySuite ran %d tests, unexpectedly', testCount))
+end