Skip to content

fix a test failing on some cultures#268

Open
praschl wants to merge 2 commits into
YarnSpinnerTool:mainfrom
praschl:bug/test-requires-english-culture
Open

fix a test failing on some cultures#268
praschl wants to merge 2 commits into
YarnSpinnerTool:mainfrom
praschl:bug/test-requires-english-culture

Conversation

@praschl
Copy link
Copy Markdown

@praschl praschl commented Feb 13, 2024

  • Tests for the changes have been added (for bug fixes / features)
    • Does it pass all existing unit tests without modification?
  • CHANGELOG.md has been updated to describe this change

The test "VariableStorage_OnExternalChanges_ReturnsExpectedValue" failed when run on computers with german culture (and probably other cultures) because they format floats & decimals as "3,52", when the test expected "3.52"

Fixed by preformatting the expected values in the same culture.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant