From 9b4698f74867ea0c22b85163a4479aa2f56b1fbb Mon Sep 17 00:00:00 2001
From: jobayle <jonathan.bayle@gael.fr>
Date: Tue, 5 Mar 2019 10:42:43 +0100
Subject: [PATCH] Add a test scenario for @BeforeGroups having a
 dependsOnGroups

TestNG ignores the dependsongroups parameter on @BeforeGroups
annotations.
---
 .../BeforeGroupDependency.java                | 61 +++++++++++++++++++
 src/test/resources/testng.xml                 |  1 +
 2 files changed, 62 insertions(+)
 create mode 100644 src/test/java/test/beforegroupdeps/BeforeGroupDependency.java

diff --git a/src/test/java/test/beforegroupdeps/BeforeGroupDependency.java b/src/test/java/test/beforegroupdeps/BeforeGroupDependency.java
new file mode 100644
index 0000000000..3f7fe01fd6
--- /dev/null
+++ b/src/test/java/test/beforegroupdeps/BeforeGroupDependency.java
@@ -0,0 +1,61 @@
+package test.beforegroupdeps;
+
+import static org.testng.Assert.*;
+import org.testng.annotations.BeforeGroups;
+import org.testng.annotations.Test;
+
+public class BeforeGroupDependency {
+  private boolean z1run = false;
+  private boolean z2run = false;
+  private boolean setupBrun = false;
+  private boolean a1run = false;
+  private boolean a2run = false;
+
+  @Test(groups="Z")
+  public void z1() {
+    System.out.println("z1");
+    assertFalse(setupBrun);
+    assertFalse(a1run);
+    assertFalse(a2run);
+    z1run = true;
+  }
+
+  @Test(groups="Z")
+  public void z2() {
+    System.out.println("z2");
+    assertFalse(setupBrun);
+    assertFalse(a1run);
+    assertFalse(a2run);
+    z2run = true;
+  }
+
+  @BeforeGroups(value="A", dependsOnGroups="Z")
+  public void setupA() {
+    System.out.println("setupB");
+    assertTrue(z1run);
+    assertTrue(z2run);
+    assertFalse(a1run);
+    assertFalse(a2run);
+    setupBrun = true;
+  }
+
+  @Test(groups="A")
+  public void a1() {
+    System.out.println("a1");
+    assertTrue(z1run);
+    assertTrue(z2run);
+    assertTrue(setupBrun);
+    a1run = true;
+  }
+
+  @Test(groups="A")
+  public void a2() {
+    System.out.println("a2");
+    assertTrue(z1run);
+    assertTrue(z2run);
+    assertTrue(setupBrun);
+    a2run = true;
+  }
+
+}
+
diff --git a/src/test/resources/testng.xml b/src/test/resources/testng.xml
index c439a0009f..6ebbf1bc4b 100644
--- a/src/test/resources/testng.xml
+++ b/src/test/resources/testng.xml
@@ -191,6 +191,7 @@
       <class name="test.methodselection.MethodSelectionTest"/>
       <class name="test.beforegroups.BeforeGroupsTest"/>
       <class name="test.invocationcount.issue1719.IssueTest"/>
+      <class name="test.beforegroupdeps.BeforeGroupDependency"/>
     </classes>
   </test>