Skip to content

Commit 55b4ce7

Browse files
committed
refactor(backend): renamed many things #541
1 parent 722eb8d commit 55b4ce7

4 files changed

Lines changed: 17 additions & 14 deletions

File tree

backend/src/main/java/ch/puzzle/pcts/controller/ConfigurationController.java

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
import ch.puzzle.pcts.dto.appconfiguration.AppConfigurationDto;
66
import ch.puzzle.pcts.dto.configuration.ConfigurationDto;
77
import ch.puzzle.pcts.mapper.AppConfigurationMapper;
8-
import ch.puzzle.pcts.mapper.ConfigurationMapper;
8+
import ch.puzzle.pcts.mapper.AuthorizationMapper;
99
import io.swagger.v3.oas.annotations.tags.Tag;
1010
import org.springframework.http.ResponseEntity;
1111
import org.springframework.web.bind.annotation.GetMapping;
@@ -16,26 +16,26 @@
1616
@RequestMapping("/api/v1/configuration")
1717
@Tag(name = "configuration", description = "Retrieve configuration properties for the application")
1818
public class ConfigurationController {
19-
private final ConfigurationMapper configMapper;
19+
private final AuthorizationMapper authMapper;
2020
private final AuthorizationConfiguration authConfig;
2121
private final AppConfigurationMapper appConfigMapper;
2222
private final AppConfiguration appConfig;
2323

24-
public ConfigurationController(ConfigurationMapper configMapper, AuthorizationConfiguration authConfig,
24+
public ConfigurationController(AuthorizationMapper authMapper, AuthorizationConfiguration authConfig,
2525
AppConfigurationMapper appConfigMapper, AppConfiguration appConfig) {
26-
this.configMapper = configMapper;
26+
this.authMapper = authMapper;
2727
this.authConfig = authConfig;
2828
this.appConfigMapper = appConfigMapper;
2929
this.appConfig = appConfig;
3030
}
3131

3232
@GetMapping("/authorization")
3333
public ResponseEntity<ConfigurationDto> getAuthorizationConfiguration() {
34-
return ResponseEntity.ok(configMapper.toDto(this.authConfig));
34+
return ResponseEntity.ok(authMapper.toDto(this.authConfig));
3535
}
3636

3737
@GetMapping("/app")
38-
public ResponseEntity<AppConfigurationDto> getHelpUrl() {
39-
return ResponseEntity.ok(appConfigMapper.toDto(this.appConfig.getHelpUrl()));
38+
public ResponseEntity<AppConfigurationDto> getAppConfiguration() {
39+
return ResponseEntity.ok(appConfigMapper.toDto(this.appConfig));
4040
}
4141
}

backend/src/main/java/ch/puzzle/pcts/dto/appconfiguration/AppConfigurationDto.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
import io.swagger.v3.oas.annotations.media.Schema;
44

5-
public record AppConfigurationDto(@Schema(description = "Provides the url the help button should lead to") String url) {
5+
public record AppConfigurationDto(
6+
@Schema(description = "Provides the URL the help button should lead to") String helpUrl) {
67

78
}
Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,12 @@
11
package ch.puzzle.pcts.mapper;
22

3+
import ch.puzzle.pcts.configuration.AppConfiguration;
34
import ch.puzzle.pcts.dto.appconfiguration.AppConfigurationDto;
45
import org.springframework.stereotype.Component;
56

67
@Component
78
public class AppConfigurationMapper {
8-
public AppConfigurationDto toDto(String url) {
9-
return new AppConfigurationDto(url);
9+
public AppConfigurationDto toDto(AppConfiguration appConfig) {
10+
return new AppConfigurationDto(appConfig.getHelpUrl());
1011
}
1112
}

backend/src/test/java/ch/puzzle/pcts/controller/ConfigurationControllerIT.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
import ch.puzzle.pcts.dto.appconfiguration.AppConfigurationDto;
1111
import ch.puzzle.pcts.dto.configuration.ConfigurationDto;
1212
import ch.puzzle.pcts.mapper.AppConfigurationMapper;
13-
import ch.puzzle.pcts.mapper.ConfigurationMapper;
13+
import ch.puzzle.pcts.mapper.AuthorizationMapper;
1414
import ch.puzzle.pcts.util.JsonDtoMatcher;
1515
import java.util.List;
1616
import org.junit.jupiter.api.BeforeEach;
@@ -32,7 +32,7 @@ class ConfigurationControllerIT extends ControllerITBase {
3232
private AppConfiguration appConfiguration;
3333

3434
@MockitoBean
35-
private ConfigurationMapper configMapper;
35+
private AuthorizationMapper configMapper;
3636

3737
@MockitoBean
3838
private AppConfigurationMapper appConfigurationMapper;
@@ -68,13 +68,14 @@ void shouldSuccessfullyGetConfiguration() throws Exception {
6868
@DisplayName("Should successfully get support page url")
6969
@Test
7070
void shouldSuccessfullyGetSupportPageUrl() throws Exception {
71+
AppConfiguration appConfig = new AppConfiguration(SUPPORT_URL);
7172
when(appConfiguration.getHelpUrl()).thenReturn(SUPPORT_URL);
72-
when(appConfigurationMapper.toDto(SUPPORT_URL)).thenReturn(appConfigurationDto);
73+
when(appConfigurationMapper.toDto(appConfig)).thenReturn(appConfigurationDto);
7374

7475
mvc
7576
.perform(get(BASEURL + "/app").with(csrf()).accept(MediaType.APPLICATION_JSON))
7677
.andExpect(status().isOk())
77-
.andExpect(JsonDtoMatcher.matchesDto(appConfigurationDto, "$"));
78+
.andExpect(JsonDtoMatcher.matchesDto(appConfigurationDto.helpUrl(), "$"));
7879
}
7980

8081
}

0 commit comments

Comments
 (0)