Skip to content

test: streamline use of NameGeneratorTestingEnvironment#31

Merged
keturn merged 2 commits into
developfrom
test/consistent-use-of-testing-environment
Apr 2, 2021
Merged

test: streamline use of NameGeneratorTestingEnvironment#31
keturn merged 2 commits into
developfrom
test/consistent-use-of-testing-environment

Conversation

@skaldarnar
Copy link
Copy Markdown
Contributor

@skaldarnar skaldarnar commented Apr 2, 2021

An attempt to address MovingBlocks/Terasology#4598 (comment)

CreatureNameProviderTest uses a HeadlessEnvironment, but for some reason does not make use of NameGeneratorTestingEnvironment as the other test classes there. That testing environment has more elaborate code in its setUpClass method:

https://github.com/Terasology/NameGenerator/blob/develop/src/test/java/org/terasology/namegenerator/NameGeneratorTestingEnvironment.java#L28-L46

Maybe this explains why this one test is failing with NUI v3.0.

I'm super confused which of those tests actually require a headless environment and which don't, to be honest.

@skaldarnar skaldarnar requested a review from keturn April 2, 2021 19:50
@skaldarnar
Copy link
Copy Markdown
Contributor Author

I think I've tested the following combinations now :face_with_monocle:

Result MovingBlocks/Terasology Terasology/NameGenerator
✔️ develop develop
✔️ develop test/consistent-use-of-testing-environment
nui-gestalt-separation develop
✔️ nui-gestalt-separation test/consistent-use-of-testing-environment

Copy link
Copy Markdown
Contributor

@keturn keturn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NameGeneratorTestingEnvironment doesn't look like it is wrong to use here, so sure. 👍

(still confused about how this could be somehow tied to NUI versions)

@keturn keturn merged commit 6d71a0f into develop Apr 2, 2021
@keturn keturn deleted the test/consistent-use-of-testing-environment branch April 2, 2021 20:06
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.

2 participants