fixes #13 aftereach not running

 - also adds tests to verify that beforeach/aftereach/setup and teardown are run as expected

samples/Overview/source/rooibos.cat.brs

@@ -1370,7 +1370,7 @@ function RBS_ItG_GetRunnableTestSuite(group) as object
   runnableSuite.SetUp = RBS_CMN_GetFunction(group.setupFunction, group.setupFunctionName)
   runnableSuite.TearDown =  RBS_CMN_GetFunction(group.teardownFunction, group.teardownFunctionName)
   runnableSuite.BeforeEach =  RBS_CMN_GetFunction(group.beforeEachFunction, group.beforeEachFunctionName) 
-  runnableSuite.AftrEach =  RBS_CMN_GetFunction(group.afterEachFunction, group.afterEachFunctionName) 
+  runnableSuite.AfterEach =  RBS_CMN_GetFunction(group.afterEachFunction, group.afterEachFunctionName) 
   return runnableSuite
 end function
 Function ItemGenerator(scheme as object) as Object
@@ -2023,7 +2023,7 @@ sub RBS_RT_RunTestCases(metaTestSuite, itGroup, testSuite, totalStatObj, config,
     end if
     testStatObj.Time = testTimer.TotalMilliseconds()
     RBS_STATS_AppendTestStatistic(suiteStatObj, testStatObj)
-    if RBS_CMN_IsFunction(testCase.afterEach)
+    if RBS_CMN_IsFunction(testSuite.afterEach)
       testSuite.afterEach()
     end if
     if testStatObj.Result <> "Success"

source/buildinfo.brs

@@ -1,6 +1,6 @@
 Function BuildDate()
-  return "Oct 1 2018 19:28:58"
+  return "Oct 1 2018 19:47:45"
 End Function
 Function BuildCommit()
-  return "91871e9"
+  return "e9f42a4"
 End Function

source/rooibos.cat.brs

@@ -1370,7 +1370,7 @@ function RBS_ItG_GetRunnableTestSuite(group) as object
   runnableSuite.SetUp = RBS_CMN_GetFunction(group.setupFunction, group.setupFunctionName)
   runnableSuite.TearDown =  RBS_CMN_GetFunction(group.teardownFunction, group.teardownFunctionName)
   runnableSuite.BeforeEach =  RBS_CMN_GetFunction(group.beforeEachFunction, group.beforeEachFunctionName) 
-  runnableSuite.AftrEach =  RBS_CMN_GetFunction(group.afterEachFunction, group.afterEachFunctionName) 
+  runnableSuite.AfterEach =  RBS_CMN_GetFunction(group.afterEachFunction, group.afterEachFunctionName) 
   return runnableSuite
 end function
 Function ItemGenerator(scheme as object) as Object
@@ -2023,7 +2023,7 @@ sub RBS_RT_RunTestCases(metaTestSuite, itGroup, testSuite, totalStatObj, config,
     end if
     testStatObj.Time = testTimer.TotalMilliseconds()
     RBS_STATS_AppendTestStatistic(suiteStatObj, testStatObj)
-    if RBS_CMN_IsFunction(testCase.afterEach)
+    if RBS_CMN_IsFunction(testSuite.afterEach)
       testSuite.afterEach()
     end if
     if testStatObj.Result <> "Success"

source/tests/issue_13_afterEach_not_running.brs

@@ -0,0 +1,28 @@
+'@TestSuite [RBSA] Rooibos before after tests
+
+'+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
+'@It tests before each and after each are running
+'+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
+
+'@BeforeEach
+function RBSA__BeforeEach() as void
+  ? "!!! Before"
+end function
+
+
+'@AfterEach
+function RBSA__AfterEach() as void
+    ? "!!! After"
+end function
+
+'@Test before after
+function RBSA__before_after() as void
+
+    assertResult = m.Fail("reason")
+
+    isFail = m.currentResult.isFail
+    m.currentResult.Reset()
+
+    m.AssertFalse(assertResult)
+    m.AssertTrue(isFail)
+end function

source/tests/verifyBeforeEachAfterEach.brs

@@ -0,0 +1,100 @@
+'@TestSuite [BEAER] BeforeEach and AfterEach Running
+
+'@Setup
+function BEAER_Setup() as void
+  ? "!!! Setup"
+end function
+
+
+'@TearDown
+function BEAER_TearDown() as void
+    ? "!!! TearDown"
+end function
+
+'@BeforeEach
+function BEAER_BeforeEach() as void
+  ? "!!! Before"
+end function
+
+
+'@AfterEach
+function BEAER_AfterEach() as void
+    ? "!!! After"
+end function
+
+'+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
+'@It tests group 1 - should have global before/after 
+'+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
+
+'@Test 1 
+function BEAER_group1_1() as void
+  m.AssertTrue(true)
+end function
+
+'@Test 2
+function BEAER_group1_2() as void
+  m.AssertTrue(true)
+end function
+
+'@Test 3
+'@Params["a"]
+'@Params["b"]
+'@Params["c"]
+function BEAER_group1_3(values) as void
+  m.AssertTrue(true)
+end function
+
+'+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
+'@It tests group 2 - should have group before after
+'+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
+
+'@BeforeEach
+function BEAER_group2_BeforeEach() as void
+  ? "!!! Before GROUP 2"
+end function
+
+
+'@AfterEach
+function BEAER_group2_AfterEach() as void
+    ? "!!! After GROUP 2"
+end function
+
+'@Test 1 
+function BEAER_group2_1() as void
+  m.AssertTrue(true)
+end function
+
+'@Test 2
+function BEAER_group2_2() as void
+  m.AssertTrue(true)
+end function
+
+'@Test 3
+'@Params["a"]
+'@Params["b"]
+'@Params["c"]
+function BEAER_group2_3(values) as void
+  m.AssertTrue(true)
+end function
+
+'+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
+'@It tests group 3 - should have global before/after
+'+++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
+
+'@Test 1 
+function BEAER_group3_1() as void
+  m.AssertTrue(true)
+end function
+
+'@Test 2
+function BEAER_group3_2() as void
+  m.AssertTrue(true)
+end function
+
+'@Test 3
+'@Params["a"]
+'@Params["b"]
+'@Params["c"]
+function BEAER_group3_3(values) as void
+  m.AssertTrue(true)
+end function

src/Rooibos_ItGroup.brs

@@ -66,7 +66,7 @@ function RBS_ItG_GetRunnableTestSuite(group) as object
   runnableSuite.SetUp = RBS_CMN_GetFunction(group.setupFunction, group.setupFunctionName)
   runnableSuite.TearDown =  RBS_CMN_GetFunction(group.teardownFunction, group.teardownFunctionName)
   runnableSuite.BeforeEach =  RBS_CMN_GetFunction(group.beforeEachFunction, group.beforeEachFunctionName) 
-  runnableSuite.AftrEach =  RBS_CMN_GetFunction(group.afterEachFunction, group.afterEachFunctionName) 
+  runnableSuite.AfterEach =  RBS_CMN_GetFunction(group.afterEachFunction, group.afterEachFunctionName) 
 
   return runnableSuite
 end function

src/Rooibos_TestRunner.brs

@@ -292,7 +292,7 @@ sub RBS_RT_RunTestCases(metaTestSuite, itGroup, testSuite, totalStatObj, config,
     testStatObj.Time = testTimer.TotalMilliseconds()
     RBS_STATS_AppendTestStatistic(suiteStatObj, testStatObj)
 
-    if RBS_CMN_IsFunction(testCase.afterEach)
+    if RBS_CMN_IsFunction(testSuite.afterEach)
       testSuite.afterEach()
     end if