display disabled tests in the report (fixes allure-framework#49, via allure-framework#275)

baev · web-flow · commit 0fadfe3ff9b8 · 2018-11-14T23:55:04.000+03:00
diff --git a/allure-testng/src/main/java/io/qameta/allure/testng/AllureTestNg.java b/allure-testng/src/main/java/io/qameta/allure/testng/AllureTestNg.java
@@ -157,6 +157,21 @@ public void onStart(final ITestContext context) {
                 .map(ITestNGMethod::getTestClass)
                 .distinct()
                 .forEach(this::onBeforeClass);
+
+        context.getExcludedMethods().stream()
+                .filter(ITestNGMethod::isTest)
+                .filter(method -> !method.getEnabled())
+                .forEach(method -> createFakeResult(context, method));
+    }
+
+    protected void createFakeResult(final ITestContext context, final ITestNGMethod method) {
+        final org.testng.internal.TestResult result = new org.testng.internal.TestResult(
+                new Object(), method, null, context
+        );
+        final String uuid = UUID.randomUUID().toString();
+        final String parentUuid = UUID.randomUUID().toString();
+        startTestCase(result, parentUuid, uuid);
+        stopTestCase(uuid, null, null);
     }
 
     @Override
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
@@ -1022,6 +1022,20 @@ public void shouldProcessConfigurationFailure() {
                 .containsExactly("fail");
     }
 
+    @Feature("Disabled tests")
+    @Issue("49")
+    @Test
+    public void shouldDisplayDisabledTests() {
+        final AllureResults results = runTestNgSuites("suites/gh-49.xml");
+
+        assertThat(results.getTestResults())
+                .extracting(TestResult::getName, TestResult::getStatus)
+                .containsExactly(
+                        tuple("disabled", null)
+                );
+
+    }
+
     private AllureResults runTestNgSuites(final String... suites) {
         final Consumer<TestNG> emptyConfigurer = testNg -> {
         };
diff --git a/allure-testng/src/test/java/io/qameta/allure/testng/samples/DisabledTest.java b/allure-testng/src/test/java/io/qameta/allure/testng/samples/DisabledTest.java
@@ -0,0 +1,13 @@
+package io.qameta.allure.testng.samples;
+
+import org.testng.annotations.Test;
+
+/**
+ * @author charlie (Dmitry Baev).
+ */
+public class DisabledTest {
+
+    @Test(enabled = false)
+    public void disabled() {
+    }
+}
diff --git a/allure-testng/src/test/resources/suites/gh-49.xml b/allure-testng/src/test/resources/suites/gh-49.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">
+    <test name="gh-49">
+        <classes>
+            <class name="io.qameta.allure.testng.samples.DisabledTest"/>
+        </classes>
+    </test>
+</suite>
