-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
use mtt
action for integration tests
#361
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd be happier if test specific stuff would be in some sub dirs dedicated for tests, I mean similar how it was.
Anyway a lot cleaner than before overall and seems like just two files anyway.
There's only the |
Actually now that I look at it closer, those are only tests for |
Click for detailed source code test coverage reportTest coverage report for Technic CNC 87.39% in 11/14 files:
Test coverage report for technic chests 45.58% in 6/6 files:
Test coverage report for technic 66.88% in 97/98 files:
Raw test runner output for geeks:CNC:
Chests:
Technic:
|
This PR removes the manual integration-testing files and switches to the
mtt
action.It currently tests against latest engine release
5.8.0
and current master/main branches of the dependencies (game and mods)Note: The test against 5.5.0 is removed now due to some gh-action limitations (BuckarooBanzay/mtt#4) IMO: we should let go of the old engine versions anyway