fix: ANY http method takes preference when comparing

Author: eduardbosch

dispatcher/src/main/kotlin/pl/droidsonroids/testing/mockwebserver/condition/PathQueryCondition.kt

@@ -19,6 +19,7 @@ data class PathQueryCondition(
 
     override fun compareTo(other: Condition) = when {
         other == this -> 0
+        other is PathQueryCondition && httpMethod == HTTPMethod.ANY && httpMethod != other.httpMethod -> -1
         other is PathQueryCondition && score == other.score -> path.compareTo(other.path)
         other is PathQueryCondition && score > other.score -> -1
         else -> 1

dispatcher/src/test/kotlin/pl/droidsonroids/testing/mockwebserver/PathQueryConditionTest.kt

@@ -7,6 +7,7 @@ import org.assertj.core.api.Assertions.assertThat
 import org.junit.Before
 import org.junit.Test
 import pl.droidsonroids.testing.mockwebserver.condition.Condition
+import pl.droidsonroids.testing.mockwebserver.condition.HTTPMethod
 import pl.droidsonroids.testing.mockwebserver.condition.PathQueryCondition
 import pl.droidsonroids.testing.mockwebserver.condition.PathQueryConditionFactory
 
@@ -153,6 +154,17 @@ class PathQueryConditionTest {
             .isGreaterThan(0)
     }
 
+    @Test
+    fun `compareTo should return -1 when the first condition has ANY http method and the second is different`() {
+        val firstCondition = PathQueryConditionFactory()
+            .withPathSuffix("/abc", HTTPMethod.ANY)
+        val secondCondition = PathQueryConditionFactory()
+            .withPathSuffix("/abc", HTTPMethod.GET)
+
+        assertThat(firstCondition.compareTo(secondCondition))
+            .isEqualTo(-1)
+    }
+
     @Test
     fun `compareTo should return -1 when the first condition has a parameter name and the second one does not`() {
         val firstCondition = PathQueryConditionFactory()