-
-
Notifications
You must be signed in to change notification settings - Fork 373
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
Test - Update for MC 1.19+ #7240
base: dev/feature
Are you sure you want to change the base?
Test - Update for MC 1.19+ #7240
Conversation
src/test/skript/tests/regressions/pull-5566-block iterator being 100 always.sk
Outdated
Show resolved
Hide resolved
I could be wrong, but I believe the test added from Equip's EffDetonate is the cause of those tests failing. As the explosion that's being detonated is at spawn where those 2 tests take place. |
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.
nice work
set block at location(0.5, 2.5, 0.5, "world") to {_oldBlock::3} | ||
|
||
test "lowest/highest solid block (new height)" when running minecraft "1.18": | ||
test "lowest/highest solid block (new height)": |
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.
test "lowest/highest solid block (new height)": | |
test "lowest/highest solid block": |
Description
This PR aims to update tests by removing outdated version checks since Skript 2.10+ will be only for MC 1.19.4+
NOTES:
boats.sk
was failing, so I temporarily commented out the line until that can be fixed (so tests can pass)Twas an Alias issue, Pickle informed me to update Aliases submodule on local and now it works.
4769-fire-visualeffect.sk
was failing. I think the fire block was being set mid air and wouldn't set, so I just added a block below itpull-5566...sk
was failing, it was getting the wrong blocks (probably edited in another test), so I shifted the test over 10 blocks along ZTarget Minecraft Versions: any
Requirements: none
Related Issues: none