MQE-3122: Logging of failed test is not working when test fails in after section

Author: okolesnyk

dev/tests/verification/Resources/ActionGroupWithDataTest.txt

@@ -36,17 +36,6 @@ class ActionGroupWithDataTestCest
 $I->comment("Exiting Action Group [beforeGroup] FunctionalActionGroup");
 }
 
-/**
- * @param AcceptanceTester $I
- * @throws \Exception
- */
-public function _after(AcceptanceTester $I)
-{
-if ($this->isSuccess) {
-unlink(__FILE__);
-}
-}
-
 /**
  * @param AcceptanceTester $I
  * @throws \Exception
@@ -57,6 +46,9 @@ class ActionGroupWithDataTestCest
 $I->fillField("#foo", "myData1"); // stepKey: fillField1AfterGroup
 $I->fillField("#bar", "myData2"); // stepKey: fillField2AfterGroup
 $I->comment("Exiting Action Group [afterGroup] FunctionalActionGroup");
+if ($this->isSuccess) {
+unlink(__FILE__);
+}
 }
 
 /**

dev/tests/verification/Resources/ExtendedActionGroupReturningValueTest.txt

@@ -23,6 +23,17 @@ class ExtendedActionGroupReturningValueTestCest
  */
 private $isSuccess = false;
 
+/**
+ * @param AcceptanceTester $I
+ * @throws \Exception
+ */
+public function _after(AcceptanceTester $I)
+{
+if ($this->isSuccess) {
+unlink(__FILE__);
+}
+}
+
 /**
  * @Severity(level = SeverityLevel::CRITICAL)
  * @Features({"TestModule"})

dev/tests/verification/Resources/ExtendingSkippedTest.txt

@@ -24,17 +24,6 @@ class ExtendingSkippedTestCest
  */
 private $isSuccess = false;
 
-/**
- * @param AcceptanceTester $I
- * @throws \Exception
- */
-public function _after(AcceptanceTester $I)
-{
-if ($this->isSuccess) {
-unlink(__FILE__);
-}
-}
-
 /**
  * @Severity(level = SeverityLevel::CRITICAL)
  * @Features({"TestModule"})

dev/tests/verification/Resources/MergeSkip.txt

@@ -22,17 +22,6 @@ class MergeSkipCest
  */
 private $isSuccess = false;
 
-/**
- * @param AcceptanceTester $I
- * @throws \Exception
- */
-public function _after(AcceptanceTester $I)
-{
-if ($this->isSuccess) {
-unlink(__FILE__);
-}
-}
-
 /**
  * @Features({"TestModule"})
  * @Parameter(name = "AcceptanceTester", value="$I")

dev/tests/verification/Resources/PersistedReplacementTest.txt

@@ -26,20 +26,20 @@ class PersistedReplacementTestCest
  * @param AcceptanceTester $I
  * @throws \Exception
  */
-public function _after(AcceptanceTester $I)
+public function _before(AcceptanceTester $I)
 {
-if ($this->isSuccess) {
-unlink(__FILE__);
-}
+$I->createEntity("createData1", "hook", "ReplacementPerson", [], []); // stepKey: createData1
 }
 
 /**
  * @param AcceptanceTester $I
  * @throws \Exception
  */
-public function _before(AcceptanceTester $I)
+public function _after(AcceptanceTester $I)
 {
-$I->createEntity("createData1", "hook", "ReplacementPerson", [], []); // stepKey: createData1
+if ($this->isSuccess) {
+unlink(__FILE__);
+}
 }
 
 /**

dev/tests/verification/Resources/SkippedTest.txt

@@ -23,17 +23,6 @@ class SkippedTestCest
  */
 private $isSuccess = false;
 
-/**
- * @param AcceptanceTester $I
- * @throws \Exception
- */
-public function _after(AcceptanceTester $I)
-{
-if ($this->isSuccess) {
-unlink(__FILE__);
-}
-}
-
 /**
  * @Stories({"skipped"})
  * @Severity(level = SeverityLevel::MINOR)

dev/tests/verification/Resources/SkippedTestTwoIssues.txt

@@ -23,17 +23,6 @@ class SkippedTestTwoIssuesCest
  */
 private $isSuccess = false;
 
-/**
- * @param AcceptanceTester $I
- * @throws \Exception
- */
-public function _after(AcceptanceTester $I)
-{
-if ($this->isSuccess) {
-unlink(__FILE__);
-}
-}
-
 /**
  * @Stories({"skippedMultiple"})
  * @Severity(level = SeverityLevel::MINOR)

dev/tests/verification/Resources/SkippedTestWithHooks.txt

@@ -23,17 +23,6 @@ class SkippedTestWithHooksCest
  */
 private $isSuccess = false;
 
-/**
- * @param AcceptanceTester $I
- * @throws \Exception
- */
-public function _after(AcceptanceTester $I)
-{
-if ($this->isSuccess) {
-unlink(__FILE__);
-}
-}
-
 /**
  * @Stories({"skippedWithHooks"})
  * @Severity(level = SeverityLevel::MINOR)