Skip to content

Add game tests for reverting mob head progression in the Purifier - #1259

Merged
rubensworks merged 1 commit into
master-1.21-ltsfrom
claude/evilcraft-issue-1258-gametest
Sep 6, 2026
Merged

rubensworks merged 1 commit into
master-1.21-ltsfrom
claude/evilcraft-issue-1258-gametest

Conversation

@rubensworks

Copy link
Copy Markdown
Member

Adds game tests only, for the mob head revert feature from #1258.

These tests fail on this branch until the implementation is upmerged from master-1.20-lts. That is intentional: the implementation lives in 2ac3fbf on master-1.20-lts, and these tests are here so the upmerge is covered the moment it lands. CI on this PR will be red until then.

Tests added to GameTestsPurifier

  • testPurifierMobHeadDowngrade — one Purifier per progression step, each given a head and a full tank of blood, asserting the Wither Skeleton Skull → Creeper Head → Zombie Head → Skeleton Skull revert chain and that the blood is consumed.
  • testPurifierMobHeadDowngradeEndOfProgression — Skeleton Skull, Player Head and Dragon Head are left untouched and their blood is not consumed, guarding against the action over-reaching past the end of the chain.

Verification

Run locally against a temporary port of the master-1.20-lts implementation:

  • With the implementation present: ./gradlew runGameTestServer → all 81 tests pass.
  • With the implementation removed (negative control): testpurifiermobheaddowngrade fails with Expected minecraft:wither_skeleton_skull to be purified into minecraft:creeper_head, but got minecraft:wither_skeleton_skull, confirming the test is not vacuously passing.

The temporary implementation port was not committed; this branch contains the test file change only.

🤖 Generated with Claude Code

https://claude.ai/code/session_01DwM8cKrtoseKe4UBtH6NLL


Generated by Claude Code

Covers the Wither Skeleton Skull -> Creeper Head -> Zombie Head ->
Skeleton Skull revert chain, and guards that heads without a previous
progression step are left untouched.

These tests only pass once the implementation is upmerged from
master-1.20-lts.

Closes #1258

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DwM8cKrtoseKe4UBtH6NLL
@rubensworks
rubensworks merged commit fd85719 into master-1.21-lts Sep 6, 2026
1 of 3 checks passed
@rubensworks
rubensworks deleted the claude/evilcraft-issue-1258-gametest branch September 6, 2026 11:54
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.

2 participants