gregandClaude Opus 5.5
944b5fdd3e
The payload builder filled anything matching the validator's broad credential pattern that wasn't a non-empty string, so RCON_PORT, ENABLE_RCON and ARK's RCONEnabled/RCONPort became 'boot-123456' in the first runner run. Only fill keys named like passwords (or x-generate), and only when missing or an empty string. Adds a test. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
67 lines
1.7 KiB
Go
67 lines
1.7 KiB
Go
package main
|
|
|
|
import (
|
|
"encoding/json"
|
|
"os"
|
|
"path/filepath"
|
|
"strings"
|
|
"testing"
|
|
)
|
|
|
|
func TestBootTestPayloadOnlyFillsBlankPasswords(t *testing.T) {
|
|
dir := t.TempDir()
|
|
game := filepath.Join(dir, "games", "demo")
|
|
if err := os.MkdirAll(game, 0o755); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
os.WriteFile(filepath.Join(game, "game.yaml"), []byte(`id: demo
|
|
name: Demo
|
|
defaultDockerImage: debian:12
|
|
installMethod: NONE
|
|
defaultConfig:
|
|
RCON_PORT: 25575
|
|
ENABLE_RCON: true
|
|
RCON_PASSWORD: ""
|
|
ServerPassword: ""
|
|
AdminPassword: kept-by-owner
|
|
MOTD: "{SERVER_NAME} on {PUBLIC_PORT_25565}"
|
|
configSchema:
|
|
properties:
|
|
RCON_PORT: {type: integer}
|
|
ENABLE_RCON: {type: boolean}
|
|
RCON_PASSWORD: {type: string, x-generate: {type: password}}
|
|
ServerPassword: {type: string}
|
|
AdminPassword: {type: string}
|
|
MOTD: {type: string}
|
|
`), 0o644)
|
|
|
|
out := filepath.Join(dir, "payload.json")
|
|
if err := runBootTestPayload([]string{"-games", filepath.Join(dir, "games"), "-out", out, "demo"}); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
raw, _ := os.ReadFile(out)
|
|
var payload struct{ GameConfig map[string]any }
|
|
if err := json.Unmarshal(raw, &payload); err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
c := payload.GameConfig
|
|
|
|
if c["RCON_PORT"] != float64(25575) {
|
|
t.Errorf("RCON_PORT changed to %v", c["RCON_PORT"])
|
|
}
|
|
if c["ENABLE_RCON"] != true {
|
|
t.Errorf("ENABLE_RCON changed to %v", c["ENABLE_RCON"])
|
|
}
|
|
for _, k := range []string{"RCON_PASSWORD", "ServerPassword"} {
|
|
if s, _ := c[k].(string); !strings.HasPrefix(s, "boot-") {
|
|
t.Errorf("%s not filled: %v", k, c[k])
|
|
}
|
|
}
|
|
if c["AdminPassword"] != "kept-by-owner" {
|
|
t.Errorf("AdminPassword overwritten: %v", c["AdminPassword"])
|
|
}
|
|
if c["MOTD"] != "gbr-boot-test on 25565" {
|
|
t.Errorf("placeholders not resolved: %v", c["MOTD"])
|
|
}
|
|
}
|