@Factory with dataProvider changes order of iterations

When a test class is instantiated by a constructor with @Factory annotation. Example:

public class TestTest {
    int num;

    @Factory(dataProvider = "data")
    public TestTest(int n) {
        num = n;
    }

    @DataProvider
    public static Object [][] data() {
        return new Object[][] {{1}, {2}, {3}, {4}};
    }

    @Test
    public void test() {
        System.out.println(num);
    }
}

Expected output would be

1
2
3
4

but real order is different upon every execution. It appears that it essentially depends on test class instance hashcodes, where the list of nodes is sorted by default toString() implementation in org.testng.internal.Graph.java:129.

If dataprovider is specified in the @Test instead of @Factory, the order remains correct.

[juherr]

Workarround: add toString() to TestTest.

The Comparator is on Graph.Node and used during Grapg#topologicalSort.

A comment says:

// Sort the nodes alphabetically to make sure that methods of the same class // get run close to each other as much as possible

[juherr]

@JoaoGFarias I supposed we can remove it. Can you make a pull request with the toString removal?