Skip to content

Add gametests#25

Draft
arconyx wants to merge 11 commits into
mainfrom
add-gametests
Draft

Add gametests#25
arconyx wants to merge 11 commits into
mainfrom
add-gametests

Conversation

@arconyx

@arconyx arconyx commented Jun 3, 2026

Copy link
Copy Markdown
Owner

Time for better tests than me launching the game and bumbling about. Ideally we can test all the triggers we supply in a variety of situations.

Todo

  • Framework for imbuing hexes in tests
  • Making sure we can actually run hexes
  • Actual tests

@arconyx

arconyx commented Jun 3, 2026

Copy link
Copy Markdown
Owner Author

This is likely to break when we try to port to Fabric 1.21 and then Neoforge (#26) but I judge having a test suite to be worth the hassle. It should help ensure the ports actually work for one.

@arconyx arconyx force-pushed the add-gametests branch 3 times, most recently from cc066ad to 659a09a Compare June 8, 2026 03:31
arconyx added 6 commits June 8, 2026 15:52
This required switching to lenient... is it just utterly broken with Fabric?
Fabric doesn't appear to support them in 1.20.1. 1.20.4+ might work?
This allows references items in tests and other code.
Again, this allows accessing them through other code.
This should remain constant. There is no need for mutability.
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