Skip to content

Commit cdc83c4

Browse files
test: add comprehensive tests for critical components
Add tests for previously untested critical areas: - QueueServerSelectorTest: Tests all load balancing strategies (ROUND_ROBIN, LEAST_PLAYERS, RANDOM), edge cases like empty servers, offline servers, and strategy changes - ConfigMigratorTest: Tests V2 to V3 migration including placeholder updates, queueServer to queueServers conversion, and idempotent migration behavior - ConfigTest: Tests getQueueType permission matching, queue type ordering, findGroupByTarget case sensitivity, kickWhenDownServers placeholder expansion, and queue group configuration - MainCommandSharedTest: Tests command parsing, shadowban duration parsing (d/h/m/s), permission checks, tab completion, and documents the NumberFormatException bug with invalid durations Also adds missing Config setters for testing: - setServerName, setQueueLoadBalancing, setRawKickWhenDownServers
1 parent 397c3e4 commit cdc83c4

5 files changed

Lines changed: 1384 additions & 0 deletions

File tree

shared/src/main/java/net/pistonmaster/pistonqueue/shared/config/Config.java

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -595,6 +595,18 @@ public void setUsernameRegexMessage(String message) {
595595
this.usernameRegexMessage = message;
596596
}
597597

598+
public void setServerName(String serverName) {
599+
this.serverName = serverName;
600+
}
601+
602+
public void setQueueLoadBalancing(LoadBalancingStrategy strategy) {
603+
this.queueLoadBalancing = strategy;
604+
}
605+
606+
public void setRawKickWhenDownServers(List<String> servers) {
607+
this.rawKickWhenDownServers = new ArrayList<>(servers);
608+
}
609+
598610
private void rebuildKickWhenDownServers() {
599611
List<String> resolved = new ArrayList<>();
600612
for (String text : rawKickWhenDownServers) {

0 commit comments

Comments
 (0)