After methods should be skipped when exclusion used

Closes #1574

Suppose there are one or more configuration methods
which don’t belong to any group and for which “alwaysRun”
attribute is not set, and when the user specifies 
a valid exclusion list for groups, TestNG would still
execute those configuration methods.

Fixed this anomaly.

Author: krmahadevan

CHANGES.txt

@@ -1,4 +1,5 @@
 Current
+Fixed: GITHUB-1574: Before/After methods are not running when groups "include" in suite (Krishnan Mahadevan)
 Fixed: GITHUB-987: Parameters threadCount and parallel doesn't work with maven (Krishnan Mahadevan)
 Fixed: GITHUB-1472: Optimize DynamicGraph.getUnfinishedNodes (Krishnan Mahadevan & Nathan Reynolds)
 Fixed: GITHUB-1566: Invalid XML characters in Params in testng-results.xml (Krishnan Mahadevan)

src/main/java/org/testng/internal/XmlMethodSelector.java

@@ -275,6 +275,10 @@ private static boolean isIncluded(Collection<String> includedGroups, boolean noG
   }
 
   private static boolean isExcluded(Collection<String> excludedGroups, String... groups) {
+    boolean noGroups = (groups == null || groups.length == 0);
+    if (!excludedGroups.isEmpty() && noGroups) {
+      return true;
+    }
     return isMemberOf(excludedGroups, groups);
   }
 

src/test/java/test/groupinvocation/GroupSuiteTest.java

@@ -14,6 +14,7 @@
 import java.io.File;
 import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Collections;
 import java.util.List;
 
 import static org.assertj.core.api.Assertions.assertThat;
@@ -123,7 +124,7 @@ private void runWithSuite(String[] withSuiteFiles, boolean groupsWithXml, List<S
       test.setExcludedGroups(excludedTestGroups);
     }
 
-    tng.setXmlSuites(Arrays.asList(suite));
+    tng.setXmlSuites(Collections.singletonList(suite));
 
     InvokedMethodNameListener listener = new InvokedMethodNameListener();
     tng.addListener((ITestNGListener) listener);
@@ -133,6 +134,16 @@ private void runWithSuite(String[] withSuiteFiles, boolean groupsWithXml, List<S
     assertThat(listener.getInvokedMethodNames()).containsExactly(methods);
   }
 
+  @Test(description = "GITHUB-1574")
+  public void testToCheckNoConfigurationsAreExecuted() {
+    XmlSuite suite = createXmlSuite("suite");
+    XmlTest test = createXmlTest(suite, "test", Issue1574TestclassSample.class);
+    test.addExcludedGroup("sometest");
+    TestNG testng = create(suite);
+    testng.run();
+    assertThat(Issue1574TestclassSample.messages).containsExactly("anothertest","@AfterSuite");
+  }
+
   private static List<String> g(String... groups) {
     return Arrays.asList(groups);
   }

src/test/java/test/groupinvocation/Issue1574TestclassSample.java

@@ -0,0 +1,54 @@
+package test.groupinvocation;
+
+import org.testng.annotations.AfterMethod;
+import org.testng.annotations.AfterSuite;
+import org.testng.annotations.AfterTest;
+import org.testng.annotations.BeforeMethod;
+import org.testng.annotations.BeforeSuite;
+import org.testng.annotations.BeforeTest;
+import org.testng.annotations.Test;
+import org.testng.collections.Lists;
+
+import java.util.List;
+
+public class Issue1574TestclassSample {
+    public static List<String> messages = Lists.newArrayList();
+    @BeforeSuite
+    public void setupSuite() {
+        messages.add("@BeforeSuite");
+    }
+
+    @BeforeTest
+    public void setUpTest() {
+        messages.add("@BeforeTest");
+    }
+
+    @BeforeMethod
+    public void setUpMethod() {
+        messages.add("@BeforeMethod");
+    }
+
+    @AfterMethod
+    public void tearDownMethod() {
+        messages.add("@AfterMethod");
+    }
+
+    @AfterTest
+    public void tearDownTest() {
+        messages.add("@AfterTest");
+    }
+    @AfterSuite(alwaysRun = true)
+    public void tearDownSuite() {
+        messages.add("@AfterSuite");
+    }
+
+    @Test(groups = {"sometest"})
+    public void someTest() {
+        messages.add("sometest");
+    }
+
+    @Test(groups = {"anothertest"})
+    public void anothertest() {
+        messages.add("anothertest");
+    }
+}