Added missing event notification for TestUnitResults

Removed unused/duplicate event notifications

Author: Arpit-Shah

pom.xml

@@ -11,7 +11,7 @@
 	<modelVersion>4.0.0</modelVersion>
 	<groupId>com.theartos</groupId>
 	<artifactId>artos</artifactId>
-	<version>0.0.13</version>
+	<version>0.0.14-beta-1</version>
 
 	<!-- Organisation Info -->
 	<organization>

src/com/artos/framework/infra/TestContext.java

@@ -326,9 +326,6 @@ protected void generateUnitTestSummary(TestUnitObjectWrapper unit) {
 		notifyTestStatusUpdate(getCurrentUnitTestStatus(), null, "\n[" + TestStatus.getEnumName(getCurrentUnitTestStatus().getValue())
 				+ "]: " + unit.getTestUnitMethod().getName() + "(context) " + bugTrackingNum);
 		notifyTestUnitResult(unit, getCurrentUnitTestStatus(), null, unit.getBugTrackingNumber());
-		
-		// notify unit final outcome
-		notifyTestUnitOutcome(unit, getCurrentTestStatus());
 		// reset status for next test
 		resetUnitTestStatus();
 	}
@@ -466,9 +463,8 @@ public void generateStepTestSummary(BDDStep step) {
 		String bugTrackingNum = "".equals(unit.getBugTrackingNumber()) ? "" : " [Bug_Reference: " + unit.getBugTrackingNumber() + "]";
 		notifyTestStatusUpdate(getCurrentUnitTestStatus(), null, "\n[" + TestStatus.getEnumName(getCurrentUnitTestStatus().getValue())
 				+ "]: " + step.getStepDescription() + " " + bugTrackingNum);
-		
 		// notify step final outcome
-		notifyTestUnitOutcome(step, getCurrentTestStatus());
+		notifyTestUnitResult(step, getCurrentUnitTestStatus(), null, unit.getBugTrackingNumber());
 		// reset status for next test
 		resetUnitTestStatus();
 	}
@@ -741,18 +737,6 @@ private void notifyTestStatusUpdate(TestStatus testStatus, File snapshot, String
 		}
 	}
 
-	private void notifyTestUnitOutcome(TestUnitObjectWrapper unit, TestStatus testStatus) {
-		for (TestProgress listener : listenerList) {
-			listener.testUnitOutcome(unit, testStatus);
-		}
-	}
-	
-	private void notifyTestUnitOutcome(BDDStep step, TestStatus testStatus) {
-		for (TestProgress listener : listenerList) {
-			listener.testUnitOutcome(step, testStatus);
-		}
-	}
-
 	private void notifyTestResult(TestObjectWrapper t, TestStatus testStatus, File snapshot, String Msg) {
 		for (TestProgress listener : listenerList) {
 			listener.testResult(t, testStatus, snapshot, Msg);
@@ -770,7 +754,13 @@ private void notifyTestUnitResult(TestUnitObjectWrapper unit, TestStatus testSta
 			listener.testUnitResult(unit, testStatus, snapshot, Msg);
 		}
 	}
-
+	
+	private void notifyTestUnitResult(BDDStep step, TestStatus testStatus, File snapshot, String Msg) {
+		for (TestProgress listener : listenerList) {
+			listener.testUnitResult(step, testStatus, snapshot, Msg);
+		}
+	}
+	
 	private void notifyTestCaseSummary(String FQCN, String description) {
 		for (TestProgress listener : listenerList) {
 			listener.testCaseSummaryPrinting(FQCN, description);

src/com/artos/framework/listener/ExtentReportListener.java

@@ -584,13 +584,7 @@ public void testUnitResult(TestUnitObjectWrapper unit, TestStatus testStatus, Fi
 	}
 
 	@Override
-	public void testUnitOutcome(TestUnitObjectWrapper unit, TestStatus testStatus) {
-		// TODO Auto-generated method stub
-		
-	}
-
-	@Override
-	public void testUnitOutcome(BDDStep step, TestStatus testStatus) {
+	public void testUnitResult(BDDStep step, TestStatus testStatus, File snapshot, String description) {
 		// TODO Auto-generated method stub
 
 	}

src/com/artos/framework/listener/JUnitReportListener.java

@@ -612,13 +612,7 @@ public void testResult(BDDScenario scenario, TestStatus testStatus, File snapsho
 	}
 
 	@Override
-	public void testUnitOutcome(TestUnitObjectWrapper unit, TestStatus testStatus) {
-		// TODO Auto-generated method stub
-		
-	}
-
-	@Override
-	public void testUnitOutcome(BDDStep step, TestStatus testStatus) {
+	public void testUnitResult(BDDStep step, TestStatus testStatus, File snapshot, String description) {
 		// TODO Auto-generated method stub
 
 	}

src/com/artos/framework/listener/TestExecutionEventListener.java

@@ -475,13 +475,7 @@ public void testUnitResult(TestUnitObjectWrapper unit, TestStatus testStatus, Fi
 	}
 
 	@Override
-	public void testUnitOutcome(TestUnitObjectWrapper unit, TestStatus testStatus) {
-		// TODO Auto-generated method stub
-		
-	}
-
-	@Override
-	public void testUnitOutcome(BDDStep step, TestStatus testStatus) {
+	public void testUnitResult(BDDStep step, TestStatus testStatus, File snapshot, String description) {
 		// TODO Auto-generated method stub
 
 	}

src/com/artos/framework/listener/UDPReportListener.java

@@ -454,13 +454,7 @@ public void testResult(BDDScenario scenario, TestStatus testStatus, File snapsho
 	}
 
 	@Override
-	public void testUnitOutcome(TestUnitObjectWrapper unit, TestStatus testStatus) {
-		// TODO Auto-generated method stub
-		
-	}
-
-	@Override
-	public void testUnitOutcome(BDDStep step, TestStatus testStatus) {
+	public void testUnitResult(BDDStep step, TestStatus testStatus, File snapshot, String description) {
 		// TODO Auto-generated method stub
 
 	}

src/com/artos/interfaces/TestProgress.java

@@ -339,16 +339,6 @@ public interface TestProgress {
 
 	public void testUnitExecutionFinished(BDDStep step);
 
-	/**
-	 * Method is called when test unit outcome is finalised
-	 * 
-	 * @param unit test object wrapper
-	 * @see TestUnitObjectWrapper
-	 */
-	public void testUnitOutcome(TestUnitObjectWrapper unit, TestStatus testStatus);
-	
-	public void testUnitOutcome(BDDStep step, TestStatus testStatus);
-
 	// ==========================================================
 	// Test Skip
 	// ==========================================================
@@ -462,6 +452,7 @@ public interface TestProgress {
 	 * @see TestStatus
 	 */
 	public void testUnitResult(TestUnitObjectWrapper unit, TestStatus testStatus, File snapshot, String description);
+	public void testUnitResult(BDDStep step, TestStatus testStatus, File snapshot, String description);
 
 	/**
 	 * Method is called when test unit execution is finished and summary requires to be printed