github/mage
1
0
Fork 0
mirror of https://github.com/correl/mage.git synced 2024-12-27 20:06:31 +00:00
Code Issues Projects Releases Packages Wiki Activity

Tests: added verify check for wrong set names (disabled until 40k cards merge)

Browse source
This commit is contained in:
Oleg Agafonov 2023-04-07 00:31:02 +04:00
parent 202f3687e9
commit 59d2b59a4f
1 changed files with 20 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
Mage.Verify/src/test/java/mage/verify/VerifyCardDataTest.java
View file

@ -877,8 +877,8 @@ public class VerifyCardDataTest {
// CHECK: wrong basic lands settings (it's for lands search, not booster construct)
for (ExpansionSet set : sets) {
Boolean needLand = set.hasBasicLands();
Boolean foundLand = false;
boolean needLand = set.hasBasicLands();
boolean foundLand = false;
Map<String, Integer> foundLandsList = new HashMap<>();
for (ExpansionSet.SetCardInfo card : set.getSetCardInfo()) {
if (isBasicLandName(card.getName())) {
@ -924,6 +924,24 @@ public class VerifyCardDataTest {
}
}
// CHECK: wrong set name
for (ExpansionSet set : sets) {
if (true) continue; // TODO: enable after merge of 40k's cards pull requests (needs before set rename)
MtgJsonSet jsonSet = MtgJsonService.sets().getOrDefault(set.getCode().toUpperCase(Locale.ENGLISH), null);
if (jsonSet == null) {
// unofficial or inner set
continue;
}
if (!Objects.equals(set.getName(), jsonSet.name)) {
// how-to fix: rename xmage set to the json version or fix a set's code
// also don't forget to change names in mtg-cards-data.txt
errorsList.add(String.format("Error: wrong set name or set code: %s (mtgjson set for same code: %s)",
set.getCode() + " - " + set.getName(),
jsonSet.name
));
}
}
// TODO: add test to check num cards for rarity (rarityStats > 0 and numRarity > 0)
printMessages(warningsList);
printMessages(errorsList);