[SDK] league level is shared between runners (!)

JBM · June 13, 2020, 10:15pm

The MultiplayerGameRunner also provides a setLeagueLevel method which you can use to test each league of your game. If left unspecified, the first level of the game will be run.

This is incorrect and has cost me a crazy amount of time pinning down.

If left unspecified, the last set value on any previous instanciation of the class is used:

github.com

CodinGame/codingame-game-engine/blob/master/runner/src/main/java/com/codingame/gameengine/runner/MultiplayerGameRunner.java#L35


 * </p>
 * 
 * @param leagueLevel
 *            the league level. 1 is the lowest level and default value.
 */


public void setLeagueLevel(int leagueLevel) {
    if (leagueLevel < 1 || leagueLevel >= 20) {
        throw new IllegalArgumentException("League level must be higher than 0 and lesser than 20");
    }
    System.setProperty("league.level", String.valueOf(leagueLevel));
}


/**
 * <p>
 * The seed is used to generated parameters such as width and height.<br>
 * If a seed is present in the given input, the input value should override the generated values.<br>
 * The seed will be sent to the GameManager.<br>
 * </p>
 * <p>
 * Typically, the seed is used to generated other parameters such as width and height, then those parameters are placed back in the

This ought to be at the very least correctly documented, at best changed to an instance variable.