The way Fields are stored violates the spec #35

New issue

Closed

opened 2021-12-08 19:55:14 +00:00 by costajon · 1 comment

costajon commented

2021-12-08 19:55:14 +00:00

(Migrated from github.zhaw.ch)

Description

Since tests in SiedlerGameTestBasic can be seen as sort of extension of our specification, it's essential that those tests pass. Looking at requirementLandPlacementTest's results tells that the current implementation of Fields is faulty or incorrect.

Steps to reproduce

Run requirementLandPlacementTest
Test expects GR but receives an object reference

Cause

Incorrect implementation of Fields or the way they are stored. Details are currently unknown.

Impact/Severity

Moderate to high severity
Since the exact cause currently unknown, this issue will either be trivially resolved or cause a lot of refactoring.

## Description Since tests in `SiedlerGameTestBasic` can be seen as sort of extension of our specification, it's essential that those tests pass. Looking at `requirementLandPlacementTest`'s results tells that the current implementation of Fields is faulty or incorrect. ## Steps to reproduce - Run `requirementLandPlacementTest` - Test expects `GR` but receives an object reference ## Cause Incorrect implementation of Fields or the way they are stored. Details are currently unknown. ## Impact/Severity **Moderate to high severity** Since the exact cause currently unknown, this issue will either be trivially resolved or cause a lot of refactoring.

thalmma5 commented

2021-12-09 13:20:16 +00:00

(Migrated from github.zhaw.ch)

Fixed by 31c76c3a0b

Fixed by 31c76c3a0b920ff715b8513fb3e36fb549a14d83