Skip to content

Commit a5427c8

Browse files
theEvilReapertheEvilReaper
committed
style(test): adjust bracket placement in tests
1 parent 076dbb7 commit a5427c8

8 files changed

Lines changed: 46 additions & 84 deletions

File tree

‎src/test/java/net/onelitefeather/labyrinth/commands/CenterCommandTest.java‎

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -16,8 +16,7 @@ class CenterCommandTest extends CommandPluginTestBase {
1616
private CenterCommand command;
1717
private MockValidationService validationService;
1818

19-
public static class MockValidationService implements ValidationService
20-
{
19+
public static class MockValidationService implements ValidationService {
2120
private boolean isValid;
2221

2322
@Override
@@ -36,8 +35,7 @@ public boolean isValid() {
3635

3736
@Override
3837
@BeforeEach
39-
void setUp()
40-
{
38+
void setUp() {
4139
super.setUp();
4240
validationService = new MockValidationService();
4341
command = new CenterCommand(plugin, validationService);
@@ -46,8 +44,7 @@ void setUp()
4644

4745
@DisplayName("Test if the player location is not zero")
4846
@Test
49-
void testIsYLocationFromPlayerNotZero()
50-
{
47+
void testIsYLocationFromPlayerNotZero() {
5148
var player = server.addPlayer();
5249
var location = new Location(new WorldMock(Material.GRASS_BLOCK, 64), 120, 64, 120);
5350
player.setLocation(location);
@@ -57,8 +54,7 @@ void testIsYLocationFromPlayerNotZero()
5754
}
5855

5956
@Test
60-
void testValidationWrong()
61-
{
57+
void testValidationWrong() {
6258
var player = server.addPlayer();
6359
var location = new Location(new WorldMock(Material.GRASS_BLOCK, 64), 120, 64, 120);
6460
player.setLocation(location);
@@ -74,8 +70,7 @@ void testValidationWrong()
7470
}
7571

7672
@Test
77-
void testValidationTrue()
78-
{
73+
void testValidationTrue() {
7974
var player = server.addPlayer();
8075
var location = new Location(new WorldMock(Material.GRASS_BLOCK, 64), 120, 64, 120);
8176
var zoneName = "Test";

‎src/test/java/net/onelitefeather/labyrinth/commands/CommandPluginTestBase.java‎

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -24,15 +24,13 @@ public void onDisable() {
2424
}
2525

2626
@BeforeEach
27-
void setUp()
28-
{
27+
void setUp() {
2928
server = MockBukkit.mock();
3029
plugin = MockBukkit.load(CommandPluginTestBase.MockLabyrinthPlugin.class);
3130
}
3231

3332
@AfterEach
34-
void tearDown()
35-
{
33+
void tearDown() {
3634
MockBukkit.unmock();
3735
}
3836
}

‎src/test/java/net/onelitefeather/labyrinth/commands/CreateZoneCommandTest.java‎

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -14,28 +14,25 @@ class CreateZoneCommandTest extends CommandPluginTestBase {
1414

1515
@Override
1616
@BeforeEach
17-
void setUp()
18-
{
17+
void setUp() {
1918
super.setUp();
2019
command = new CreateZoneCommand(plugin);
2120
}
2221

2322
@Test
24-
void testZoneNameNotMatchesPattern()
25-
{
23+
void testZoneNameNotMatchesPattern() {
2624
var player = server.addPlayer();
2725
var zoneName = "Test%";
2826
var expectedMessage = MiniMessage.miniMessage().deserialize(Constants.ZONE_INVALID_MESSAGE,
29-
Placeholder.component("prefix", Constants.PREFIX));
27+
Placeholder.component("prefix", Constants.PREFIX));
3028
command.createZone(player, zoneName);
3129

3230
assertFalse(plugin.getConfig().isSet(Constants.CONFIG_ZONE_PATH.formatted(zoneName)));
3331
assertEquals(expectedMessage, player.nextComponentMessage());
3432
}
3533

3634
@Test
37-
void testZoneCreated()
38-
{
35+
void testZoneCreated() {
3936
var player = server.addPlayer();
4037
var zoneName = "Test";
4138
var expectedMessage = MiniMessage.miniMessage().deserialize(Constants.CREATE_ZONE_MESSAGE_SUCCESS,

‎src/test/java/net/onelitefeather/labyrinth/commands/DeleteZoneCommandTest.java‎

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -8,26 +8,24 @@
88

99
import static org.junit.jupiter.api.Assertions.*;
1010

11-
class DeleteZoneCommandTest extends CommandPluginTestBase{
11+
class DeleteZoneCommandTest extends CommandPluginTestBase {
1212

1313
private DeleteZoneCommand command;
1414

1515
@Override
1616
@BeforeEach
17-
void setUp()
18-
{
17+
void setUp() {
1918
super.setUp();
2019
command = new DeleteZoneCommand(plugin);
2120
}
2221

2322
@Test
24-
void testZoneDeleted()
25-
{
23+
void testZoneDeleted() {
2624
var player = server.addPlayer();
2725
var zoneName = "Test";
2826
var expectedMessage = MiniMessage.miniMessage().deserialize(Constants.DELETE_ZONE_MESSAGE_SUCCESS,
29-
Placeholder.unparsed("zone", zoneName),
30-
Placeholder.component("prefix", Constants.PREFIX));
27+
Placeholder.unparsed("zone", zoneName),
28+
Placeholder.component("prefix", Constants.PREFIX));
3129
var configSectionName = Constants.CONFIG_ZONE_PATH.formatted(zoneName);
3230
plugin.getConfig().createSection(configSectionName);
3331
command.deleteZone(player, zoneName);
@@ -36,8 +34,7 @@ void testZoneDeleted()
3634
}
3735

3836
@Test
39-
void testZoneNotDeleted()
40-
{
37+
void testZoneNotDeleted() {
4138
var player = server.addPlayer();
4239
var zoneName = "Test";
4340
var expectedMessage = MiniMessage.miniMessage().deserialize(Constants.ZONE_INVALID_MESSAGE,

‎src/test/java/net/onelitefeather/labyrinth/commands/SetRadiusCommandTest.java‎

Lines changed: 8 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -14,13 +14,12 @@
1414

1515
import static org.junit.jupiter.api.Assertions.*;
1616

17-
class SetRadiusCommandTest extends CommandPluginTestBase{
17+
class SetRadiusCommandTest extends CommandPluginTestBase {
1818

1919
private SetRadiusCommand command;
2020
private SetRadiusCommandTest.MockValidationService validationService;
2121

22-
public static class MockValidationService implements ValidationService
23-
{
22+
public static class MockValidationService implements ValidationService {
2423
private boolean isValid;
2524

2625
@Override
@@ -39,16 +38,14 @@ public boolean isValid() {
3938

4039
@Override
4140
@BeforeEach
42-
void setUp()
43-
{
41+
void setUp() {
4442
super.setUp();
4543
validationService = new SetRadiusCommandTest.MockValidationService();
4644
command = new SetRadiusCommand(plugin, validationService);
4745
}
4846

4947
@Test
50-
void testValidationWrong()
51-
{
48+
void testValidationWrong() {
5249
var player = server.addPlayer();
5350
var zoneName = "Test";
5451
var expectedMessage = MiniMessage.miniMessage().deserialize(Constants.ZONE_INVALID_MESSAGE,
@@ -59,8 +56,7 @@ void testValidationWrong()
5956
}
6057

6158
@Test
62-
void testNoLocationSet()
63-
{
59+
void testNoLocationSet() {
6460
var player = server.addPlayer();
6561
var zoneName = "Test";
6662
validationService.setValid(true);
@@ -69,8 +65,7 @@ void testNoLocationSet()
6965
}
7066

7167
@Test
72-
void testRadiusSet()
73-
{
68+
void testRadiusSet() {
7469
var player = server.addPlayer();
7570
var zoneName = "Test";
7671
var expectedMessage = MiniMessage.miniMessage().deserialize(Constants.SET_RADIUS_MESSAGE,
@@ -84,11 +79,11 @@ void testRadiusSet()
8479
plugin.getConfig().set(Constants.CONFIG_ZONE_CENTER_PATH.formatted(zoneName), centerLocation);
8580
player.setLocation(playerLocation);
8681

87-
command.setRadius(player,zoneName);
82+
command.setRadius(player, zoneName);
8883

8984
playerLocation.setY(0);
9085
var expectedDistance = centerLocation.distance(playerLocation);
91-
assertEquals(expectedDistance, plugin.getConfig().getDouble(Constants.CONFIG_ZONE_RADIUS_PATH.formatted(zoneName)));
86+
assertEquals(expectedDistance, plugin.getConfig().getDouble(Constants.CONFIG_ZONE_RADIUS_PATH.formatted(zoneName)));
9287
assertEquals(expectedMessage, player.nextComponentMessage());
9388
}
9489

‎src/test/java/net/onelitefeather/labyrinth/commands/ToggleMobSpawnCommandTest.java‎

Lines changed: 7 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,12 @@
1111

1212
import static org.junit.jupiter.api.Assertions.*;
1313

14-
class ToggleMobSpawnCommandTest extends CommandPluginTestBase{
14+
class ToggleMobSpawnCommandTest extends CommandPluginTestBase {
1515

1616
private ToggleMobSpawnCommand command;
1717
private ToggleMobSpawnCommandTest.MockValidationService validationService;
1818

19-
public static class MockValidationService implements ValidationService
20-
{
19+
public static class MockValidationService implements ValidationService {
2120
private boolean isValid;
2221

2322
@Override
@@ -36,16 +35,14 @@ public boolean isValid() {
3635

3736
@Override
3837
@BeforeEach
39-
void setUp()
40-
{
38+
void setUp() {
4139
super.setUp();
4240
validationService = new ToggleMobSpawnCommandTest.MockValidationService();
4341
command = new ToggleMobSpawnCommand(plugin, validationService);
4442
}
4543

4644
@Test
47-
void testZoneNotValid()
48-
{
45+
void testZoneNotValid() {
4946
var player = server.addPlayer();
5047
var expectedMessage = MiniMessage.miniMessage().deserialize(Constants.ZONE_INVALID_MESSAGE,
5148
Placeholder.component("prefix", Constants.PREFIX));
@@ -57,8 +54,7 @@ void testZoneNotValid()
5754
}
5855

5956
@Test
60-
void testActivateMobSpawningWithNotSetInConfig()
61-
{
57+
void testActivateMobSpawningWithNotSetInConfig() {
6258
var player = server.addPlayer();
6359
var zoneName = "Test";
6460
var expectedMessage = MiniMessage.miniMessage().deserialize(Constants.TOGGLE_MOB_SPAWN_COMMAND_MESSAGE_SUCCESS,
@@ -73,8 +69,7 @@ void testActivateMobSpawningWithNotSetInConfig()
7369
}
7470

7571
@Test
76-
void testMobSpawningChangedToTrue()
77-
{
72+
void testMobSpawningChangedToTrue() {
7873
var player = server.addPlayer();
7974
var zoneName = "Test";
8075
var expectedMessage = MiniMessage.miniMessage().deserialize(Constants.TOGGLE_MOB_SPAWN_COMMAND_MESSAGE_SUCCESS,
@@ -90,8 +85,7 @@ void testMobSpawningChangedToTrue()
9085
}
9186

9287
@Test
93-
void testMobSpawningChangedToFalse()
94-
{
88+
void testMobSpawningChangedToFalse() {
9589
var player = server.addPlayer();
9690
var zoneName = "Test";
9791
var expectedMessage = MiniMessage.miniMessage().deserialize(Constants.TOGGLE_MOB_SPAWN_COMMAND_MESSAGE_SUCCESS,

‎src/test/java/net/onelitefeather/labyrinth/listener/MobspawnListenerTest.java‎

Lines changed: 8 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -38,22 +38,19 @@ public void onDisable() {
3838
}
3939

4040
@BeforeEach
41-
void setUp()
42-
{
41+
void setUp() {
4342
server = MockBukkit.mock();
4443
plugin = MockBukkit.load(MobspawnListenerTest.MockLabyrinthPlugin.class);
4544
listener = new MobspawnListener(this.plugin);
4645
}
4746

4847
@AfterEach
49-
void tearDown()
50-
{
48+
void tearDown() {
5149
MockBukkit.unmock();
5250
}
5351

5452
@Test
55-
void testNoZone()
56-
{
53+
void testNoZone() {
5754
this.server.getPluginManager().registerEvents(listener, this.plugin);
5855
var zombie = new ZombieMock(this.server, UUID.randomUUID());
5956
zombie.setLocation(new Location(new WorldMock(Material.GRASS_BLOCK, 64), 120, 64, 120));
@@ -63,8 +60,7 @@ void testNoZone()
6360
}
6461

6562
@Test
66-
void testNoCenterLocation()
67-
{
63+
void testNoCenterLocation() {
6864
this.server.getPluginManager().registerEvents(listener, this.plugin);
6965
var zombie = new ZombieMock(this.server, UUID.randomUUID());
7066
zombie.setLocation(new Location(new WorldMock(Material.GRASS_BLOCK, 64), 120, 64, 120));
@@ -77,8 +73,7 @@ void testNoCenterLocation()
7773
}
7874

7975
@Test
80-
void testNotInRadius()
81-
{
76+
void testNotInRadius() {
8277
this.server.getPluginManager().registerEvents(listener, this.plugin);
8378

8479
var world = new WorldMock(Material.GRASS_BLOCK, 64);
@@ -98,8 +93,7 @@ void testNotInRadius()
9893
}
9994

10095
@Test
101-
void testNotDisabled()
102-
{
96+
void testNotDisabled() {
10397
this.server.getPluginManager().registerEvents(listener, this.plugin);
10498

10599
var world = new WorldMock(Material.GRASS_BLOCK, 64);
@@ -120,8 +114,7 @@ void testNotDisabled()
120114
}
121115

122116
@Test
123-
void testNoSpawning()
124-
{
117+
void testNoSpawning() {
125118
this.server.getPluginManager().registerEvents(listener, this.plugin);
126119

127120
var world = new WorldMock(Material.GRASS_BLOCK, 64);
@@ -142,8 +135,7 @@ void testNoSpawning()
142135
}
143136

144137
@Test
145-
void testNoMonster()
146-
{
138+
void testNoMonster() {
147139
this.server.getPluginManager().registerEvents(listener, this.plugin);
148140

149141
var world = new WorldMock(Material.GRASS_BLOCK, 64);

0 commit comments

Comments
 (0)