workaround for issue testng-team/testng#2960

harmin-parra · Aug 16, 2023 · f9b5ba7 · f9b5ba7
1 parent e302e53
commit f9b5ba7
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/src/com/github/report4s/TestListener.java b/src/com/github/report4s/TestListener.java
@@ -52,6 +52,10 @@ public void onTestStart(ITestResult result) {
 
         //Force the test to skip if the necessary conditions are met.
         if (SuiteListener.test_failure && Report4s.skipSuiteAfterTestFailure) {
+            if (Utils.getDependencies(result).length > 0) {
+                startTestReport(result);
+                return;
+            }
             result.setEndMillis(result.getStartMillis());
             result.setStatus(ITestResult.SKIP);
             HtmlWriter.printTestTitle(result);

diff --git a/src/com/github/report4s/Utils.java b/src/com/github/report4s/Utils.java
@@ -95,6 +95,15 @@ protected static String getParameters(ITestResult result) {
             return null;
     }
 
+    /**
+     * Return the dependsOnMethods attribute of a test method.
+     * @param result The test under execution.
+     * @return The dependsOnMethods attribute of the @test annotation of the test method.
+     */
+    protected static String[] getDependencies(ITestResult result) {
+        return result.getMethod().getMethodsDependedUpon();
+    }
+
     /**
      * Return the success percentage of a test method.
      * @param result The test under execution.