Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#236 closed defect (released)

Race loading should fail cleanly if system doesn't exist

Reported by: ibboard Owned by:
Priority: major Milestone: WarFoundry 0.1
Component: WarFoundry-API Version: Unknown or N/A
Keywords: Cc:
Blocked By: Blocking:

Description

Currently if a race references a non-existent Game System then the CompleteLoading method causes a null reference exception when getting the parent system's categories. This shouldn't happen, and we should error in a cleaner way if a game system doesn't exist.

Change History (5)

comment:1 Changed 10 years ago by ibboard

(In r442) Re #236: Race loading should fail cleanly if system doesn't exist

  • Add tests for race factory
  • Add test data that references non-existstent system
  • Make game system names for existing test data more distinct

Re #228: Crash with missing abilityID

  • Initial failing test

comment:2 Changed 10 years ago by ibboard

Resolution: fixed
Status: newclosed

(In r443) Fixes #236: Race loading should fail cleanly if system doesn't exist

  • Exception if we get a null game system

Also:

  • Remove unused variable

comment:3 Changed 10 years ago by ibboard

(In r445) Re #236: Handle null game system in race

  • Fix error message text

no-open-ticket

comment:4 Changed 10 years ago by ibboard

released: noyes

Release as part of v0.1b6

comment:5 Changed 10 years ago by ibboard

Resolution: fixedreleased

Mark fix as released under a previous version

Note: See TracTickets for help on using tickets.