Issue 2467 (#2468)

* #2467

* #2467; test case for bug

* #2467; updated CHANGES.txt

CHANGES.txt

@@ -1,4 +1,5 @@
 Current
+Fixed: GITHUB-2467: XmlTest does not copy the xmlClasses during clone (C.V.Aditya)
 Fixed: GITHUB-2296: Fix for assertEquals not working for sets as order is not guaranteed. (Prashant Maroti)
 Fixed: GITHUB-2465: Fix bux where Strings.join returns empty String
 Fixed: GITHUB-1632: throwing SkipException sets iTestResult status to Failure instead of Skip (Julien Herr & Krishnan Mahadevan)

src/main/java/org/testng/xml/XmlTest.java

@@ -429,6 +429,7 @@ public Object clone() {
     result.setParameters(getLocalParameters());
     result.setXmlPackages(getXmlPackages());
     result.setTimeOut(getTimeOut());
+    result.setXmlClasses(getXmlClasses());
 
     Map<String, List<String>> metagroups = getMetaGroups();
     for (Map.Entry<String, List<String>> group : metagroups.entrySet()) {

src/test/java/org/testng/xml/XmlTestTest.java

@@ -5,6 +5,7 @@
 import org.testng.annotations.Test;
 import org.testng.collections.Maps;
 import test.SimpleBaseTest;
+import test.junitreports.SimpleTestSample;
 
 import java.util.Collections;
 import java.util.Map;
@@ -33,6 +34,17 @@ public Object[][] getData() {
     return new Object[][] {{newSetOfParameters(null, "value")}, {newSetOfParameters("foo", null)}};
   }
 
+  @Test(description = "GITHUB-2467")
+  public void testXMLClassesInCloneMethod() {
+    XmlSuite xmlSuite = createXmlSuite("suite");
+    XmlTest xmlTest = createXmlTest(xmlSuite, "test");
+    createXmlClass(xmlTest, SimpleTestSample.class);
+    XmlTest copyXmlTest = (XmlTest) xmlTest.clone();
+    Assert.assertNotNull(copyXmlTest);
+    Assert.assertNotNull(copyXmlTest.getXmlClasses());
+    Assert.assertEquals(xmlTest.getXmlClasses().size(), copyXmlTest.getXmlClasses().size());
+  }
+
   private static Map<String, String> newSetOfParameters(String key, String value) {
     Map<String, String> map = Maps.newHashMap();
     map.put(key, value);