Skip to content

Commit 8b507ad

Browse files
Copilotcommjoen
andcommitted
Fix code formatting issues with Spotless
Co-authored-by: commjoen <[email protected]>
1 parent ddd33ac commit 8b507ad

File tree

3 files changed

+12
-8
lines changed

3 files changed

+12
-8
lines changed

src/main/java/org/owasp/wrongsecrets/challenges/ChallengesController.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -204,8 +204,8 @@ public String reset(
204204
@PostMapping(value = "/challenge/{name}", params = "action=submit")
205205
@Operation(description = "Post your answer to the challenge for a given challenge")
206206
public String postController(
207-
@ModelAttribute ChallengeForm challengeForm,
208-
Model model,
207+
@ModelAttribute ChallengeForm challengeForm,
208+
Model model,
209209
@PathVariable String name,
210210
HttpServletRequest request) {
211211
var challengeDefinition = findByShortName(name);

src/main/java/org/owasp/wrongsecrets/challenges/docker/SlackNotificationService.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,10 @@ private boolean isSlackConfigured() {
8383

8484
private String buildCompletionMessage(String challengeName, String userName, String userAgent) {
8585
String userPart = (userName != null && !userName.trim().isEmpty()) ? " by " + userName : "";
86-
String userAgentPart = (userAgent != null && !userAgent.trim().isEmpty()) ? " (User-Agent: " + userAgent + ")" : "";
86+
String userAgentPart =
87+
(userAgent != null && !userAgent.trim().isEmpty())
88+
? " (User-Agent: " + userAgent + ")"
89+
: "";
8790

8891
return String.format(
8992
"🎉 Challenge %s completed%s%s! Another secret vulnerability discovered in WrongSecrets.",

src/test/java/org/owasp/wrongsecrets/challenges/docker/SlackNotificationServiceTest.java

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@
1212
import org.mockito.Mock;
1313
import org.mockito.junit.jupiter.MockitoExtension;
1414
import org.springframework.http.HttpEntity;
15-
import org.springframework.http.HttpHeaders;
1615
import org.springframework.http.ResponseEntity;
1716
import org.springframework.web.client.RestTemplate;
1817

@@ -69,9 +68,10 @@ void shouldIncludeUserAgentInMessageWhenProvided() {
6968
ArgumentCaptor<HttpEntity> entityCaptor = ArgumentCaptor.forClass(HttpEntity.class);
7069
verify(restTemplate, times(1))
7170
.postForEntity(eq(webhookUrl), entityCaptor.capture(), eq(String.class));
72-
71+
7372
HttpEntity capturedEntity = entityCaptor.getValue();
74-
SlackNotificationService.SlackMessage slackMessage = (SlackNotificationService.SlackMessage) capturedEntity.getBody();
73+
SlackNotificationService.SlackMessage slackMessage =
74+
(SlackNotificationService.SlackMessage) capturedEntity.getBody();
7575
assertTrue(slackMessage.getText().contains("(User-Agent: " + userAgent + ")"));
7676
}
7777

@@ -93,9 +93,10 @@ void shouldNotIncludeUserAgentInMessageWhenNotProvided() {
9393
ArgumentCaptor<HttpEntity> entityCaptor = ArgumentCaptor.forClass(HttpEntity.class);
9494
verify(restTemplate, times(1))
9595
.postForEntity(eq(webhookUrl), entityCaptor.capture(), eq(String.class));
96-
96+
9797
HttpEntity capturedEntity = entityCaptor.getValue();
98-
SlackNotificationService.SlackMessage slackMessage = (SlackNotificationService.SlackMessage) capturedEntity.getBody();
98+
SlackNotificationService.SlackMessage slackMessage =
99+
(SlackNotificationService.SlackMessage) capturedEntity.getBody();
99100
assertFalse(slackMessage.getText().contains("User-Agent"));
100101
}
101102

0 commit comments

Comments
 (0)