Skip to content

Commit

Permalink
Fixes 5804-is-burning regression test (#7124)
Browse files Browse the repository at this point in the history
* Fixes 5804-is-burning regression test
- Previously used a wait to allow the pig's fire ticks to update, but waits aren't allowed in tests, so now just uses the ignite effect

* Suggestions

Co-authored-by: Efy <35348263+Efnilite@users.noreply.github.com>

---------

Co-authored-by: Efy <35348263+Efnilite@users.noreply.github.com>
  • Loading branch information
cheeezburga and Efnilite authored Oct 3, 2024
1 parent 9e61f38 commit b785591
Showing 1 changed file with 12 additions and 7 deletions.
19 changes: 12 additions & 7 deletions src/test/skript/tests/regressions/5804-is-burning.sk
Original file line number Diff line number Diff line change
@@ -1,9 +1,14 @@
# This test is to see if the `is burning` condition works properly.
# Before, something like `if {x} is burning` would try to compare {x} to the burning damage cause
# instead of whether or not the entity was burning.
test "burning":
spawn a pig at event-location
set {_pig} to the last spawned pig
spawn a pig at event-location:
set {_pig} to entity

assert "burning" parsed as damage cause is burning with "burning damage cause compare"
set burning time of {_pig} to 9000 ticks
wait a tick
assert entity within {_pig} is burning with "is burning failed"
assert {_pig} is burning with "is burning failed ##2"
clear entity within {_pig}

ignite {_pig}
assert entity within {_pig} is burning with "entity within is burning failed"
assert {_pig} is burning with "is burning failed"

delete entity within {_pig}

0 comments on commit b785591

Please sign in to comment.