eclipse-sailing-analytics/sailing-analytics
1
0
Fork 0
mirror of https://github.com/eclipse-sailing-analytics/sailing-analytics.git synced 2026-09-22 21:55:39 +00:00
Code Activity

fix for RemoveLeaderboardGroupTest to now changed behaviour of

Browse Source 
RemoveLeaderboardGroup
This commit is contained in:
Kai Börnert
2018-12-06 11:54:32 +01:00
parent 196ac83ec7
commit 68015a1728
1 changed files with 8 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
java/com.sap.sailing.server.test/src/com/sap/sailing/server/test/RemoveLeaderboardGroupTest.java
+8 -3
View File
@@ -103,12 +103,17 @@ public class RemoveLeaderboardGroupTest {
assertNull(pfingstbusch.getLeaderboardGroups().iterator().next().getOverallLeaderboard());
}
/**
* Unlike earlier versions, the RemoveLeaderboardGroup no longer removes the OverallLeaderboard, because it cannot
* ensure proper Ownership handling. This is no handled in the SailingServiceImpl
*/
@Test
public void testRemovingLeaderboardGroupRemovesOverallLeaderboardAndUnlinksItFromEvent() {
public void testRemovingLeaderboardGroupDoesNotRemoveOverallLeaderboard() {
assertEquals(1, Util.size(pfingstbusch.getLeaderboardGroups()));
server.apply(new RemoveLeaderboardGroup("Pfingstbusch"));
final Leaderboard overallLeaderboard = server.getLeaderboardByName("Pfingstbusch "+LeaderboardNameConstants.OVERALL);
assertNull(overallLeaderboard);
final Leaderboard overallLeaderboard = server
.getLeaderboardByName("Pfingstbusch " + LeaderboardNameConstants.OVERALL);
assertNotNull(overallLeaderboard);
assertFalse(pfingstbusch.getLeaderboardGroups().iterator().hasNext());
}
}