-
Notifications
You must be signed in to change notification settings - Fork 547
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
E2E testing #3884
E2E testing #3884
Conversation
Your Pull Request was automatically labelled as: "🎈 Feature" |
Slimefun preview buildA Slimefun preview build is available for testing! https://preview-builds.walshy.dev/download/Slimefun/3884/fb37d498
|
TODO list:
|
460ec9a
to
7387306
Compare
93ae01d
to
0a70324
Compare
d8de794
to
813985a
Compare
813985a
to
2f56579
Compare
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.
If anything goes wrong blame Jeff KEKW
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.
Totally not at gunpoint
🤣 |
Description
We want to start putting more velocity into PRs, this means we also want more validation around the PRs we want to merge in.
Unit tests are good but can't catch everything, they can all pass but the plugin fails to start for XYZ reason. Therefore, we want to start adding some e2e tests for these cases where unit tests cannot provide the validation we want.
Proposed changes
Adds a new workflow to test all supported MC versions (+ Java versions) in a live server environment.
This uses a new "e2e-tester" tool which is currently private within the org but will soon be open.
Checklist
Nonnull
andNullable
annotations to my methods to indicate their behaviour for null values