Skip to content

Commit 404b3e4

Browse files
committed
Tidy up changes in deck_validator_test.rb
1 parent 36e5e91 commit 404b3e4

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

test/unit/deck_validator_test.rb

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -505,7 +505,7 @@ def test_runner_identities_are_not_valid_cards_for_basic_deckbuilding
505505
assert_equal v.validations.size, deck['validations'].size
506506
assert_not v.validations[0].valid?
507507
assert_includes v.validations[0].errors,
508-
'Decks may not include multiple identities. Identity card `armand_geist_walker_tech_lord` is not allowed.'
508+
'Decks may not include multiple identities. Identity card `armand_geist_walker_tech_lord` is not allowed.' # rubocop:disable Layout/LineLength
509509
end
510510

511511
def test_too_much_program_influence_professor
@@ -583,7 +583,7 @@ def test_out_of_faction_agendas_ampere
583583
assert_equal v.validations.size, @ampere_too_many_agendas_from_one_faction['validations'].size
584584
assert_not v.validations[0].valid?, 'Basic deckbuilding validation fails.'
585585
assert_includes v.validations[0].errors,
586-
'Ampere decks may not include more than 2 agendas per non-neutral faction. There are 3 `nbn` agendas present.'
586+
'Ampere decks may not include more than 2 agendas per non-neutral faction. There are 3 `nbn` agendas present.' # rubocop:disable Layout/LineLength
587587
end
588588

589589
def test_mismatched_side_corp_id
@@ -685,8 +685,8 @@ def test_invalid_format_id
685685
v = DeckValidator.new(deck)
686686
assert_not v.valid?
687687
assert_equal v.validations.size, deck['validations'].size
688-
# TODO: Update validation to explicitly set valid? to false and have the validator set it to true as a literal iff valid.
689-
# assert v.validations[0].valid?
688+
# TODO: Update validation to explicitly set valid? to false
689+
# and have the validator set it to true as a literal iff valid.
690690
assert_includes v.errors, 'Format `magic_the_gathering` does not exist.'
691691
end
692692

@@ -739,7 +739,7 @@ def test_cards_not_in_specified_card_pool
739739
spin_doctor
740740
tollbooth
741741
].each do |c|
742-
assert_includes v.validations[0].errors, 'Card `%s` is not present in Card Pool `standard_02`.' % c
742+
assert_includes v.validations[0].errors, "Card `#{c}` is not present in Card Pool `standard_02`."
743743
end
744744
end
745745

@@ -768,7 +768,7 @@ def test_too_many_restricted_cards
768768
assert_not v.valid?
769769
assert_equal v.validations.size, deck['validations'].size
770770
assert_includes v.validations[0].errors,
771-
'Deck has too many cards marked restricted in restriction `standard_restricted`: send_a_message, trieste_model_bioroids.'
771+
'Deck has too many cards marked restricted in restriction `standard_restricted`: send_a_message, trieste_model_bioroids.' # rubocop:disable Layout/LineLength
772772
end
773773

774774
def test_global_penalty_reduces_influence
@@ -782,7 +782,7 @@ def test_global_penalty_reduces_influence
782782
assert_not v.valid?
783783
assert_equal v.validations.size, deck['validations'].size
784784
assert_includes v.validations[0].errors,
785-
'Influence limit for Asa Group: Security Through Vigilance is 13 after Global Penalty applied from restriction `standard_global_penalty`, but deck has spent 2 influence from tyr (2).'
785+
'Influence limit for Asa Group: Security Through Vigilance is 13 after Global Penalty applied from restriction `standard_global_penalty`, but deck has spent 2 influence from tyr (2).' # rubocop:disable Layout/LineLength
786786
end
787787

788788
def test_universal_influence
@@ -796,7 +796,7 @@ def test_universal_influence
796796
assert_not v.valid?
797797
assert_equal v.validations.size, deck['validations'].size
798798
assert_includes v.validations[0].errors,
799-
'Influence limit for Asa Group: Security Through Vigilance is 15, but after Universal Influence applied from restriction `standard_universal_faction_cost`, deck has spent 24 influence from punitive_counterstrike (9).'
799+
'Influence limit for Asa Group: Security Through Vigilance is 15, but after Universal Influence applied from restriction `standard_universal_faction_cost`, deck has spent 24 influence from punitive_counterstrike (9).' # rubocop:disable Layout/LineLength
800800
end
801801

802802
def test_over_eternal_points_limit
@@ -810,7 +810,7 @@ def test_over_eternal_points_limit
810810
assert_not v.valid?
811811
assert_equal v.validations.size, deck['validations'].size
812812
assert_includes v.validations[0].errors,
813-
'Deck has too many points (9) for eternal restriction `eternal_points_list`: send_a_message (3), punitive_counterstrike (3), tyr (3).'
813+
'Deck has too many points (9) for eternal restriction `eternal_points_list`: send_a_message (3), punitive_counterstrike (3), tyr (3).' # rubocop:disable Layout/LineLength
814814
end
815815

816816
# Mumba Temple costs 0 instead of 2 influence if the deck has >= 15 ice

0 commit comments

Comments
 (0)