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);
   }