Falling-Metal-Interactive/Blacksite
2
0
Fork 0
Code Issues 28 Pull Requests Actions Packages Projects Releases Wiki Activity

BS-JD-502 - Tool regression tests and undo invariants #33

New Issue
Open
opened 2026-06-06 01:49:05 -04:00 by Rbanh · 0 comments
Rbanh commented 2026-06-06 01:49:05 -04:00
Owner
Copy Link

Summary

Add reusable tests for every modal tool and every editor operator.

Why

The roadmap depends on consistent preview/commit/cancel/undo behavior across all tools.

Implementation notes

  • Create shared test harness for operators/modal tools.
  • Cover commit, cancel, dirty flag, helper cleanup, and undo grouping.
  • Add fixtures for asset drops, brush strokes, terrain strokes, and physics settle sessions.
  • Make failures identify the invariant that broke.

Acceptance criteria

  • Every production modal tool has commit/cancel/undo coverage.
  • Helper entities are verified absent after cancel/commit as appropriate.
  • Undo grouping is tested for continuous actions.
  • Test harness is easy to reuse for new tools.

Tests/docs

This ticket is test infrastructure. Document how to add tool regression tests.

## Summary Add reusable tests for every modal tool and every editor operator. ## Why The roadmap depends on consistent preview/commit/cancel/undo behavior across all tools. ## Implementation notes - Create shared test harness for operators/modal tools. - Cover commit, cancel, dirty flag, helper cleanup, and undo grouping. - Add fixtures for asset drops, brush strokes, terrain strokes, and physics settle sessions. - Make failures identify the invariant that broke. ## Acceptance criteria - [ ] Every production modal tool has commit/cancel/undo coverage. - [ ] Helper entities are verified absent after cancel/commit as appropriate. - [ ] Undo grouping is tested for continuous actions. - [ ] Test harness is easy to reuse for new tools. ## Tests/docs This ticket is test infrastructure. Document how to add tool regression tests.
Rbanh added this to the M5 - Regression, docs, and first-hour UX milestone 2026-06-06 01:49:05 -04:00
Rbanh added the
roadmap:jackdaw
type:architecture
priority:P0
area:quality
 labels 2026-06-06 01:49:05 -04:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels   
area:assets

Asset browser, material browser, thumbnails, import, processing, and asset watcher.

  
area:brushes

Brush geometry, CSG, face editing, UVs, and brush diagnostics.

  
area:extensions

Static extension API, extension scaffolds, dylib exploration.

  
area:governance

Policy, attribution, planning, and production process.

  
area:inspector

Inspector, component metadata, component editing UX.

  
area:operators

Editor operator/action framework, undo, commit/cancel semantics.

  
area:physics

Physics placement, colliders, rigid bodies, diagnostics, and overlays.

  
area:project-ux

Project launcher, profiles, templates, or first-run project workflow.

  
area:quality

Regression packs, tests, performance budgets, profiler, first-hour UX.

  
area:runtime

BRP, remote inspector, PIE, game preview, runtime integration.

  
area:scene-io

Scene document, save/load, schema, migration, or reference repair.

  
area:terrain

Terrain authoring, terrain sculpting, heightmaps, and terrain materials.

  
area:viewport

Viewport interaction, selection, shortcuts, camera, overlays, and controls.

  
priority:P0

Must-have prerequisite or milestone-critical item.

  
priority:P1

Important production feature after P0 work is stable.

  
priority:P2

Useful follow-up or polish item.

  
priority:P3

Exploratory or later spike.

  
roadmap:jackdaw

Jackdaw-inspired Blacksite editor roadmap item.

  
type:architecture

Architecture, internal framework, or hard-to-reverse technical decision.

  
type:docs

Documentation, policy, samples, or issue-board structure.

  
type:epic

Tracking epic that groups related milestone work.

  
type:feature

User-visible feature or capability.

No Label
area:assets
area:brushes
area:extensions
area:governance
area:inspector
area:operators
area:physics
area:project-ux
area:quality
area:runtime
area:scene-io
area:terrain
area:viewport
priority:P0
priority:P1
priority:P2
priority:P3
roadmap:jackdaw
type:architecture
type:docs
type:epic
type:feature
Milestone
Clear milestone
No items
No Milestone
M5 - Regression, docs, and first-hour UX
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Falling-Metal-Interactive/Blacksite#33
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?