Skip to content

Commit

Permalink
faf-java-api-362 Fix after rebase
Browse files Browse the repository at this point in the history
  • Loading branch information
bukajsytlos committed Dec 6, 2020
1 parent 221bcb8 commit 2dff99a
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 5 deletions.
9 changes: 5 additions & 4 deletions src/main/java/com/faforever/api/clan/ClanService.java
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@

import java.time.Instant;
import java.time.temporal.ChronoUnit;
import java.util.List;
import java.util.Set;

@Service
@RequiredArgsConstructor
Expand Down Expand Up @@ -58,9 +58,10 @@ public void preCreate(Clan clan) {
});

clan.setLeader(player);
clan.setMemberships(List.of(new ClanMembership()
.setClan(clan)
.setPlayer(player)));
final ClanMembership clanMembership = new ClanMembership();
clanMembership.setClan(clan);
clanMembership.setPlayer(player);
clan.setMemberships(Set.of(clanMembership));
}

@SneakyThrows
Expand Down
2 changes: 1 addition & 1 deletion src/test/java/com/faforever/api/clan/ClanServiceTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ void success() {
assertThat(clan.getFounder(), is(player));
assertThat(clan.getLeader(), is(player));
assertThat(clan.getMemberships().size(), is(1));
assertThat(clan.getMemberships().get(0).getPlayer(), is(player));
assertThat(clan.getMemberships().iterator().next().getPlayer(), is(player));

verify(playerService).getCurrentPlayer();
verify(clanRepository).findOneByName(any());
Expand Down

0 comments on commit 2dff99a

Please sign in to comment.