Skip to content

Commit 41935a7

Browse files
committed
Fix cross variant spec
This was broken when introducing the spam protection for events.
1 parent b07c103 commit 41935a7

File tree

1 file changed

+31
-14
lines changed

1 file changed

+31
-14
lines changed

spec/unittests/trophies_achievements/cross_variant_spec.rb

+31-14
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@
3636
end
3737

3838
describe Game,"saving of cross variant achievements" do
39-
39+
4040
before :each do
4141
clean_database
4242
$user = User.create(:login => "test_user")
@@ -60,24 +60,41 @@ def update_games
6060
expect(Event.count).to eq 0
6161

6262
Game.create(params)
63-
repository.adapter.execute "UPDATE games SET version= 'v'||id"
63+
repository.adapter.execute "UPDATE games SET version = 'v' || id"
6464
update_games
65+
6566
expect(Individualtrophy.count).to eq 4
66-
expect(Individualtrophy.all.map(&:trophy).sort).to eq ["anti_stoner_1", "ascended_variants_1", "globetrotter_1", "sightseeing_tour_1"]
67-
expect(Event.count).to eq 4
68-
expect(Event.all.map(&:text).sort).to eq(
69-
['Achievement "Diversity Ascender: Ascended one variant" unlocked by test_user!',
70-
'Achievement "Sightseeing Tour: finish a game in one variant" unlocked by test_user!',
71-
'Achievement "Anti-Stoner: defeated Medusa in one variant" unlocked by test_user!',
72-
'Achievement "Globetrotter: get a trophy in one variant" unlocked by test_user!'].sort
73-
)
67+
expect(Individualtrophy.all.map(&:trophy).sort).to match_array([
68+
'anti_stoner_1',
69+
'ascended_variants_1',
70+
'globetrotter_1',
71+
'sightseeing_tour_1'
72+
])
73+
74+
# spam protection, no events generated
75+
expect(Event.count).to eq 0
7476

7577
Game.create(params)
76-
repository.adapter.execute "UPDATE games SET version= 'v'||id"
78+
repository.adapter.execute "UPDATE games SET version = 'v' || id"
7779
update_games
78-
expect(Individualtrophy.all.map(&:trophy)).to include "anti_stoner_2"
80+
81+
expect(Individualtrophy.all.map(&:trophy).sort).to match_array([
82+
'anti_stoner_1',
83+
'anti_stoner_2',
84+
'ascended_variants_1',
85+
'ascended_variants_2',
86+
'globetrotter_1',
87+
'globetrotter_2',
88+
'sightseeing_tour_1',
89+
'sightseeing_tour_2'
90+
])
91+
7992
expect(Individualtrophy.count).to eq 8
80-
expect(Event.all.map(&:text)).to include 'Achievement "Anti-Stoner: defeated Medusa in two variants" unlocked by test_user!'
81-
expect(Event.count).to eq 8
93+
expect(Event.all.map(&:text)).to match_array([
94+
'Achievement "Anti-Stoner: defeated Medusa in two variants" unlocked by test_user!',
95+
'Achievement "Diversity Ascender: Ascended two variants" unlocked by test_user!'
96+
])
97+
# because of spam protection, only the 2 last achievements generated events
98+
expect(Event.count).to eq 2
8299
end
83100
end

0 commit comments

Comments
 (0)