Refactor unit tests and original clan create

FAForever · Nov 13, 2019 · 75de6d1 · 75de6d1
1 parent fba6a91
commit 75de6d1
Show file tree

Hide file tree

Showing 5 changed files with 181 additions and 271 deletions.
diff --git a/.idea/codeStyles/codeStyleConfig.xml b/.idea/codeStyles/codeStyleConfig.xml
diff --git a/.idea/compiler.xml b/.idea/compiler.xml
diff --git a/.idea/modules.xml b/.idea/modules.xml
diff --git a/src/main/java/com/faforever/api/clan/ClanService.java b/src/main/java/com/faforever/api/clan/ClanService.java
@@ -22,7 +22,6 @@
 
 import java.time.Instant;
 import java.time.temporal.ChronoUnit;
-import java.util.Collections;
 import java.util.List;
 
 @Service
@@ -67,35 +66,15 @@ public void preCreate(Clan clan) {
   @SneakyThrows
   @Transactional
   @Deprecated
-    // use preCreate instead
+    // use POST via Elide instead
   Clan create(String name, String tag, String description) {
-    Player creator = playerService.getCurrentPlayer();
-
-    if (creator.getClanMembership() != null) {
-      throw ApiException.of(ErrorCode.CLAN_CREATE_FOUNDER_IS_IN_A_CLAN);
-    }
-    if (clanRepository.findOneByName(name).isPresent()) {
-      throw ApiException.of(ErrorCode.CLAN_NAME_EXISTS, name);
-    }
-    if (clanRepository.findOneByTag(tag).isPresent()) {
-      throw ApiException.of(ErrorCode.CLAN_TAG_EXISTS, tag);
-    }
-
     Clan clan = new Clan();
     clan.setName(name);
     clan.setTag(tag);
     clan.setDescription(description);
+    clan.setFounder(playerService.getCurrentPlayer());
 
-    clan.setFounder(creator);
-    clan.setLeader(creator);
-
-    ClanMembership membership = new ClanMembership();
-    membership.setClan(clan);
-    membership.setPlayer(creator);
-
-    clan.setMemberships(Collections.singletonList(membership));
-
-    // clan membership is saved over cascading, otherwise validation will fail
+    // validation is done at preCreate() called by ClanListener
     clanRepository.save(clan);
     return clan;
   }