add test for an issue (fixes allure-framework#304, via allure-framework#325)

baev · web-flow · commit dbb6ac0171f4 · 2019-02-12T21:22:38.000+03:00
diff --git a/allure-testng/src/test/java/io/qameta/allure/testng/AllureTestNgTest.java b/allure-testng/src/test/java/io/qameta/allure/testng/AllureTestNgTest.java
@@ -1041,6 +1041,32 @@ public void shouldProcessArrayParameters() {
                 );
     }
 
+    @SuppressWarnings("unchecked")
+    @AllureFeatures.Fixtures
+    @Issue("304")
+    @Test(dataProvider = "parallelConfiguration")
+    public void shouldProcessFailedSetUps(final XmlSuite.ParallelMode mode, final int threadCount) {
+        final AllureResults results = runTestNgSuites(parallel(mode, threadCount), "suites/gh-304.xml");
+
+        assertThat(results.getTestResults())
+                .extracting(TestResult::getName, TestResult::getStatus)
+                .contains(tuple("skippedTest", Status.SKIPPED));
+
+        assertThat(results.getTestResultContainers())
+                .flatExtracting(TestResultContainer::getAfters)
+                .extracting(FixtureResult::getName, FixtureResult::getStatus)
+                .contains(tuple("afterAlways", Status.PASSED));
+
+        assertThat(results.getTestResultContainers())
+                .flatExtracting(TestResultContainer::getAfters)
+                .filteredOn("name", "afterAlways")
+                .flatExtracting(FixtureResult::getSteps)
+                .extracting(StepResult::getName)
+                .containsExactly(
+                        "first", "second"
+                );
+    }
+
     private AllureResults runTestNgSuites(final String... suites) {
         final Consumer<TestNG> emptyConfigurer = testNg -> {
         };
diff --git a/allure-testng/src/test/java/io/qameta/allure/testng/samples/FailedSetUp.java b/allure-testng/src/test/java/io/qameta/allure/testng/samples/FailedSetUp.java
@@ -0,0 +1,43 @@
+/*
+ *  Copyright 2019 Qameta Software OÜ
+ *
+ *  Licensed under the Apache License, Version 2.0 (the "License");
+ *  you may not use this file except in compliance with the License.
+ *  You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ *  Unless required by applicable law or agreed to in writing, software
+ *  distributed under the License is distributed on an "AS IS" BASIS,
+ *  WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ *  See the License for the specific language governing permissions and
+ *  limitations under the License.
+ */
+package io.qameta.allure.testng.samples;
+
+import org.testng.annotations.AfterMethod;
+import org.testng.annotations.BeforeMethod;
+import org.testng.annotations.Test;
+
+import static io.qameta.allure.Allure.step;
+
+/**
+ * @author charlie (Dmitry Baev).
+ */
+public class FailedSetUp {
+
+    @BeforeMethod
+    public void setUp() {
+        throw new RuntimeException("Hey");
+    }
+
+    @Test
+    public void skippedTest() {
+    }
+
+    @AfterMethod(alwaysRun = true)
+    public void afterAlways() {
+        step("first");
+        step("second");
+    }
+}
diff --git a/allure-testng/src/test/resources/suites/gh-304.xml b/allure-testng/src/test/resources/suites/gh-304.xml
@@ -0,0 +1,10 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE suite SYSTEM "http://testng.org/testng-1.0.dtd" >
+
+<suite name="Github Issues" configfailurepolicy="continue">
+    <test name="gh-304">
+        <classes>
+            <class name="io.qameta.allure.testng.samples.FailedSetUp"/>
+        </classes>
+    </test>
+</suite>
