.golangci.yml is the prompts-repo standard verbatim plus one approved
exception (paralleltest, sneak 2026-07-06). Roughly 1,500 findings
fixed:
- autofix sweep (wsl_v5/nlreturn/intrange/modernize formatting), with
the misspell autofix REVERTED where it rewrote authentic C game text
("missle vanishes" stays, with nolint and a comment)
- error handling: errcheck sites get real handling — saveFile now
closes explicitly and removes corrupt saves, scoreboard writes are
documented best-effort, err113 sentinel errors (ErrSaveOutOfDate,
ErrScreenTooSmall); panics allowed for unrecoverable states per
MEMORY.md policy
- gosec: real fixes (ParseInt for SEED, 0600 scorefile) and justified
per-line nolints for provably-bounded conversions (randomMonsterLetter
helper collapses eight rnd(26)+'A' sites)
- API tidying from linters: pointer receivers on flag types
(recvcheck), msg helpers renamed addmsgf/doaddf/Printwf/MvPrintwf
(goprintffuncname), myExit()/findFloor(monst)/mkGameInput(t) drop
always-constant params (unparam), gameEnd carries no status
- gocritic/staticcheck: if-else chains to switches, the pack.c
inventory filter untangled into matchesFilter, main.go split so
defers run before exit (exitAfterDefer, funlen)
- revive doc comments on all exported flag types/consts/methods
Remaining findings are confined to linters pending sneak's exception
decision (mnd, gochecknoglobals, cyclop, nestif, gocognit, exhaustive,
goconst, testpackage); TODO.md records the state. MEMORY.md added at
repo root as the project's agent working notes.
140 lines
3.1 KiB
Go
140 lines
3.1 KiB
Go
package game
|
|
|
|
import (
|
|
"encoding/gob"
|
|
"os"
|
|
"path/filepath"
|
|
"testing"
|
|
)
|
|
|
|
func TestSaveRestoreRoundTrip(t *testing.T) {
|
|
g := mkGame(t, 4242)
|
|
// Dirty up some state so the round trip is meaningful.
|
|
g.Player.Purse = 123
|
|
g.Player.FoodLeft = 777
|
|
g.HasAmulet = true
|
|
g.Items.Potions[PotionHealing].Know = true
|
|
g.Items.Scrolls[ScrollMagicMapping].Guess = "map???"
|
|
|
|
g.Monsters['F'-'A'].Stats.Dmg = dice("3x1") // mutated bestiary must survive
|
|
if len(g.Level.Monsters) > 0 {
|
|
g.Level.Monsters[0].Flags.Set(Awake)
|
|
g.Level.Monsters[0].Dest = &g.Player.Pos
|
|
}
|
|
|
|
path := filepath.Join(t.TempDir(), "rogue.save")
|
|
|
|
saveErr := g.saveFile(path)
|
|
if saveErr != nil {
|
|
t.Fatalf("saveFile: %v", saveErr)
|
|
}
|
|
|
|
h, err := Restore(path, Config{Term: &testTerm{}})
|
|
if err != nil {
|
|
t.Fatalf("Restore: %v", err)
|
|
}
|
|
|
|
_, statErr := os.Stat(path)
|
|
if !os.IsNotExist(statErr) {
|
|
t.Error("save file not deleted on restore (C anti-restart rule)")
|
|
}
|
|
|
|
if h.Player.Purse != 123 || h.Player.FoodLeft != 777 {
|
|
t.Errorf("player state lost: purse=%d food=%d",
|
|
h.Player.Purse, h.Player.FoodLeft)
|
|
}
|
|
|
|
if !h.HasAmulet {
|
|
t.Error("amulet flag lost")
|
|
}
|
|
|
|
if !h.Items.Potions[PotionHealing].Know {
|
|
t.Error("potion identification lost")
|
|
}
|
|
|
|
if h.Items.Scrolls[ScrollMagicMapping].Guess != "map???" {
|
|
t.Error("scroll guess lost")
|
|
}
|
|
|
|
if h.Monsters['F'-'A'].Stats.Dmg.String() != "3x1" {
|
|
t.Error("mutated bestiary lost")
|
|
}
|
|
|
|
if h.Rng.Seed != g.Rng.Seed {
|
|
t.Error("RNG state lost")
|
|
}
|
|
|
|
if renderMap(h) != renderMap(g) {
|
|
t.Error("restored level map differs")
|
|
}
|
|
|
|
if len(h.Level.Monsters) != len(g.Level.Monsters) {
|
|
t.Fatalf("monster count %d != %d",
|
|
len(h.Level.Monsters), len(g.Level.Monsters))
|
|
}
|
|
|
|
if len(g.Level.Monsters) > 0 {
|
|
m := h.Level.Monsters[0]
|
|
if m.Dest != &h.Player.Pos {
|
|
t.Error("monster chase target not re-aliased to the hero")
|
|
}
|
|
|
|
if h.Level.MonsterAt(m.Pos.Y, m.Pos.X) != m {
|
|
t.Error("map monster index not rebuilt")
|
|
}
|
|
|
|
if m.Room == nil {
|
|
t.Error("monster room pointer not rebuilt")
|
|
}
|
|
}
|
|
// Equipment aliasing: the wielded mace must be the same *Object as the
|
|
// one in the pack.
|
|
st := g.snapshot()
|
|
t.Logf("snapshot indices: weapon=%d armor=%d rings=%v packlen=%d",
|
|
st.Player.CurWeapon, st.Player.CurArmor, st.Player.CurRing,
|
|
len(st.Player.Body.Pack))
|
|
t.Logf("restored: CurWeapon=%p pack has %d items", h.Player.CurWeapon,
|
|
len(h.Player.Pack))
|
|
|
|
found := false
|
|
|
|
for i, o := range h.Player.Pack {
|
|
t.Logf(" pack[%d]=%p type=%v which=%d", i, o, o.Kind, o.Which)
|
|
|
|
if o == h.Player.CurWeapon {
|
|
found = true
|
|
}
|
|
}
|
|
|
|
if !found {
|
|
t.Error("restored CurWeapon is not aliased into the pack")
|
|
}
|
|
}
|
|
|
|
func TestRestoreRejectsWrongVersion(t *testing.T) {
|
|
g := mkGame(t, 1)
|
|
path := filepath.Join(t.TempDir(), "rogue.save")
|
|
st := g.snapshot()
|
|
st.Version = "0.0.0"
|
|
|
|
f, err := os.Create(path) //nolint:gosec // G304: test temp path
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
encErr := gob.NewEncoder(f).Encode(st)
|
|
if encErr != nil {
|
|
t.Fatal(encErr)
|
|
}
|
|
|
|
closeErr := f.Close()
|
|
if closeErr != nil {
|
|
t.Fatal(closeErr)
|
|
}
|
|
|
|
_, restoreErr := Restore(path, Config{})
|
|
if restoreErr == nil {
|
|
t.Error("restore accepted an out-of-date save")
|
|
}
|
|
}
|