From 84b9b65777d9b5ba1faa44171ca66877540e6809 Mon Sep 17 00:00:00 2001 From: Rob Winch Date: Wed, 28 Apr 2021 11:41:02 -0500 Subject: [PATCH] Use headers instead of WithMockUser This ensures that we don't get false positives when the Reactor return types are not continuous. --- .../sample/graphql/EmployeeServiceApplicationTest.java | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/samples/employee-service/src/test/java/io/spring/sample/graphql/EmployeeServiceApplicationTest.java b/samples/employee-service/src/test/java/io/spring/sample/graphql/EmployeeServiceApplicationTest.java index c129f51f..54a32006 100644 --- a/samples/employee-service/src/test/java/io/spring/sample/graphql/EmployeeServiceApplicationTest.java +++ b/samples/employee-service/src/test/java/io/spring/sample/graphql/EmployeeServiceApplicationTest.java @@ -6,15 +6,12 @@ import org.springframework.beans.factory.annotation.Autowired; import org.springframework.boot.test.context.SpringBootTest; import org.springframework.boot.web.reactive.context.ReactiveWebApplicationContext; import org.springframework.http.MediaType; -import org.springframework.security.test.context.support.WithMockUser; import org.springframework.security.test.web.reactive.server.SecurityMockServerConfigurers; import org.springframework.test.web.reactive.server.WebTestClient; import org.springframework.web.reactive.function.client.ExchangeFilterFunctions; import java.util.Collections; -import static org.springframework.security.test.web.reactive.server.SecurityMockServerConfigurers.mockUser; - @SpringBootTest() class EmployeeServiceApplicationTest { @@ -78,7 +75,6 @@ class EmployeeServiceApplicationTest { } @Test - @WithMockUser("hasRole('ADMIN')") void canQuerySalaryAsAdmin() { String query = "{" + " employees{ " + @@ -89,6 +85,7 @@ class EmployeeServiceApplicationTest { client.post().uri("") + .headers(h -> h.setBasicAuth("admin", "admin")) .bodyValue("{ \"query\": \"" + query + "\"}") .exchange() .expectStatus().isOk()