integration test WIP

build.gradle

@@ -107,6 +107,7 @@ tasks.withType(Test) {
 apply plugin: 'java'
 apply plugin: 'org.springframework.boot'
 apply plugin: 'propdeps'
+apply plugin: 'idea'
 
 group = 'faforever'
 version = '0.9.1'
@@ -132,6 +133,12 @@ configurations {
   compile.exclude module: "assertj-core"
 }
 
+idea {
+  module {
+    testSourceDirs += file('src/inttest/java')
+  }
+}
+
 processResources {
   filesMatching('**/application.yml') {
     filter {
@@ -234,6 +241,27 @@ configurations.all {
   resolutionStrategy.cacheChangingModulesFor 60, 'seconds'
 }
 
+sourceSets {
+  inttest {
+    java.srcDir 'src/inttest/java'
+    resources.srcDir 'src/inttest/resources'
+    compileClasspath += main.output + test.output
+    runtimeClasspath += main.output + test.output
+  }
+}
+
+configurations {
+  inttestCompile.extendsFrom testCompile
+  inttestRuntime.extendsFrom testRuntime
+}
+
+
+task inttest(type: Test) {
+  testClassesDir = sourceSets.inttest.output.classesDir
+  classpath = sourceSets.inttest.runtimeClasspath
+  mustRunAfter test
+}
+
 dependencyManagement {
   dependencies {
     dependency("org.hibernate:hibernate-core:${hibernateVersion}")

src/inttest/java/com/faforever/api/config/MockConfiguration.java

@@ -0,0 +1,19 @@
+package com.faforever.api.config;
+
+import com.faforever.api.user.AnopeUserRepository;
+import org.springframework.context.annotation.Bean;
+import org.springframework.context.annotation.Configuration;
+import org.springframework.context.annotation.Primary;
+import org.springframework.context.annotation.Profile;
+
+import static org.mockito.Mockito.mock;
+
+@Profile("integration-test")
+@Configuration
+public class MockConfiguration {
+  @Bean
+  @Primary
+  public AnopeUserRepository anopeUserRepository() {
+    return mock(AnopeUserRepository.class);
+  }
+}

src/inttest/java/com/faforever/api/user/UserControllerTest.java

@@ -0,0 +1,111 @@
+package com.faforever.api.user;
+
+import com.faforever.api.data.domain.User;
+import com.faforever.api.error.ErrorCode;
+import com.faforever.api.security.OAuthScope;
+import com.faforever.integration.OAuthHelper;
+import com.google.common.collect.Sets;
+import org.json.JSONObject;
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.skyscreamer.jsonassert.JSONAssert;
+import org.springframework.beans.factory.annotation.Autowired;
+import org.springframework.boot.test.context.SpringBootTest;
+import org.springframework.boot.test.context.SpringBootTest.WebEnvironment;
+import org.springframework.context.annotation.Import;
+import org.springframework.http.HttpHeaders;
+import org.springframework.security.test.context.support.WithUserDetails;
+import org.springframework.test.context.ActiveProfiles;
+import org.springframework.test.context.jdbc.Sql;
+import org.springframework.test.context.jdbc.Sql.ExecutionPhase;
+import org.springframework.test.context.junit4.SpringRunner;
+import org.springframework.test.web.servlet.MockMvc;
+import org.springframework.test.web.servlet.MvcResult;
+import org.springframework.test.web.servlet.request.RequestPostProcessor;
+import org.springframework.test.web.servlet.setup.MockMvcBuilders;
+import org.springframework.util.MultiValueMap;
+import org.springframework.web.context.WebApplicationContext;
+
+import javax.transaction.Transactional;
+import java.util.Collections;
+
+import static junitx.framework.Assert.assertEquals;
+import static org.springframework.security.test.web.servlet.setup.SecurityMockMvcConfigurers.springSecurity;
+import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.post;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;
+
+@RunWith(SpringRunner.class)
+@SpringBootTest(webEnvironment = WebEnvironment.RANDOM_PORT)
+@ActiveProfiles("integration-test")
+@Import(OAuthHelper.class)
+@Transactional
+@Sql(executionPhase = ExecutionPhase.BEFORE_TEST_METHOD, scripts = "classpath:sql/createUsers.sql")
+public class UserControllerTest {
+  protected final static String AUTH_USER = "USER";
+  protected final static String AUTH_MODERATOR = "MODERATOR";
+  protected final static String AUTH_ADMIN = "ADMIN";
+  MockMvc mockMvc;
+  @Autowired
+  private WebApplicationContext context;
+  @Autowired
+  private OAuthHelper oAuthHelper;
+
+  @Autowired
+  private UserRepository userRepository;
+
+  @Before
+  public void setUp() {
+    this.mockMvc = MockMvcBuilders
+      .webAppContextSetup(this.context)
+      .apply(springSecurity())
+      .build();
+  }
+
+  void assertApiError(MvcResult mvcResult, ErrorCode errorCode) throws Exception {
+    JSONObject resonseJson = new JSONObject(mvcResult.getResponse().getContentAsString());
+    JSONAssert.assertEquals(String.format("{\"errors\":[{\"code\":\"%s\"}]}", errorCode.getCode()), resonseJson, false);
+  }
+
+  @Test
+  @WithUserDetails(AUTH_USER)
+  public void changePassword_success() throws Exception {
+    MultiValueMap<String, String> params = new HttpHeaders();
+    params.add("currentPassword", AUTH_USER);
+    params.add("newPassword", "newPassword");
+
+    RequestPostProcessor oauthToken = oAuthHelper.addBearerToken(Sets.newHashSet(OAuthScope._WRITE_ACCOUNT_DATA));
+    mockMvc.perform(post("/users/changePassword").with(oauthToken).params(params))
+      .andExpect(status().isOk());
+
+    User user = userRepository.findOneByLoginIgnoreCase(AUTH_USER).get();
+    assertEquals(user.getPassword(), "5c29a959abce4eda5f0e7a4e7ea53dce4fa0f0abbe8eaa63717e2fed5f193d31");
+  }
+
+  @Test
+  @WithUserDetails(AUTH_USER)
+  public void changePassword_wrongScope() throws Exception {
+    MultiValueMap<String, String> params = new HttpHeaders();
+    params.add("currentPassword", AUTH_USER);
+    params.add("newPassword", "newPassword");
+
+    RequestPostProcessor oauthToken = oAuthHelper.addBearerToken(Collections.emptySet());
+    mockMvc.perform(post("/users/changePassword").with(oauthToken).params(params))
+      .andExpect(status().isForbidden());
+  }
+
+  @Test
+  @WithUserDetails(AUTH_USER)
+  public void changePassword_wrongPassword() throws Exception {
+    MultiValueMap<String, String> params = new HttpHeaders();
+    params.add("currentPassword", "wrongPassword");
+    params.add("newPassword", "newPassword");
+
+    RequestPostProcessor oauthToken = oAuthHelper.addBearerToken(Sets.newHashSet(OAuthScope._WRITE_ACCOUNT_DATA));
+    MvcResult mvcResult = mockMvc.perform(post("/users/changePassword").with(oauthToken).params(params))
+      .andExpect(status().is4xxClientError())
+      .andReturn();
+
+    assertApiError(mvcResult, ErrorCode.PASSWORD_CHANGE_FAILED_WRONG_PASSWORD);
+  }
+}

src/inttest/resources/sql/createUsers.sql

@@ -0,0 +1,14 @@
+DELETE FROM oauth_clients;
+DELETE FROM login;
+
+INSERT INTO oauth_clients (id, name, client_secret, client_type, redirect_uris, default_redirect_uri, default_scope)
+VALUES ('test', 'test', 'test', 'public', 'http://localhost', 'http://localhost',
+        'read_events read_achievements upload_map upload_mod write_account_data');
+
+INSERT INTO login (id, login, email, password)
+VALUES (1, 'USER', 'user@faforever.com', '92b7b421992ef490f3b75898ec0e511f1a5c02422819d89719b20362b023ee4f');
+INSERT INTO login (id, login, email, password)
+VALUES (2, 'MODERATOR', 'moderator@faforever.com', '778ac5b81fa251b450f827846378739caee510c31b01cfa9d31822b88bed8441');
+INSERT INTO login (id, login, email, password)
+VALUES (3, 'ADMIN', 'admin@faforever.com', '835d6dc88b708bc646d6db82c853ef4182fabbd4a8de59c213f2b5ab3ae7d9be');
+

src/main/java/com/faforever/api/error/GlobalControllerExceptionHandler.java

@@ -2,6 +2,7 @@
 
 import lombok.extern.slf4j.Slf4j;
 import org.springframework.http.HttpStatus;
+import org.springframework.security.access.AccessDeniedException;
 import org.springframework.security.authentication.InsufficientAuthenticationException;
 import org.springframework.web.bind.MissingServletRequestParameterException;
 import org.springframework.web.bind.annotation.ControllerAdvice;
@@ -91,6 +92,22 @@ public ErrorResponse processProgrammingError(ProgrammingError ex) {
     return response;
   }
 
+
+  @ExceptionHandler(AccessDeniedException.class)
+  @ResponseStatus(HttpStatus.FORBIDDEN)
+  @ResponseBody
+  public ErrorResponse processAccessDeniedException(Throwable ex) throws MissingServletRequestPartException {
+    log.debug("Access denied", ex);
+
+    ErrorResponse response = new ErrorResponse();
+    response.addError(new ErrorResult(
+      String.valueOf(HttpStatus.FORBIDDEN.value()),
+      ex.getClass().getName(),
+      ex.getMessage()
+    ));
+    return response;
+  }
+
   @ExceptionHandler(Exception.class)
   @ResponseStatus(HttpStatus.INTERNAL_SERVER_ERROR)
   @ResponseBody

src/main/java/com/faforever/api/user/UserController.java

@@ -33,14 +33,14 @@ public void activate(@RequestParam("token") String token) {
     userService.activate(token);
   }
 
-  @PreAuthorize("#oauth2.hasScope('change_password') and hasRole('ROLE_USER')")
+  @PreAuthorize("#oauth2.hasScope('write_account_data') and hasRole('ROLE_USER')")
   @ApiOperation("Changes the password of a previously registered account.")
   @RequestMapping(path = "/changePassword", method = RequestMethod.POST, produces = MediaType.APPLICATION_JSON_VALUE)
   public void changePassword(@RequestParam("currentPassword") String currentPassword, @RequestParam("newPassword") String newPassword, Authentication authentication) {
     userService.changePassword(currentPassword, newPassword, userService.getUser(authentication));
   }
 
-  @PreAuthorize("#oauth2.hasScope('change_login') and hasRole('ROLE_USER')")
+  @PreAuthorize("#oauth2.hasScope('write_account_data') and hasRole('ROLE_USER')")
   @ApiOperation("Changes the login of a previously registered account.")
   @RequestMapping(path = "/changeUsername", method = RequestMethod.POST, produces = MediaType.APPLICATION_JSON_VALUE)
   public void changeLogin(@RequestParam("newUsername") String newUsername, Authentication authentication) {