Skip to content

Commit 29033e8

Browse files
committed
Use short syntax for empty collections
1 parent 054c342 commit 29033e8

6 files changed

Lines changed: 13 additions & 16 deletions

File tree

src/inttest/java/com/faforever/api/moderationreport/ModerationReportTest.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,6 @@
1515
import org.springframework.test.context.jdbc.Sql;
1616
import org.springframework.test.context.jdbc.Sql.ExecutionPhase;
1717

18-
import java.util.Collections;
1918
import java.util.Set;
2019

2120
import static com.faforever.api.data.JsonApiMediaType.JSON_API_MEDIA_TYPE;
@@ -375,7 +374,7 @@ public void userCanSeeOnlyOwnReports() throws Exception {
375374
@Test
376375
public void cannotUpdateReportedUsers() throws Exception {
377376
final ModerationReport updatedReportedUsersModerationReport = (ModerationReport) new ModerationReport()
378-
.setReportedUsers(Collections.emptySet())
377+
.setReportedUsers(Set.of())
379378
.setId("3");
380379
mockMvc.perform(
381380
patch("/data/moderationReport/3")

src/main/java/com/faforever/api/config/elide/ElideConfig.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
import java.time.Duration;
2222
import java.time.Instant;
2323
import java.time.OffsetDateTime;
24-
import java.util.Collections;
24+
import java.util.Map;
2525

2626
@Configuration
2727
public class ElideConfig {
@@ -81,7 +81,7 @@ public <T> T convert(Class<T> type, Object value) {
8181

8282
@Bean
8383
public EntityDictionary entityDictionary(AutowireCapableBeanFactory beanFactory) {
84-
final EntityDictionary entityDictionary = new EntityDictionary(Collections.emptyMap(), new Injector() {
84+
final EntityDictionary entityDictionary = new EntityDictionary(Map.of(), new Injector() {
8585
@Override
8686
public void inject(Object entity) {
8787
beanFactory.autowireBean(entity);

src/main/java/com/faforever/api/security/FafMultiTokenStore.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
import org.springframework.stereotype.Component;
1616

1717
import java.util.Collection;
18-
import java.util.Collections;
18+
import java.util.List;
1919
import java.util.Map;
2020
import java.util.Objects;
2121

@@ -112,12 +112,12 @@ public OAuth2AccessToken getAccessToken(OAuth2Authentication authentication) {
112112
@Override
113113
public Collection<OAuth2AccessToken> findTokensByClientIdAndUserName(String clientId, String userName) {
114114
// equal to JwtTokenStore
115-
return Collections.emptySet();
115+
return List.of();
116116
}
117117

118118
@Override
119119
public Collection<OAuth2AccessToken> findTokensByClientId(String clientId) {
120120
// equal to JwtTokenStore
121-
return Collections.emptySet();
121+
return List.of();
122122
}
123123
}

src/main/java/com/faforever/api/voting/VotingService.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,10 @@
1616
import javax.transaction.Transactional;
1717
import java.time.OffsetDateTime;
1818
import java.util.ArrayList;
19-
import java.util.Collections;
2019
import java.util.List;
2120
import java.util.Objects;
2221
import java.util.Optional;
22+
import java.util.Set;
2323

2424
@Service
2525
public class VotingService {
@@ -42,7 +42,7 @@ public void saveVote(Vote vote, Player player) {
4242
List<Error> errors = ableToVote(player, vote.getVotingSubject().getId());
4343

4444
if (vote.getVotingAnswers() == null) {
45-
vote.setVotingAnswers(Collections.emptySet());
45+
vote.setVotingAnswers(Set.of());
4646
}
4747

4848
VotingSubject subject = votingSubjectRepository.findById(vote.getVotingSubject().getId())

src/test/java/com/faforever/api/deployment/LegacyFeaturedModDeploymentTaskTest.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
import java.nio.file.Files;
2323
import java.nio.file.Path;
2424
import java.nio.file.StandardCopyOption;
25-
import java.util.Collections;
2625
import java.util.List;
2726
import java.util.Map;
2827
import java.util.Objects;
@@ -100,7 +99,7 @@ public void testRunNoFileIds() throws Exception {
10099
when(featuredModService.getFeaturedMods()).thenReturn(List.of(
101100
new FeaturedMod().setTechnicalName("faf")
102101
));
103-
when(featuredModService.getFileIds("faf")).thenReturn(Collections.emptyMap());
102+
when(featuredModService.getFileIds("faf")).thenReturn(Map.of());
104103

105104
Path dummyExe = repositoriesFolder.resolve("TemplateForgedAlliance.exe");
106105
createDummyExe(dummyExe);

src/test/java/com/faforever/api/security/FafTokenServiceTest.java

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@
1616
import java.nio.file.Files;
1717
import java.nio.file.Paths;
1818
import java.time.Duration;
19-
import java.util.Collections;
2019
import java.util.Map;
2120

2221
import static com.faforever.api.error.ApiExceptionMatcher.hasErrorCode;
@@ -83,7 +82,7 @@ public void setUp() throws Exception {
8382

8483
@Test
8584
public void createToken() throws Exception {
86-
String token = instance.createToken(FafTokenType.REGISTRATION, Duration.ofSeconds(100), Collections.emptyMap());
85+
String token = instance.createToken(FafTokenType.REGISTRATION, Duration.ofSeconds(100), Map.of());
8786

8887
Jwt jwt = JwtHelper.decodeAndVerify(token, rsaVerifier);
8988
Map<String, String> claims = objectMapper.readValue(jwt.getClaims(), new TypeReference<Map<String, String>>() {
@@ -113,7 +112,7 @@ public void createTokenWithAttributes() throws Exception {
113112

114113
@Test
115114
public void resolveToken() {
116-
String token = instance.createToken(FafTokenType.REGISTRATION, Duration.ofSeconds(100), Collections.emptyMap());
115+
String token = instance.createToken(FafTokenType.REGISTRATION, Duration.ofSeconds(100), Map.of());
117116
Map<String, String> result = instance.resolveToken(FafTokenType.REGISTRATION, token);
118117

119118
assertThat(result.size(), is(0));
@@ -133,15 +132,15 @@ public void resolveTokenWithAttributes() {
133132

134133
@Test
135134
public void resolveTokenExpired() {
136-
String token = instance.createToken(FafTokenType.REGISTRATION, Duration.ofSeconds(-1), Collections.emptyMap());
135+
String token = instance.createToken(FafTokenType.REGISTRATION, Duration.ofSeconds(-1), Map.of());
137136

138137
ApiException result = assertThrows(ApiException.class, () -> instance.resolveToken(FafTokenType.REGISTRATION, token));
139138
assertThat(result, hasErrorCode(ErrorCode.TOKEN_EXPIRED));
140139
}
141140

142141
@Test
143142
public void resolveTokenInvalidType() {
144-
String token = instance.createToken(FafTokenType.REGISTRATION, Duration.ofSeconds(-1), Collections.emptyMap());
143+
String token = instance.createToken(FafTokenType.REGISTRATION, Duration.ofSeconds(-1), Map.of());
145144

146145
ApiException result = assertThrows(ApiException.class, () -> instance.resolveToken(FafTokenType.PASSWORD_RESET, token));
147146
assertThat(result, hasErrorCode(ErrorCode.TOKEN_INVALID));

0 commit comments

Comments
 (0)