Fixed test #3440

Author: chrisala

src/test/groovy/au/org/ala/merit/DownloadControllerSpec.groovy

@@ -1,14 +1,17 @@
 package au.org.ala.merit
 
 import org.apache.http.HttpStatus
+import org.h2.engine.User
 import org.springframework.mock.web.MockMultipartFile
 import spock.lang.Specification
 import grails.testing.web.controllers.ControllerUnitTest
 
 class DownloadControllerSpec extends Specification implements ControllerUnitTest<DownloadController>{
 
     WebService webService = Mock(WebService)
+    UserService userService = Mock(UserService)
     def setup() {
+        controller.userService = userService
         controller.webService = webService
     }
 
@@ -20,6 +23,7 @@ class DownloadControllerSpec extends Specification implements ControllerUnitTest
         def resp = controller.get()
 
         then:
+        1 * userService.userIsSiteAdmin() >> true
         1 * webService.proxyGetRequest(_, {it.endsWith('download/uuid1234')}, true, true, _) >> [status:HttpStatus.SC_OK]
 
         and: "We return null to inform grails to not attempt to process a view as we are proxying a response from ecodata"
@@ -32,6 +36,7 @@ class DownloadControllerSpec extends Specification implements ControllerUnitTest
         controller.get()
 
         then:
+        1 * userService.userIsSiteAdmin() >> true
         response.status == HttpStatus.SC_BAD_REQUEST
     }
 
@@ -45,6 +50,7 @@ class DownloadControllerSpec extends Specification implements ControllerUnitTest
         controller.get()
 
         then:
+        1 * userService.userIsSiteAdmin() >> true
         1 * webService.proxyGetRequest(_, {it.contains('download/file')}, true, true, _) >> {
             resp, url, userId, apiKey, timeout ->
                 formatPassedToEcodata = url.endsWith(format)