Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[WHO] Implement Everybody Lives! #13369

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

padfoothelix
Copy link
Contributor

@padfoothelix padfoothelix commented Feb 20, 2025

Implement [[Everybody Lives!]].

Part of #10653.

I created several common files to implement this ; I am not sure if this is the preferred way.
In GainAbilityAllPlayersEffect, I copied some logic from GainAbilityControllerEffect, including use of custom duration, but I'm not sure if it is necessary - how is it different from Duration.WhileOnBattlefield ?

As always, thanks for your work.

@padfoothelix
Copy link
Contributor Author

Hmm I have no idea why the tests failed during the first build… and why it worked on the second.

@JayDi85
Copy link
Member

JayDi85 commented Feb 21, 2025

Cause on first run master contained a failed verify test: 17e0c82

@padfoothelix
Copy link
Contributor Author

Yes, but what I don't get is that after I pulled from remote my own master branch now gives me a different test error :

[INFO] Standard out
Ability text checks started for DFT

Wrong card 3320: Brightglass Gearhulk

  • First strike
  • Trample
  • When Brightglass Gearhulk enters, you may search your library for up to two artifact, creature, and/or enchantment cards with mana value 1 or less, reveal them, put them into your hand, then shuffle.
    ref:
  • First strike
  • Trample
  • When this creature enters, you may search your library for up to two artifact, creature, and/or enchantment cards with mana value 1 or less, reveal them, put them into your hand, then shuffle.

Wrong card 3355: Earthrumbler

  • Crew 3
  • Trample
  • Vigilance
  • Exile an artifact or creature card from your graveyard: Earthrumbler becomes an artifact creature in addition to its other types until end of turn.
    ref:
  • Crew 3
  • Trample
  • Vigilance
  • Exile an artifact or creature card from your graveyard: This Vehicle becomes an artifact creature until end of turn.

........................

[INFO] Standard out
Error: unknown official set: DRC - Aetherdrift Commander (make sure it use correct set code or mark it as SetType.CUSTOM_SET)
Error: unknown official set: FIC - Final Fantasy Commander (make sure it use correct set code or mark it as SetType.CUSTOM_SET)
Error: unknown official set: FIN - Final Fantasy (make sure it use correct set code or mark it as SetType.CUSTOM_SET)
Error: unknown official set: PW25 - Wizards Play Network 2025 (make sure it use correct set code or mark it as SetType.CUSTOM_SET)
Error: wrong booster settings (set MUST HAVEN'T booster, but have) - 2025 - DFT - Aetherdrift
[INFO]
[INFO] Standard error
[INFO]
[INFO] Results:
[INFO]
[ERROR] Failures:
[ERROR] VerifyCardDataTest.test_checkMissingSetData:1095 Found set errors: 5
[ERROR] VerifyCardDataTest.test_verifyCards:321 found 2 errors in 81315 cards verify (see errors list above)
[INFO]
[ERROR] Tests run: 23, Failures: 2, Errors: 0, Skipped: 5

Compilation works with -DskipTests.
Any idea ?

@JayDi85
Copy link
Member

JayDi85 commented Feb 21, 2025

Open Mage.Verify folder and delete two zip-files with mtgjson data. It will help to redownload actual data on next run.

@padfoothelix
Copy link
Contributor Author

Thanks ! That did it.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants