#951

- fixed a logic flaw
AtlasOfLivingAustralia · May 19, 2024 · 2c69988 · 2c69988
1 parent 59fa0ba
commit 2c69988
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/grails-app/services/au/org/ala/ecodata/SpeciesReMatchService.groovy b/grails-app/services/au/org/ala/ecodata/SpeciesReMatchService.groovy
@@ -93,7 +93,7 @@ class SpeciesReMatchService {
 
     Map searchByName (String name, boolean addDetails = false, boolean useVernacularSearch = false ) {
         Map result
-        if (useVernacularSearch)
+        if (!useVernacularSearch)
             result = searchNameMatchingServer(name)
         else
             result = searchByVernacularNameOnNameMatchingServer(name)

diff --git a/src/test/groovy/au/org/ala/ecodata/SpeciesReMatchServiceSpec.groovy b/src/test/groovy/au/org/ala/ecodata/SpeciesReMatchServiceSpec.groovy
@@ -97,7 +97,7 @@ class SpeciesReMatchServiceSpec extends Specification implements ServiceUnitTest
                         "noIssue"
                 ]
         ]
-        service.webService.getJson(_) >> resp
+        service.webService.getJson({it.contains("search?q=")}) >> resp
         when:
         def result = service.searchByName("name")
 
@@ -109,7 +109,7 @@ class SpeciesReMatchServiceSpec extends Specification implements ServiceUnitTest
         def result2 = service.searchByName("name")
 
         then:
-        service.webService.getJson(_) >> resp
+        service.webService.getJson({it.contains("search?q=")}) >> resp
         result2 == [
                 scientificName: "Red",
                 commonName: null,
@@ -122,7 +122,7 @@ class SpeciesReMatchServiceSpec extends Specification implements ServiceUnitTest
         def result3 = service.searchByName("name", false, true)
 
         then:
-        service.webService.getJson(_) >> resp
+        service.webService.getJson({it.contains("searchByVernacularName")}) >> resp
         result3 == [
                 scientificName: "Red",
                 commonName: null,