hpi-swa-teaching / TabletopUI

73%

develop: 70%

LAST BUILD ON BRANCH ci_repair

branch: SELECT

CHANGE BRANCH

x

Sync Branches

• No branch selected
• InitialJoinNetwork
• JoinNetworking
• NW-Import
• character_sheet_serializer
• charsheet_ui
• ci_fix
• ci_repair
• closeAllWindows
• closeGainTakeDamageDialogs
• design
• develop
• develop_branch
• develop_test
• ivo-test
• load_store
• main
• multipanel
• saving_sheets
• state
• state_synchronisation
• styling
• swt26-g12/design
• swt26-g12/load_store
• swt26-g12/saving_sheets
• swt26-g12/template
• swt26-g12/tests
• swt26-g14/ci_fix
• swt26-g14/ci_repair
• template
• tests
• toolbar

Build # 26472226495

Build Type

push

github

Committed by IvayloSta

Commit Message

Centralize state schema in TTGameStateKeys and make get/loadState JSON-safe

Introduce TTGameStateKeys in TabletopUI-Core as a class-side vocabulary
holding every state-serialization key as a named accessor. Route all
getState/loadState: methods across the state tree through it instead of
inline symbol literals, and finish the previously half-done loadState
restoration for fight and initiative contexts.

Production
- New TTGameStateKeys class with 17 schema-key accessors
- TTCharacter, TTActionLog, TTToken, TTMap, TTInputField, TTCommandParser,
  TTDefaultContext, TTFightContext, TTInitiativeContext all route
  get/loadState: through TTGameStateKeys instead of inline literals
- TTFightContext>>getState emits keyFightingPlayerNames as an Array of
  String (was emitting raw TTCharacter Morph objects under #fightingPlayers,
  which could not survive JSON serialization)
- TTInitiativeContext>>getState keys the rolls Dictionary by character
  name String instead of raw TTCharacter, for the same JSON-safety reason
- TTCommandParser>>loadState: restores fight order and initiative rolls
  (previously flagged as 'doesn't load real state'); characters are
  reconstructed from names with a default #player role - documented in
  the method's flag as a known follow-up

Tests
- State-related tests now reference TTGameStateKeys; previously commented
  or no-op bodies replaced with real value and round-trip assertions
- TTContextTestBase>>test01GetState renamed to assertContextHasGameStateKey
  to fix a Test Runner failure on the abstract base
- TTDefaultContextTest>>test04GetState restored
- TTFightContextTest, TTInitiativeContextTest, TTCharacterTest,
  TTActionLogTest, TTTokenTest, TTMapTest all updated to call the helper
  or assert real values
- TTMapTest>>mapInstantiationDict (115-line literal fixture) removed; the
  rewritten test methods construct their own data

TTActionLog's chat-synchronization additions (networkmanager wiring and
the registerMessage:/logI... (continued)

Coverage Stats

96 of 137 new or added lines in 31 files covered. (70.07%)

2 existing lines in 2 files now uncovered.

4305 of 5862 relevant lines covered (73.44%)

2.89 hits per line