commit 7e5c60a7d723f88224004c0074c35824c10f9c4a
author Lukáš Petrovický <lpetrovi@redhat.com> Sat Feb 15 13:33:48 2014 -0500
committer Lukáš Petrovický <lpetrovi@redhat.com> Sat Feb 15 13:33:48 2014 -0500
tree e6b1cabfc7d4c3b7e93dac1ca989d426924b93ec
parent 72b7bfa5b805b8f9f98f79c53d66c8c9f1ea5aff

Fixes issue #116: org.testng.internal.BaseTestMethod does respect general contract of Comparable

pom-test.xml

diff --git a/pom-test.xml b/pom-test.xml
index 0af9d07..8a29968 100644
--- a/pom-test.xml
+++ b/pom-test.xml
@@ -65,9 +65,14 @@
       <version>1.27</version>
     </dependency>
     <dependency>
+        <groupId>org.yaml</groupId>
+        <artifactId>snakeyaml</artifactId>
+        <version>1.12</version>
+    </dependency>
+    <dependency>
       <groupId>org.testng</groupId>
       <artifactId>testng</artifactId>
-      <version>6.8.6-SNAPSHOT</version>
+      <version>6.8.8-SNAPSHOT</version>
       <scope>test</scope>
     </dependency>
    </dependencies>

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

diff --git a/src/main/java/org/testng/internal/BaseTestMethod.java b/src/main/java/org/testng/internal/BaseTestMethod.java
index 8aa78a7..5257ebd 100755
--- a/src/main/java/org/testng/internal/BaseTestMethod.java
+++ b/src/main/java/org/testng/internal/BaseTestMethod.java
@@ -153,7 +153,9 @@
     int result = -2;

     Class<?> thisClass = getRealClass();

     Class<?> otherClass = ((ITestNGMethod) o).getRealClass();

-    if (thisClass.isAssignableFrom(otherClass)) {

+    if (this == o) {
+      result = 0;
+    } else if (thisClass.isAssignableFrom(otherClass)) {

       result = -1;

     } else if (otherClass.isAssignableFrom(thisClass)) {

       result = 1;