Skip to content
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

[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.

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

Successfully merging this pull request may close these issues.

2 participants