leaderboard 3.6.0 → 3.7.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- checksums.yaml +4 -4
- data/.rspec +1 -1
- data/.ruby-version +1 -3
- data/.travis.yml +1 -2
- data/CHANGELOG.markdown +4 -0
- data/README.markdown +33 -1
- data/Rakefile +0 -5
- data/leaderboard.gemspec +0 -3
- data/lib/competition_ranking_leaderboard.rb +98 -0
- data/lib/leaderboard.rb +1 -1
- data/lib/leaderboard/version.rb +1 -1
- data/lib/tie_ranking_leaderboard.rb +253 -0
- data/spec/competition_ranking_leaderboard_spec.rb +110 -0
- data/spec/leaderboard_spec.rb +250 -250
- data/spec/reverse_competition_ranking_leaderboard_spec.rb +116 -0
- data/spec/reverse_leaderboard_spec.rb +127 -127
- data/spec/reverse_tie_ranking_leaderboard_spec.rb +141 -0
- data/spec/spec_helper.rb +1 -1
- data/spec/tie_ranking_leaderboard_spec.rb +207 -0
- data/spec/version_spec.rb +1 -1
- metadata +26 -16
@@ -0,0 +1,110 @@
|
|
1
|
+
require 'spec_helper'
|
2
|
+
require 'competition_ranking_leaderboard'
|
3
|
+
|
4
|
+
describe 'CompetitionRankingLeaderboard' do
|
5
|
+
before(:each) do
|
6
|
+
@redis_connection = Redis.new(:host => "127.0.0.1", :db => 15)
|
7
|
+
end
|
8
|
+
|
9
|
+
after(:each) do
|
10
|
+
@redis_connection.flushdb
|
11
|
+
@redis_connection.client.disconnect
|
12
|
+
end
|
13
|
+
|
14
|
+
context 'ties' do
|
15
|
+
it 'should retrieve the correct rankings for #leaders' do
|
16
|
+
leaderboard = CompetitionRankingLeaderboard.new('ties', Leaderboard::DEFAULT_OPTIONS, {:host => "127.0.0.1", :db => 15})
|
17
|
+
leaderboard.rank_member('member_1', 50)
|
18
|
+
leaderboard.rank_member('member_2', 50)
|
19
|
+
leaderboard.rank_member('member_3', 30)
|
20
|
+
leaderboard.rank_member('member_4', 30)
|
21
|
+
leaderboard.rank_member('member_5', 10)
|
22
|
+
|
23
|
+
leaderboard.leaders(1).tap do |leaders|
|
24
|
+
expect(leaders[0][:rank]).to eq(1)
|
25
|
+
expect(leaders[1][:rank]).to eq(1)
|
26
|
+
expect(leaders[2][:rank]).to eq(3)
|
27
|
+
expect(leaders[3][:rank]).to eq(3)
|
28
|
+
expect(leaders[4][:rank]).to eq(5)
|
29
|
+
end
|
30
|
+
|
31
|
+
leaderboard.disconnect
|
32
|
+
end
|
33
|
+
|
34
|
+
it 'should retrieve the correct rankings for #leaders with different page sizes' do
|
35
|
+
leaderboard = CompetitionRankingLeaderboard.new('ties', Leaderboard::DEFAULT_OPTIONS, {:host => "127.0.0.1", :db => 15})
|
36
|
+
leaderboard.rank_member('member_1', 50)
|
37
|
+
leaderboard.rank_member('member_2', 50)
|
38
|
+
leaderboard.rank_member('member_6', 50)
|
39
|
+
leaderboard.rank_member('member_7', 50)
|
40
|
+
leaderboard.rank_member('member_3', 30)
|
41
|
+
leaderboard.rank_member('member_4', 30)
|
42
|
+
leaderboard.rank_member('member_8', 30)
|
43
|
+
leaderboard.rank_member('member_9', 30)
|
44
|
+
leaderboard.rank_member('member_5', 10)
|
45
|
+
leaderboard.rank_member('member_10', 10)
|
46
|
+
|
47
|
+
leaderboard.leaders(1, :page_size => 3).tap do |leaders|
|
48
|
+
expect(leaders[0][:rank]).to eq(1)
|
49
|
+
expect(leaders[1][:rank]).to eq(1)
|
50
|
+
expect(leaders[2][:rank]).to eq(1)
|
51
|
+
end
|
52
|
+
|
53
|
+
leaderboard.leaders(2, :page_size => 3).tap do |leaders|
|
54
|
+
expect(leaders[0][:rank]).to eq(1)
|
55
|
+
expect(leaders[1][:rank]).to eq(5)
|
56
|
+
expect(leaders[2][:rank]).to eq(5)
|
57
|
+
end
|
58
|
+
|
59
|
+
leaderboard.disconnect
|
60
|
+
end
|
61
|
+
|
62
|
+
it 'should retrieve the correct rankings for #around_me' do
|
63
|
+
leaderboard = CompetitionRankingLeaderboard.new('ties', Leaderboard::DEFAULT_OPTIONS, {:host => "127.0.0.1", :db => 15})
|
64
|
+
leaderboard.rank_member('member_1', 50)
|
65
|
+
leaderboard.rank_member('member_2', 50)
|
66
|
+
leaderboard.rank_member('member_6', 50)
|
67
|
+
leaderboard.rank_member('member_7', 50)
|
68
|
+
leaderboard.rank_member('member_3', 30)
|
69
|
+
leaderboard.rank_member('member_4', 30)
|
70
|
+
leaderboard.rank_member('member_8', 30)
|
71
|
+
leaderboard.rank_member('member_9', 30)
|
72
|
+
leaderboard.rank_member('member_5', 10)
|
73
|
+
leaderboard.rank_member('member_10', 10)
|
74
|
+
|
75
|
+
leaderboard.around_me('member_4').tap do |leaders|
|
76
|
+
expect(leaders[0][:rank]).to eq(1)
|
77
|
+
expect(leaders[4][:rank]).to eq(5)
|
78
|
+
expect(leaders[9][:rank]).to eq(9)
|
79
|
+
end
|
80
|
+
|
81
|
+
leaderboard.disconnect
|
82
|
+
end
|
83
|
+
|
84
|
+
it 'should allow you to retrieve the rank of a single member using #rank_for' do
|
85
|
+
leaderboard = CompetitionRankingLeaderboard.new('ties', Leaderboard::DEFAULT_OPTIONS, {:host => "127.0.0.1", :db => 15})
|
86
|
+
leaderboard.rank_member('member_1', 50)
|
87
|
+
leaderboard.rank_member('member_2', 50)
|
88
|
+
leaderboard.rank_member('member_3', 30)
|
89
|
+
|
90
|
+
expect(leaderboard.rank_for('member_1')).to eq(1)
|
91
|
+
expect(leaderboard.rank_for('member_2')).to eq(1)
|
92
|
+
expect(leaderboard.rank_for('member_3')).to eq(3)
|
93
|
+
|
94
|
+
leaderboard.disconnect
|
95
|
+
end
|
96
|
+
|
97
|
+
it 'should allow you to retrieve the score and rank of a single member using #score_and_rank_for' do
|
98
|
+
leaderboard = CompetitionRankingLeaderboard.new('ties', Leaderboard::DEFAULT_OPTIONS, {:host => "127.0.0.1", :db => 15})
|
99
|
+
leaderboard.rank_member('member_1', 50)
|
100
|
+
leaderboard.rank_member('member_2', 50)
|
101
|
+
leaderboard.rank_member('member_3', 30)
|
102
|
+
|
103
|
+
expect(leaderboard.score_and_rank_for('member_1')[:rank]).to eq(1)
|
104
|
+
expect(leaderboard.score_and_rank_for('member_2')[:rank]).to eq(1)
|
105
|
+
expect(leaderboard.score_and_rank_for('member_3')[:rank]).to eq(3)
|
106
|
+
|
107
|
+
leaderboard.disconnect
|
108
|
+
end
|
109
|
+
end
|
110
|
+
end
|
data/spec/leaderboard_spec.rb
CHANGED
@@ -13,149 +13,149 @@ describe 'Leaderboard' do
|
|
13
13
|
end
|
14
14
|
|
15
15
|
it 'should be initialized with defaults' do
|
16
|
-
@leaderboard.leaderboard_name.
|
17
|
-
@leaderboard.page_size.
|
16
|
+
expect(@leaderboard.leaderboard_name).to eql('name')
|
17
|
+
expect(@leaderboard.page_size).to eql(Leaderboard::DEFAULT_PAGE_SIZE)
|
18
18
|
end
|
19
19
|
|
20
20
|
it 'should be able to disconnect its connection to Redis' do
|
21
|
-
@leaderboard.disconnect.
|
21
|
+
expect(@leaderboard.disconnect).to be_nil
|
22
22
|
end
|
23
23
|
|
24
24
|
it 'should automatically reconnect to Redis after a disconnect' do
|
25
|
-
@leaderboard.total_members.
|
25
|
+
expect(@leaderboard.total_members).to be(0)
|
26
26
|
rank_members_in_leaderboard(5)
|
27
|
-
@leaderboard.total_members.
|
28
|
-
@leaderboard.disconnect.
|
29
|
-
@leaderboard.total_members.
|
27
|
+
expect(@leaderboard.total_members).to be(5)
|
28
|
+
expect(@leaderboard.disconnect).to be_nil
|
29
|
+
expect(@leaderboard.total_members).to be(5)
|
30
30
|
end
|
31
31
|
|
32
32
|
it 'should set the page size to the default page size if passed an invalid value' do
|
33
33
|
some_leaderboard = Leaderboard.new('name', {:page_size => 0}, {:host => "127.0.0.1", :db => 15})
|
34
34
|
|
35
|
-
some_leaderboard.page_size.
|
35
|
+
expect(some_leaderboard.page_size).to be(Leaderboard::DEFAULT_PAGE_SIZE)
|
36
36
|
some_leaderboard.disconnect
|
37
37
|
end
|
38
38
|
|
39
39
|
it 'should allow you to delete a leaderboard' do
|
40
40
|
rank_members_in_leaderboard
|
41
41
|
|
42
|
-
@redis_connection.exists('name').
|
43
|
-
@redis_connection.exists('name:member_data').
|
42
|
+
expect(@redis_connection.exists('name')).to be_truthy
|
43
|
+
expect(@redis_connection.exists('name:member_data')).to be_truthy
|
44
44
|
@leaderboard.delete_leaderboard
|
45
|
-
@redis_connection.exists('name').
|
46
|
-
@redis_connection.exists('name:member_data').
|
45
|
+
expect(@redis_connection.exists('name')).to be_falsey
|
46
|
+
expect(@redis_connection.exists('name:member_data')).to be_falsey
|
47
47
|
end
|
48
48
|
|
49
49
|
it 'should allow you to pass in an existing redis connection in the initializer' do
|
50
50
|
@leaderboard = Leaderboard.new('name', Leaderboard::DEFAULT_OPTIONS, {:redis_connection => @redis_connection})
|
51
51
|
rank_members_in_leaderboard
|
52
52
|
|
53
|
-
@redis_connection.info["connected_clients"].to_i.
|
53
|
+
expect(@redis_connection.info["connected_clients"].to_i).to be(1)
|
54
54
|
end
|
55
55
|
|
56
56
|
it 'should allow you to rank a member and see that reflected in total members' do
|
57
57
|
@leaderboard.rank_member('member', 1)
|
58
58
|
|
59
|
-
@leaderboard.total_members.
|
59
|
+
expect(@leaderboard.total_members).to be(1)
|
60
60
|
end
|
61
61
|
|
62
62
|
it 'should return the correct number of members in a given score range' do
|
63
63
|
rank_members_in_leaderboard(5)
|
64
64
|
|
65
|
-
@leaderboard.total_members_in_score_range(2, 4).
|
65
|
+
expect(@leaderboard.total_members_in_score_range(2, 4)).to be(3)
|
66
66
|
end
|
67
67
|
|
68
68
|
it 'should return the correct rank when calling rank_for' do
|
69
69
|
rank_members_in_leaderboard(5)
|
70
70
|
|
71
|
-
@leaderboard.rank_for('member_4').
|
71
|
+
expect(@leaderboard.rank_for('member_4')).to be(2)
|
72
72
|
end
|
73
73
|
|
74
74
|
it 'should return the correct score when calling score_for' do
|
75
75
|
rank_members_in_leaderboard(5)
|
76
76
|
|
77
|
-
@leaderboard.score_for('member_4').
|
78
|
-
@leaderboard.score_for('jones').
|
77
|
+
expect(@leaderboard.score_for('member_4')).to eql(4.0)
|
78
|
+
expect(@leaderboard.score_for('jones')).to be_nil
|
79
79
|
end
|
80
80
|
|
81
81
|
it 'should return the correct total pages' do
|
82
82
|
rank_members_in_leaderboard(10)
|
83
83
|
|
84
|
-
@leaderboard.total_pages.
|
85
|
-
@leaderboard.total_pages(5).
|
84
|
+
expect(@leaderboard.total_pages).to be(1)
|
85
|
+
expect(@leaderboard.total_pages(5)).to be(2)
|
86
86
|
|
87
87
|
@redis_connection.flushdb
|
88
88
|
|
89
89
|
rank_members_in_leaderboard(Leaderboard::DEFAULT_PAGE_SIZE + 1)
|
90
90
|
|
91
|
-
@leaderboard.total_pages.
|
91
|
+
expect(@leaderboard.total_pages).to be(2)
|
92
92
|
end
|
93
93
|
|
94
94
|
it 'should return the correct list when calling leaders' do
|
95
95
|
rank_members_in_leaderboard(25)
|
96
96
|
|
97
|
-
@leaderboard.total_members.
|
97
|
+
expect(@leaderboard.total_members).to be(25)
|
98
98
|
|
99
99
|
leaders = @leaderboard.leaders(1)
|
100
100
|
|
101
|
-
leaders.size.
|
102
|
-
leaders[0][:member].
|
103
|
-
leaders[-2][:member].
|
104
|
-
leaders[-1][:member].
|
105
|
-
leaders[-1][:score].to_i.
|
101
|
+
expect(leaders.size).to be(25)
|
102
|
+
expect(leaders[0][:member]).to eql 'member_25'
|
103
|
+
expect(leaders[-2][:member]).to eql 'member_2'
|
104
|
+
expect(leaders[-1][:member]).to eql 'member_1'
|
105
|
+
expect(leaders[-1][:score].to_i).to be(1)
|
106
106
|
end
|
107
107
|
|
108
108
|
it 'should return the correct list when calling members' do
|
109
109
|
rank_members_in_leaderboard(25)
|
110
110
|
|
111
|
-
@leaderboard.total_members.
|
111
|
+
expect(@leaderboard.total_members).to be(25)
|
112
112
|
|
113
113
|
members = @leaderboard.members(1)
|
114
114
|
|
115
|
-
members.size.
|
116
|
-
members[0][:member].
|
117
|
-
members[-2][:member].
|
118
|
-
members[-1][:member].
|
119
|
-
members[-1][:score].to_i.
|
115
|
+
expect(members.size).to be(25)
|
116
|
+
expect(members[0][:member]).to eql('member_25')
|
117
|
+
expect(members[-2][:member]).to eql('member_2')
|
118
|
+
expect(members[-1][:member]).to eql('member_1')
|
119
|
+
expect(members[-1][:score].to_i).to be(1)
|
120
120
|
end
|
121
121
|
|
122
122
|
it 'should return the correct number of members when calling leaders with multiple pages' do
|
123
123
|
rank_members_in_leaderboard(Leaderboard::DEFAULT_PAGE_SIZE * 3 + 1)
|
124
124
|
|
125
|
-
@leaderboard.total_members.
|
125
|
+
expect(@leaderboard.total_members).to be(Leaderboard::DEFAULT_PAGE_SIZE * 3 + 1)
|
126
126
|
|
127
127
|
leaders = @leaderboard.leaders(1)
|
128
|
-
leaders.size.
|
128
|
+
expect(leaders.size).to be(@leaderboard.page_size)
|
129
129
|
|
130
130
|
leaders = @leaderboard.leaders(2)
|
131
|
-
leaders.size.
|
131
|
+
expect(leaders.size).to be(@leaderboard.page_size)
|
132
132
|
|
133
133
|
leaders = @leaderboard.leaders(3)
|
134
|
-
leaders.size.
|
134
|
+
expect(leaders.size).to be(@leaderboard.page_size)
|
135
135
|
|
136
136
|
leaders = @leaderboard.leaders(4)
|
137
|
-
leaders.size.
|
137
|
+
expect(leaders.size).to be(1)
|
138
138
|
|
139
139
|
leaders = @leaderboard.leaders(-5)
|
140
|
-
leaders.size.
|
140
|
+
expect(leaders.size).to be(@leaderboard.page_size)
|
141
141
|
|
142
142
|
leaders = @leaderboard.leaders(10)
|
143
|
-
leaders.size.
|
143
|
+
expect(leaders.size).to be(1)
|
144
144
|
end
|
145
145
|
|
146
146
|
%w(members leaders).each do |method|
|
147
147
|
it "should return the entire leaderboard when you call 'all_#{method}'" do
|
148
148
|
rank_members_in_leaderboard(27)
|
149
149
|
|
150
|
-
@leaderboard.total_members.
|
150
|
+
expect(@leaderboard.total_members).to be(27)
|
151
151
|
|
152
152
|
members = @leaderboard.send("all_#{method}")
|
153
153
|
|
154
|
-
members.size.
|
155
|
-
members[0][:member].
|
156
|
-
members[-2][:member].
|
157
|
-
members[-1][:member].
|
158
|
-
members[-1][:score].to_i.
|
154
|
+
expect(members.size).to be(27)
|
155
|
+
expect(members[0][:member]).to eql('member_27')
|
156
|
+
expect(members[-2][:member]).to eql('member_2')
|
157
|
+
expect(members[-1][:member]).to eql('member_1')
|
158
|
+
expect(members[-1][:score].to_i).to be(1)
|
159
159
|
end
|
160
160
|
end
|
161
161
|
|
@@ -165,150 +165,150 @@ describe 'Leaderboard' do
|
|
165
165
|
members = @leaderboard.members_from_score_range(10, 15)
|
166
166
|
|
167
167
|
member_15 = {:member => 'member_15', :score => 15.0, :rank => 11}
|
168
|
-
members[0].
|
168
|
+
expect(members[0]).to eql(member_15)
|
169
169
|
|
170
170
|
member_10 = {:member => 'member_10', :score => 10.0, :rank => 16}
|
171
|
-
members[5].
|
171
|
+
expect(members[5]).to eql(member_10)
|
172
172
|
|
173
173
|
members = @leaderboard.members_from_score_range(10, 15, {:with_member_data => true})
|
174
174
|
|
175
175
|
member_15 = {:member => 'member_15', :rank => 11, :score => 15.0, :member_data => {:member_name => 'Leaderboard member 15'}.to_s}
|
176
|
-
members[0].
|
176
|
+
expect(members[0]).to eq(member_15)
|
177
177
|
|
178
178
|
member_10 = {:member => 'member_10', :rank => 16, :score => 10.0, :member_data => {:member_name => 'Leaderboard member 10'}.to_s}
|
179
|
-
members[5].
|
179
|
+
expect(members[5]).to eq(member_10)
|
180
180
|
end
|
181
181
|
|
182
182
|
it 'should allow you to retrieve leaders with extra data' do
|
183
183
|
rank_members_in_leaderboard(Leaderboard::DEFAULT_PAGE_SIZE)
|
184
184
|
|
185
|
-
@leaderboard.total_members.
|
185
|
+
expect(@leaderboard.total_members).to be(Leaderboard::DEFAULT_PAGE_SIZE)
|
186
186
|
leaders = @leaderboard.leaders(1, {:with_member_data => true})
|
187
187
|
|
188
188
|
member_25 = {:member => 'member_25', :score => 25.0, :rank => 1, :member_data => { :member_name => "Leaderboard member 25" }.to_s }
|
189
|
-
leaders[0].
|
189
|
+
expect(leaders[0]).to eq(member_25)
|
190
190
|
|
191
191
|
member_1 = {:member => 'member_1', :score => 1.0, :rank => 25, :member_data => { :member_name => "Leaderboard member 1" }.to_s }
|
192
|
-
leaders[24].
|
192
|
+
expect(leaders[24]).to eq(member_1)
|
193
193
|
end
|
194
194
|
|
195
195
|
it 'should allow you to retrieve optional member data' do
|
196
196
|
@leaderboard.rank_member('member_id', 1, {'username' => 'member_name', 'other_data_key' => 'other_data_value'})
|
197
197
|
|
198
|
-
@leaderboard.member_data_for('unknown_member').
|
199
|
-
@leaderboard.member_data_for('member_id').
|
198
|
+
expect(@leaderboard.member_data_for('unknown_member')).to be_nil
|
199
|
+
expect(@leaderboard.member_data_for('member_id')).to eq({'username' => 'member_name', 'other_data_key' => 'other_data_value'}.to_s)
|
200
200
|
end
|
201
201
|
|
202
202
|
it 'should allow you to update optional member data' do
|
203
203
|
@leaderboard.rank_member('member_id', 1, {'username' => 'member_name'})
|
204
204
|
|
205
|
-
@leaderboard.member_data_for('member_id').
|
205
|
+
expect(@leaderboard.member_data_for('member_id')).to eq({'username' => 'member_name'}.to_s)
|
206
206
|
@leaderboard.update_member_data('member_id', {'username' => 'member_name', 'other_data_key' => 'other_data_value'})
|
207
|
-
@leaderboard.member_data_for('member_id').
|
207
|
+
expect(@leaderboard.member_data_for('member_id')).to eq({'username' => 'member_name', 'other_data_key' => 'other_data_value'}.to_s)
|
208
208
|
end
|
209
209
|
|
210
210
|
it 'should allow you to remove optional member data' do
|
211
211
|
@leaderboard.rank_member('member_id', 1, {'username' => 'member_name'})
|
212
212
|
|
213
|
-
@leaderboard.member_data_for('member_id').
|
213
|
+
expect(@leaderboard.member_data_for('member_id')).to eq({'username' => 'member_name'}.to_s)
|
214
214
|
@leaderboard.remove_member_data('member_id')
|
215
|
-
@leaderboard.member_data_for('member_id').
|
215
|
+
expect(@leaderboard.member_data_for('member_id')).to be_nil
|
216
216
|
end
|
217
217
|
|
218
218
|
it 'should allow you to call leaders with various options that respect the defaults for the options not passed in' do
|
219
219
|
rank_members_in_leaderboard(Leaderboard::DEFAULT_PAGE_SIZE + 1)
|
220
220
|
|
221
221
|
leaders = @leaderboard.leaders(1, :page_size => 1)
|
222
|
-
leaders.size.
|
222
|
+
expect(leaders.size).to be(1)
|
223
223
|
member_26 = {:member => 'member_26', :score => 26.0, :rank => 1}
|
224
|
-
leaders[0].
|
224
|
+
expect(leaders[0]).to eql(member_26)
|
225
225
|
end
|
226
226
|
|
227
227
|
it 'should return a single member when calling member_at' do
|
228
228
|
rank_members_in_leaderboard(50)
|
229
|
-
@leaderboard.member_at(1)[:rank].
|
230
|
-
@leaderboard.member_at(1)[:score].
|
231
|
-
@leaderboard.member_at(26)[:rank].
|
232
|
-
@leaderboard.member_at(50)[:rank].
|
233
|
-
@leaderboard.member_at(51).
|
234
|
-
@leaderboard.member_at(1, :with_member_data => true)[:member_data].
|
229
|
+
expect(@leaderboard.member_at(1)[:rank]).to eql(1)
|
230
|
+
expect(@leaderboard.member_at(1)[:score]).to eql(50.0)
|
231
|
+
expect(@leaderboard.member_at(26)[:rank]).to eql(26)
|
232
|
+
expect(@leaderboard.member_at(50)[:rank]).to eql(50)
|
233
|
+
expect(@leaderboard.member_at(51)).to be_nil
|
234
|
+
expect(@leaderboard.member_at(1, :with_member_data => true)[:member_data]).to eq({:member_name => 'Leaderboard member 50'}.to_s)
|
235
235
|
end
|
236
236
|
|
237
237
|
it 'should return the correct information when calling around_me' do
|
238
238
|
rank_members_in_leaderboard(Leaderboard::DEFAULT_PAGE_SIZE * 3 + 1)
|
239
239
|
|
240
|
-
@leaderboard.total_members.
|
240
|
+
expect(@leaderboard.total_members).to be(Leaderboard::DEFAULT_PAGE_SIZE * 3 + 1)
|
241
241
|
|
242
242
|
leaders_around_me = @leaderboard.around_me('member_30')
|
243
|
-
(leaders_around_me.size / 2).
|
243
|
+
expect(leaders_around_me.size / 2).to be(@leaderboard.page_size / 2)
|
244
244
|
|
245
245
|
leaders_around_me = @leaderboard.around_me('member_1')
|
246
|
-
leaders_around_me.size.
|
246
|
+
expect(leaders_around_me.size).to be(@leaderboard.page_size / 2 + 1)
|
247
247
|
|
248
248
|
leaders_around_me = @leaderboard.around_me('member_76')
|
249
|
-
(leaders_around_me.size / 2).
|
249
|
+
expect(leaders_around_me.size / 2).to be(@leaderboard.page_size / 2)
|
250
250
|
end
|
251
251
|
|
252
252
|
it 'should return the correct information when calling ranked_in_list' do
|
253
253
|
rank_members_in_leaderboard(Leaderboard::DEFAULT_PAGE_SIZE)
|
254
254
|
|
255
|
-
@leaderboard.total_members.
|
255
|
+
expect(@leaderboard.total_members).to be(Leaderboard::DEFAULT_PAGE_SIZE)
|
256
256
|
|
257
257
|
members = ['member_1', 'member_5', 'member_10']
|
258
258
|
ranked_members = @leaderboard.ranked_in_list(members, Leaderboard::DEFAULT_LEADERBOARD_REQUEST_OPTIONS)
|
259
259
|
|
260
|
-
ranked_members.size.
|
260
|
+
expect(ranked_members.size).to be(3)
|
261
261
|
|
262
|
-
ranked_members[0][:rank].
|
263
|
-
ranked_members[0][:score].
|
262
|
+
expect(ranked_members[0][:rank]).to be(25)
|
263
|
+
expect(ranked_members[0][:score]).to eql(1.0)
|
264
264
|
|
265
|
-
ranked_members[1][:rank].
|
266
|
-
ranked_members[1][:score].
|
265
|
+
expect(ranked_members[1][:rank]).to be(21)
|
266
|
+
expect(ranked_members[1][:score]).to eql(5.0)
|
267
267
|
|
268
|
-
ranked_members[2][:rank].
|
269
|
-
ranked_members[2][:score].
|
268
|
+
expect(ranked_members[2][:rank]).to be(16)
|
269
|
+
expect(ranked_members[2][:score]).to eql(10.0)
|
270
270
|
end
|
271
271
|
|
272
272
|
it 'should return the correct information when calling ranked_in_list without scores' do
|
273
273
|
rank_members_in_leaderboard(Leaderboard::DEFAULT_PAGE_SIZE)
|
274
274
|
|
275
|
-
@leaderboard.total_members.
|
275
|
+
expect(@leaderboard.total_members).to be(Leaderboard::DEFAULT_PAGE_SIZE)
|
276
276
|
|
277
277
|
members = ['member_1', 'member_5', 'member_10']
|
278
278
|
ranked_members = @leaderboard.ranked_in_list(members, {:with_rank => true})
|
279
|
-
ranked_members.size.
|
280
|
-
ranked_members[0][:rank].
|
281
|
-
ranked_members[1][:rank].
|
282
|
-
ranked_members[2][:rank].
|
279
|
+
expect(ranked_members.size).to be(3)
|
280
|
+
expect(ranked_members[0][:rank]).to be(25)
|
281
|
+
expect(ranked_members[1][:rank]).to be(21)
|
282
|
+
expect(ranked_members[2][:rank]).to be(16)
|
283
283
|
end
|
284
284
|
|
285
285
|
it 'should allow you to remove members' do
|
286
286
|
rank_members_in_leaderboard(Leaderboard::DEFAULT_PAGE_SIZE)
|
287
287
|
|
288
|
-
@leaderboard.total_members.
|
288
|
+
expect(@leaderboard.total_members).to be(Leaderboard::DEFAULT_PAGE_SIZE)
|
289
289
|
|
290
290
|
@leaderboard.remove_member('member_1')
|
291
291
|
|
292
|
-
@leaderboard.total_members.
|
293
|
-
@leaderboard.rank_for('member_1').
|
292
|
+
expect(@leaderboard.total_members).to be(Leaderboard::DEFAULT_PAGE_SIZE - 1)
|
293
|
+
expect(@leaderboard.rank_for('member_1')).to be_nil
|
294
294
|
end
|
295
295
|
|
296
296
|
it 'should allow you to change the score for a member' do
|
297
297
|
@leaderboard.rank_member('member_1', 5)
|
298
|
-
@leaderboard.score_for('member_1').
|
298
|
+
expect(@leaderboard.score_for('member_1')).to eql(5.0)
|
299
299
|
|
300
300
|
@leaderboard.change_score_for('member_1', 5)
|
301
|
-
@leaderboard.score_for('member_1').
|
301
|
+
expect(@leaderboard.score_for('member_1')).to eql(10.0)
|
302
302
|
|
303
303
|
@leaderboard.change_score_for('member_1', -5)
|
304
|
-
@leaderboard.score_for('member_1').
|
304
|
+
expect(@leaderboard.score_for('member_1')).to eql(5.0)
|
305
305
|
end
|
306
306
|
|
307
307
|
it 'should allow you to check if a member exists' do
|
308
308
|
@leaderboard.rank_member('member_1', 10)
|
309
309
|
|
310
|
-
@leaderboard.check_member?('member_1').
|
311
|
-
@leaderboard.check_member?('member_2').
|
310
|
+
expect(@leaderboard.check_member?('member_1')).to be_truthy
|
311
|
+
expect(@leaderboard.check_member?('member_2')).to be_falsey
|
312
312
|
end
|
313
313
|
|
314
314
|
it 'should allow you to change the page size and have that reflected in the size of the result set' do
|
@@ -316,58 +316,58 @@ describe 'Leaderboard' do
|
|
316
316
|
|
317
317
|
@leaderboard.page_size = 5
|
318
318
|
|
319
|
-
@leaderboard.total_pages.
|
320
|
-
@leaderboard.leaders(1).size.
|
319
|
+
expect(@leaderboard.total_pages).to be(5)
|
320
|
+
expect(@leaderboard.leaders(1).size).to be(5)
|
321
321
|
end
|
322
322
|
|
323
323
|
it 'should not allow you to set the page size to an invalid page size' do
|
324
324
|
rank_members_in_leaderboard(Leaderboard::DEFAULT_PAGE_SIZE)
|
325
325
|
|
326
326
|
@leaderboard.page_size = 0
|
327
|
-
@leaderboard.total_pages.
|
328
|
-
@leaderboard.leaders(1).size.
|
327
|
+
expect(@leaderboard.total_pages).to be(1)
|
328
|
+
expect(@leaderboard.leaders(1).size).to be(Leaderboard::DEFAULT_PAGE_SIZE)
|
329
329
|
end
|
330
330
|
|
331
331
|
it 'should return the correct information when calling score_and_rank_for' do
|
332
332
|
rank_members_in_leaderboard
|
333
333
|
|
334
334
|
data = @leaderboard.score_and_rank_for('member_1')
|
335
|
-
data[:member].
|
336
|
-
data[:score].
|
337
|
-
data[:rank].
|
335
|
+
expect(data[:member]).to eql('member_1')
|
336
|
+
expect(data[:score]).to eql(1.0)
|
337
|
+
expect(data[:rank]).to eql(5)
|
338
338
|
end
|
339
339
|
|
340
340
|
it 'should allow you to remove members in a given score range' do
|
341
341
|
rank_members_in_leaderboard
|
342
342
|
|
343
|
-
@leaderboard.total_members.
|
343
|
+
expect(@leaderboard.total_members).to be(5)
|
344
344
|
|
345
345
|
@leaderboard.rank_member('cheater_1', 100)
|
346
346
|
@leaderboard.rank_member('cheater_2', 101)
|
347
347
|
@leaderboard.rank_member('cheater_3', 102)
|
348
348
|
|
349
|
-
@leaderboard.total_members.
|
349
|
+
expect(@leaderboard.total_members).to be(8)
|
350
350
|
|
351
351
|
@leaderboard.remove_members_in_score_range(100, 102)
|
352
352
|
|
353
|
-
@leaderboard.total_members.
|
353
|
+
expect(@leaderboard.total_members).to be(5)
|
354
354
|
|
355
355
|
leaders = @leaderboard.leaders(1)
|
356
356
|
leaders.each do |leader|
|
357
|
-
leader[:score].
|
357
|
+
expect(leader[:score]).to be < 100
|
358
358
|
end
|
359
359
|
end
|
360
360
|
|
361
361
|
it 'should allow you to remove members outside a given rank' do
|
362
362
|
rank_members_in_leaderboard
|
363
363
|
|
364
|
-
@leaderboard.total_members.
|
365
|
-
@leaderboard.remove_members_outside_rank(3).
|
364
|
+
expect(@leaderboard.total_members).to be(5)
|
365
|
+
expect(@leaderboard.remove_members_outside_rank(3)).to be(2)
|
366
366
|
|
367
367
|
leaders = @leaderboard.leaders(1)
|
368
|
-
leaders.size.
|
369
|
-
leaders[0][:member].
|
370
|
-
leaders[2][:member].
|
368
|
+
expect(leaders.size).to be(3)
|
369
|
+
expect(leaders[0][:member]).to eq('member_5')
|
370
|
+
expect(leaders[2][:member]).to eq('member_3')
|
371
371
|
end
|
372
372
|
|
373
373
|
it 'should allow you to merge leaderboards' do
|
@@ -381,15 +381,15 @@ describe 'Leaderboard' do
|
|
381
381
|
bar.rank_member('bar_3', 5)
|
382
382
|
|
383
383
|
foobar_keys = foo.merge_leaderboards('foobar', ['bar'])
|
384
|
-
foobar_keys.
|
384
|
+
expect(foobar_keys).to be(5)
|
385
385
|
|
386
386
|
foobar = Leaderboard.new('foobar', Leaderboard::DEFAULT_OPTIONS, {:host => "127.0.0.1", :db => 15})
|
387
|
-
foobar.total_members.
|
387
|
+
expect(foobar.total_members).to be(5)
|
388
388
|
|
389
389
|
first_leader_in_foobar = foobar.leaders(1).first
|
390
|
-
first_leader_in_foobar[:rank].
|
391
|
-
first_leader_in_foobar[:member].
|
392
|
-
first_leader_in_foobar[:score].
|
390
|
+
expect(first_leader_in_foobar[:rank]).to eql(1)
|
391
|
+
expect(first_leader_in_foobar[:member]).to eql('bar_3')
|
392
|
+
expect(first_leader_in_foobar[:score]).to eql(5.0)
|
393
393
|
|
394
394
|
foo.disconnect
|
395
395
|
bar.disconnect
|
@@ -408,15 +408,15 @@ describe 'Leaderboard' do
|
|
408
408
|
bar.rank_member('bar_3', 5)
|
409
409
|
|
410
410
|
foobar_keys = foo.intersect_leaderboards('foobar', ['bar'], {:aggregate => :max})
|
411
|
-
foobar_keys.
|
411
|
+
expect(foobar_keys).to be(2)
|
412
412
|
|
413
413
|
foobar = Leaderboard.new('foobar', Leaderboard::DEFAULT_OPTIONS, {:host => "127.0.0.1", :db => 15})
|
414
|
-
foobar.total_members.
|
414
|
+
expect(foobar.total_members).to be(2)
|
415
415
|
|
416
416
|
first_leader_in_foobar = foobar.leaders(1).first
|
417
|
-
first_leader_in_foobar[:rank].
|
418
|
-
first_leader_in_foobar[:member].
|
419
|
-
first_leader_in_foobar[:score].
|
417
|
+
expect(first_leader_in_foobar[:rank]).to eql(1)
|
418
|
+
expect(first_leader_in_foobar[:member]).to eql('bar_3')
|
419
|
+
expect(first_leader_in_foobar[:score]).to eql(6.0)
|
420
420
|
|
421
421
|
foo.disconnect
|
422
422
|
bar.disconnect
|
@@ -426,82 +426,82 @@ describe 'Leaderboard' do
|
|
426
426
|
it 'should respect options in the massage_leader_data method' do
|
427
427
|
rank_members_in_leaderboard(25)
|
428
428
|
|
429
|
-
@leaderboard.total_members.
|
429
|
+
expect(@leaderboard.total_members).to be(25)
|
430
430
|
|
431
431
|
leaders = @leaderboard.leaders(1)
|
432
|
-
leaders[0][:member].
|
433
|
-
leaders[0][:score].
|
434
|
-
leaders[0][:rank].
|
432
|
+
expect(leaders[0][:member]).not_to be_nil
|
433
|
+
expect(leaders[0][:score]).not_to be_nil
|
434
|
+
expect(leaders[0][:rank]).not_to be_nil
|
435
435
|
|
436
436
|
@leaderboard.page_size = Leaderboard::DEFAULT_PAGE_SIZE
|
437
437
|
leaders = @leaderboard.leaders(1, Leaderboard::DEFAULT_LEADERBOARD_REQUEST_OPTIONS)
|
438
|
-
leaders[0][:member].
|
439
|
-
leaders[0][:score].
|
440
|
-
leaders[0][:rank].
|
438
|
+
expect(leaders[0][:member]).not_to be_nil
|
439
|
+
expect(leaders[0][:score]).not_to be_nil
|
440
|
+
expect(leaders[0][:rank]).not_to be_nil
|
441
441
|
|
442
442
|
@leaderboard.page_size = 25
|
443
443
|
leaders = @leaderboard.leaders(1, Leaderboard::DEFAULT_LEADERBOARD_REQUEST_OPTIONS)
|
444
|
-
leaders.size.
|
444
|
+
expect(leaders.size).to be(25)
|
445
445
|
end
|
446
446
|
|
447
447
|
it 'should return the correct number of pages when calling total_pages_in page size option' do
|
448
448
|
rank_members_in_leaderboard(25)
|
449
449
|
|
450
|
-
@leaderboard.total_pages_in(@leaderboard.leaderboard_name).
|
451
|
-
@leaderboard.total_pages_in(@leaderboard.leaderboard_name, 5).
|
450
|
+
expect(@leaderboard.total_pages_in(@leaderboard.leaderboard_name)).to be(1)
|
451
|
+
expect(@leaderboard.total_pages_in(@leaderboard.leaderboard_name, 5)).to be(5)
|
452
452
|
end
|
453
453
|
|
454
454
|
it 'should return the correct number of members when calling leaders with a page_size option' do
|
455
455
|
rank_members_in_leaderboard(25)
|
456
456
|
|
457
|
-
@leaderboard.leaders(1, Leaderboard::DEFAULT_LEADERBOARD_REQUEST_OPTIONS.merge({:page_size => 5})).size.
|
458
|
-
@leaderboard.leaders(1, Leaderboard::DEFAULT_LEADERBOARD_REQUEST_OPTIONS.merge({:page_size => 10})).size.
|
459
|
-
@leaderboard.leaders(2, Leaderboard::DEFAULT_LEADERBOARD_REQUEST_OPTIONS.merge({:page_size => 10})).size.
|
460
|
-
@leaderboard.leaders(3, Leaderboard::DEFAULT_LEADERBOARD_REQUEST_OPTIONS.merge({:page_size => 10})).size.
|
457
|
+
expect(@leaderboard.leaders(1, Leaderboard::DEFAULT_LEADERBOARD_REQUEST_OPTIONS.merge({:page_size => 5})).size).to be(5)
|
458
|
+
expect(@leaderboard.leaders(1, Leaderboard::DEFAULT_LEADERBOARD_REQUEST_OPTIONS.merge({:page_size => 10})).size).to be(10)
|
459
|
+
expect(@leaderboard.leaders(2, Leaderboard::DEFAULT_LEADERBOARD_REQUEST_OPTIONS.merge({:page_size => 10})).size).to be(10)
|
460
|
+
expect(@leaderboard.leaders(3, Leaderboard::DEFAULT_LEADERBOARD_REQUEST_OPTIONS.merge({:page_size => 10})).size).to be(5)
|
461
461
|
end
|
462
462
|
|
463
463
|
it 'should return the correct number of members when calling around_me with a page_size options' do
|
464
464
|
rank_members_in_leaderboard(Leaderboard::DEFAULT_PAGE_SIZE * 3 + 1)
|
465
465
|
|
466
466
|
leaders_around_me = @leaderboard.around_me('member_30', Leaderboard::DEFAULT_LEADERBOARD_REQUEST_OPTIONS.merge({:page_size => 3}))
|
467
|
-
leaders_around_me.size.
|
468
|
-
leaders_around_me[0][:member].
|
469
|
-
leaders_around_me[2][:member].
|
467
|
+
expect(leaders_around_me.size).to be(3)
|
468
|
+
expect(leaders_around_me[0][:member]).to eql('member_31')
|
469
|
+
expect(leaders_around_me[2][:member]).to eql('member_29')
|
470
470
|
end
|
471
471
|
|
472
472
|
it 'should return the correct information when calling percentile_for' do
|
473
473
|
rank_members_in_leaderboard(12)
|
474
474
|
|
475
|
-
@leaderboard.percentile_for('member_1').
|
476
|
-
@leaderboard.percentile_for('member_2').
|
477
|
-
@leaderboard.percentile_for('member_3').
|
478
|
-
@leaderboard.percentile_for('member_4').
|
479
|
-
@leaderboard.percentile_for('member_12').
|
475
|
+
expect(@leaderboard.percentile_for('member_1')).to eql(0)
|
476
|
+
expect(@leaderboard.percentile_for('member_2')).to eql(9)
|
477
|
+
expect(@leaderboard.percentile_for('member_3')).to eql(17)
|
478
|
+
expect(@leaderboard.percentile_for('member_4')).to eql(25)
|
479
|
+
expect(@leaderboard.percentile_for('member_12')).to eql(92)
|
480
480
|
end
|
481
481
|
|
482
482
|
it 'should return the correct information when calling score_for_percentile' do
|
483
483
|
rank_members_in_leaderboard(5)
|
484
484
|
|
485
|
-
@leaderboard.score_for_percentile(0).
|
486
|
-
@leaderboard.score_for_percentile(75).
|
487
|
-
@leaderboard.score_for_percentile(87.5).
|
488
|
-
@leaderboard.score_for_percentile(93.75).
|
489
|
-
@leaderboard.score_for_percentile(100).
|
485
|
+
expect(@leaderboard.score_for_percentile(0)).to eql(1.0)
|
486
|
+
expect(@leaderboard.score_for_percentile(75)).to eql(4.0)
|
487
|
+
expect(@leaderboard.score_for_percentile(87.5)).to eql(4.5)
|
488
|
+
expect(@leaderboard.score_for_percentile(93.75)).to eql(4.75)
|
489
|
+
expect(@leaderboard.score_for_percentile(100)).to eql(5.0)
|
490
490
|
end
|
491
491
|
|
492
492
|
it 'should not throw an exception when calling around_me with a member not in the leaderboard' do
|
493
493
|
rank_members_in_leaderboard(Leaderboard::DEFAULT_PAGE_SIZE * 3 + 1)
|
494
494
|
|
495
495
|
leaders_around_me = @leaderboard.around_me('jones', Leaderboard::DEFAULT_LEADERBOARD_REQUEST_OPTIONS.merge({:page_size => 3}))
|
496
|
-
leaders_around_me.size.
|
496
|
+
expect(leaders_around_me.size).to be(0)
|
497
497
|
end
|
498
498
|
|
499
499
|
it 'should not throw an exception when calling score_and_rank_for with a member not in the leaderboard' do
|
500
500
|
score_and_rank_for_member = @leaderboard.score_and_rank_for('jones')
|
501
501
|
|
502
|
-
score_and_rank_for_member[:member].
|
503
|
-
score_and_rank_for_member[:score].
|
504
|
-
score_and_rank_for_member[:rank].
|
502
|
+
expect(score_and_rank_for_member[:member]).to eql('jones')
|
503
|
+
expect(score_and_rank_for_member[:score]).to be_nil
|
504
|
+
expect(score_and_rank_for_member[:rank]).to be_nil
|
505
505
|
end
|
506
506
|
|
507
507
|
it 'should not throw an exception when calling ranked_in_list with a member not in the leaderboard' do
|
@@ -510,36 +510,36 @@ describe 'Leaderboard' do
|
|
510
510
|
members = ['member_1', 'member_5', 'jones']
|
511
511
|
ranked_members = @leaderboard.ranked_in_list(members)
|
512
512
|
|
513
|
-
ranked_members.size.
|
514
|
-
ranked_members[2][:rank].
|
513
|
+
expect(ranked_members.size).to be(3)
|
514
|
+
expect(ranked_members[2][:rank]).to be_nil
|
515
515
|
end
|
516
516
|
|
517
517
|
it 'should not throw an exception when calling percentile_for with a member not in the leaderboard' do
|
518
518
|
percentile = @leaderboard.percentile_for('jones')
|
519
519
|
|
520
|
-
percentile.
|
520
|
+
expect(percentile).to be_nil
|
521
521
|
end
|
522
522
|
|
523
523
|
it 'should allow you to change the score for a member not in the leaderboard' do
|
524
|
-
@leaderboard.score_for('jones').
|
524
|
+
expect(@leaderboard.score_for('jones')).to be_nil
|
525
525
|
@leaderboard.change_score_for('jones', 5)
|
526
|
-
@leaderboard.score_for('jones').
|
526
|
+
expect(@leaderboard.score_for('jones')).to eql(5.0)
|
527
527
|
end
|
528
528
|
|
529
529
|
it 'should return the correct page when calling page_for a given member in the leaderboard' do
|
530
|
-
@leaderboard.page_for('jones').
|
530
|
+
expect(@leaderboard.page_for('jones')).to be(0)
|
531
531
|
|
532
532
|
rank_members_in_leaderboard(20)
|
533
533
|
|
534
|
-
@leaderboard.page_for('member_17').
|
535
|
-
@leaderboard.page_for('member_11').
|
536
|
-
@leaderboard.page_for('member_10').
|
537
|
-
@leaderboard.page_for('member_1').
|
534
|
+
expect(@leaderboard.page_for('member_17')).to be(1)
|
535
|
+
expect(@leaderboard.page_for('member_11')).to be(1)
|
536
|
+
expect(@leaderboard.page_for('member_10')).to be(1)
|
537
|
+
expect(@leaderboard.page_for('member_1')).to be(1)
|
538
538
|
|
539
|
-
@leaderboard.page_for('member_17', 10).
|
540
|
-
@leaderboard.page_for('member_11', 10).
|
541
|
-
@leaderboard.page_for('member_10', 10).
|
542
|
-
@leaderboard.page_for('member_1', 10).
|
539
|
+
expect(@leaderboard.page_for('member_17', 10)).to be(1)
|
540
|
+
expect(@leaderboard.page_for('member_11', 10)).to be(1)
|
541
|
+
expect(@leaderboard.page_for('member_10', 10)).to be(2)
|
542
|
+
expect(@leaderboard.page_for('member_1', 10)).to be(2)
|
543
543
|
end
|
544
544
|
|
545
545
|
it 'should set an expire on the leaderboard' do
|
@@ -547,12 +547,12 @@ describe 'Leaderboard' do
|
|
547
547
|
|
548
548
|
@leaderboard.expire_leaderboard(3)
|
549
549
|
@redis_connection.ttl(@leaderboard.leaderboard_name).tap do |ttl|
|
550
|
-
ttl.
|
551
|
-
ttl.
|
550
|
+
expect(ttl).to be > 1
|
551
|
+
expect(ttl).to be <= 3
|
552
552
|
end
|
553
553
|
@redis_connection.ttl(@leaderboard.send(:member_data_key, @leaderboard.leaderboard_name)).tap do |ttl|
|
554
|
-
ttl.
|
555
|
-
ttl.
|
554
|
+
expect(ttl).to be > 1
|
555
|
+
expect(ttl).to be <= 3
|
556
556
|
end
|
557
557
|
end
|
558
558
|
|
@@ -561,27 +561,27 @@ describe 'Leaderboard' do
|
|
561
561
|
|
562
562
|
@leaderboard.expire_leaderboard_at((Time.now + 10).to_i)
|
563
563
|
@redis_connection.ttl(@leaderboard.leaderboard_name).tap do |ttl|
|
564
|
-
ttl.
|
565
|
-
ttl.
|
564
|
+
expect(ttl).to be > 1
|
565
|
+
expect(ttl).to be <= 10
|
566
566
|
end
|
567
567
|
@redis_connection.ttl(@leaderboard.send(:member_data_key, @leaderboard.leaderboard_name)).tap do |ttl|
|
568
|
-
ttl.
|
569
|
-
ttl.
|
568
|
+
expect(ttl).to be > 1
|
569
|
+
expect(ttl).to be <= 10
|
570
570
|
end
|
571
571
|
end
|
572
572
|
|
573
573
|
it 'should allow you to rank multiple members with a variable number of arguments' do
|
574
|
-
@leaderboard.total_members.
|
574
|
+
expect(@leaderboard.total_members).to be(0)
|
575
575
|
@leaderboard.rank_members('member_1', 1, 'member_10', 10)
|
576
|
-
@leaderboard.total_members.
|
577
|
-
@leaderboard.leaders(1).first[:member].
|
576
|
+
expect(@leaderboard.total_members).to be(2)
|
577
|
+
expect(@leaderboard.leaders(1).first[:member]).to eql('member_10')
|
578
578
|
end
|
579
579
|
|
580
580
|
it 'should allow you to rank multiple members with an array' do
|
581
|
-
@leaderboard.total_members.
|
581
|
+
expect(@leaderboard.total_members).to be(0)
|
582
582
|
@leaderboard.rank_members(['member_1', 1, 'member_10', 10])
|
583
|
-
@leaderboard.total_members.
|
584
|
-
@leaderboard.leaders(1).first[:member].
|
583
|
+
expect(@leaderboard.total_members).to be(2)
|
584
|
+
expect(@leaderboard.leaders(1).first[:member]).to eql('member_10')
|
585
585
|
end
|
586
586
|
|
587
587
|
it 'should allow you to set reverse after creating a leaderboard to see results in highest-to-lowest or lowest-to-highest order' do
|
@@ -589,41 +589,41 @@ describe 'Leaderboard' do
|
|
589
589
|
|
590
590
|
leaders = @leaderboard.leaders(1)
|
591
591
|
|
592
|
-
leaders.size.
|
593
|
-
leaders[0][:member].
|
594
|
-
leaders[-2][:member].
|
595
|
-
leaders[-1][:member].
|
596
|
-
leaders[-1][:score].to_i.
|
592
|
+
expect(leaders.size).to be(25)
|
593
|
+
expect(leaders[0][:member]).to eql('member_25')
|
594
|
+
expect(leaders[-2][:member]).to eql('member_2')
|
595
|
+
expect(leaders[-1][:member]).to eql('member_1')
|
596
|
+
expect(leaders[-1][:score].to_i).to be(1)
|
597
597
|
|
598
598
|
@leaderboard.reverse = true
|
599
599
|
|
600
600
|
leaders = @leaderboard.leaders(1)
|
601
601
|
|
602
|
-
leaders.size.
|
603
|
-
leaders[0][:member].
|
604
|
-
leaders[-2][:member].
|
605
|
-
leaders[-1][:member].
|
606
|
-
leaders[-1][:score].to_i.
|
602
|
+
expect(leaders.size).to be(25)
|
603
|
+
expect(leaders[0][:member]).to eql('member_1')
|
604
|
+
expect(leaders[-2][:member]).to eql('member_24')
|
605
|
+
expect(leaders[-1][:member]).to eql('member_25')
|
606
|
+
expect(leaders[-1][:score].to_i).to be(25)
|
607
607
|
end
|
608
608
|
|
609
609
|
it 'should allow you to retrieve a given set of members from the leaderboard in a rank range' do
|
610
610
|
rank_members_in_leaderboard(25)
|
611
611
|
|
612
612
|
members = @leaderboard.members_from_rank_range(5, 9)
|
613
|
-
members.size.
|
614
|
-
members[0][:member].
|
615
|
-
members[0][:score].to_i.
|
616
|
-
members[4][:member].
|
613
|
+
expect(members.size).to be(5)
|
614
|
+
expect(members[0][:member]).to eql('member_21')
|
615
|
+
expect(members[0][:score].to_i).to be(21)
|
616
|
+
expect(members[4][:member]).to eql('member_17')
|
617
617
|
|
618
618
|
members = @leaderboard.members_from_rank_range(1, 1)
|
619
|
-
members.size.
|
620
|
-
members[0][:member].
|
619
|
+
expect(members.size).to be(1)
|
620
|
+
expect(members[0][:member]).to eql('member_25')
|
621
621
|
|
622
622
|
members = @leaderboard.members_from_rank_range(-1, 26)
|
623
|
-
members.size.
|
624
|
-
members[0][:member].
|
625
|
-
members[0][:score].to_i.
|
626
|
-
members[24][:member].
|
623
|
+
expect(members.size).to be(25)
|
624
|
+
expect(members[0][:member]).to eql('member_25')
|
625
|
+
expect(members[0][:score].to_i).to be(25)
|
626
|
+
expect(members[24][:member]).to eql('member_1')
|
627
627
|
end
|
628
628
|
|
629
629
|
it 'should sort by rank if the :sort_by option is set to :rank' do
|
@@ -632,16 +632,16 @@ describe 'Leaderboard' do
|
|
632
632
|
members = ['member_5', 'member_1', 'member_10']
|
633
633
|
ranked_members = @leaderboard.ranked_in_list(members, :sort_by => :rank)
|
634
634
|
|
635
|
-
ranked_members.size.
|
635
|
+
expect(ranked_members.size).to be(3)
|
636
636
|
|
637
|
-
ranked_members[0][:rank].
|
638
|
-
ranked_members[0][:score].
|
637
|
+
expect(ranked_members[0][:rank]).to be(16)
|
638
|
+
expect(ranked_members[0][:score]).to eql(10.0)
|
639
639
|
|
640
|
-
ranked_members[1][:rank].
|
641
|
-
ranked_members[1][:score].
|
640
|
+
expect(ranked_members[1][:rank]).to be(21)
|
641
|
+
expect(ranked_members[1][:score]).to eql(5.0)
|
642
642
|
|
643
|
-
ranked_members[2][:rank].
|
644
|
-
ranked_members[2][:score].
|
643
|
+
expect(ranked_members[2][:rank]).to be(25)
|
644
|
+
expect(ranked_members[2][:score]).to eql(1.0)
|
645
645
|
end
|
646
646
|
|
647
647
|
it 'should sort by score if the :sort_by option is set to :score' do
|
@@ -650,16 +650,16 @@ describe 'Leaderboard' do
|
|
650
650
|
members = ['member_5', 'member_1', 'member_10']
|
651
651
|
ranked_members = @leaderboard.ranked_in_list(members, :sort_by => :score)
|
652
652
|
|
653
|
-
ranked_members.size.
|
653
|
+
expect(ranked_members.size).to be(3)
|
654
654
|
|
655
|
-
ranked_members[0][:rank].
|
656
|
-
ranked_members[0][:score].
|
655
|
+
expect(ranked_members[0][:rank]).to be(25)
|
656
|
+
expect(ranked_members[0][:score]).to eql(1.0)
|
657
657
|
|
658
|
-
ranked_members[1][:rank].
|
659
|
-
ranked_members[1][:score].
|
658
|
+
expect(ranked_members[1][:rank]).to be(21)
|
659
|
+
expect(ranked_members[1][:score]).to eql(5.0)
|
660
660
|
|
661
|
-
ranked_members[2][:rank].
|
662
|
-
ranked_members[2][:score].
|
661
|
+
expect(ranked_members[2][:rank]).to be(16)
|
662
|
+
expect(ranked_members[2][:score]).to eql(10.0)
|
663
663
|
end
|
664
664
|
|
665
665
|
it 'should return nil for the score and rank for ranked_in_list if a member is not in the leaderboard' do
|
@@ -667,10 +667,10 @@ describe 'Leaderboard' do
|
|
667
667
|
|
668
668
|
ranked_members = @leaderboard.ranked_in_list(['jones'])
|
669
669
|
|
670
|
-
ranked_members.size.
|
671
|
-
ranked_members[0][:member].
|
672
|
-
ranked_members[0][:score].
|
673
|
-
ranked_members[0][:rank].
|
670
|
+
expect(ranked_members.size).to be(1)
|
671
|
+
expect(ranked_members[0][:member]).to eql('jones')
|
672
|
+
expect(ranked_members[0][:score]).to be_nil
|
673
|
+
expect(ranked_members[0][:rank]).to be_nil
|
674
674
|
end
|
675
675
|
|
676
676
|
it 'should rank a member in the leaderboard with conditional execution' do
|
@@ -680,60 +680,60 @@ describe 'Leaderboard' do
|
|
680
680
|
false
|
681
681
|
end
|
682
682
|
|
683
|
-
@leaderboard.total_members.
|
683
|
+
expect(@leaderboard.total_members).to be(0)
|
684
684
|
@leaderboard.rank_member_if(highscore_check, 'david', 1337)
|
685
|
-
@leaderboard.total_members.
|
686
|
-
@leaderboard.score_for('david').
|
685
|
+
expect(@leaderboard.total_members).to be(1)
|
686
|
+
expect(@leaderboard.score_for('david')).to eql(1337.0)
|
687
687
|
@leaderboard.rank_member_if(highscore_check, 'david', 1336)
|
688
|
-
@leaderboard.score_for('david').
|
688
|
+
expect(@leaderboard.score_for('david')).to eql(1337.0)
|
689
689
|
@leaderboard.rank_member_if(highscore_check, 'david', 1338)
|
690
|
-
@leaderboard.score_for('david').
|
690
|
+
expect(@leaderboard.score_for('david')).to eql(1338.0)
|
691
691
|
end
|
692
692
|
|
693
693
|
it 'should not delete all the member data when calling remove_member' do
|
694
694
|
rank_members_in_leaderboard
|
695
695
|
|
696
|
-
@redis_connection.exists("name:member_data").
|
697
|
-
@redis_connection.hgetall("name:member_data").size.
|
698
|
-
@leaderboard.total_members.
|
696
|
+
expect(@redis_connection.exists("name:member_data")).to be_truthy
|
697
|
+
expect(@redis_connection.hgetall("name:member_data").size).to be(5)
|
698
|
+
expect(@leaderboard.total_members).to be(5)
|
699
699
|
@leaderboard.remove_member('member_1')
|
700
|
-
@redis_connection.exists("name:member_data").
|
701
|
-
@redis_connection.hgetall("name:member_data").size.
|
702
|
-
@leaderboard.total_members.
|
700
|
+
expect(@redis_connection.exists("name:member_data")).to be_truthy
|
701
|
+
expect(@redis_connection.hgetall("name:member_data").size).to be(4)
|
702
|
+
expect(@leaderboard.total_members).to be(4)
|
703
703
|
end
|
704
704
|
|
705
705
|
it 'should return the members only if the :members_only option is passed' do
|
706
706
|
rank_members_in_leaderboard(25)
|
707
707
|
|
708
708
|
leaders = @leaderboard.leaders(1, :page_size => 10, :members_only => true)
|
709
|
-
leaders.size.
|
710
|
-
leaders.collect { |leader| leader.keys.
|
709
|
+
expect(leaders.size).to eq(10)
|
710
|
+
leaders.collect { |leader| expect(leader.keys).to eq([:member]) }
|
711
711
|
|
712
712
|
leaders = @leaderboard.all_leaders(:members_only => true)
|
713
|
-
leaders.size.
|
714
|
-
leaders.collect { |leader| leader.keys.
|
713
|
+
expect(leaders.size).to eq(25)
|
714
|
+
leaders.collect { |leader| expect(leader.keys).to eq([:member]) }
|
715
715
|
|
716
716
|
leaders = @leaderboard.members_from_score_range(10, 14, :members_only => true)
|
717
|
-
leaders.size.
|
718
|
-
leaders.collect { |leader| leader.keys.
|
717
|
+
expect(leaders.size).to eq(5)
|
718
|
+
leaders.collect { |leader| expect(leader.keys).to eq([:member]) }
|
719
719
|
|
720
720
|
leaders = @leaderboard.members_from_rank_range(1, 5, :page_size => 10, :members_only => true)
|
721
|
-
leaders.size.
|
722
|
-
leaders.collect { |leader| leader.keys.
|
721
|
+
expect(leaders.size).to eq(5)
|
722
|
+
leaders.collect { |leader| expect(leader.keys).to eq([:member]) }
|
723
723
|
|
724
724
|
leaders = @leaderboard.around_me('member_10', :page_size => 3, :members_only => true)
|
725
|
-
leaders.size.
|
726
|
-
leaders.collect { |leader| leader.keys.
|
725
|
+
expect(leaders.size).to eq(3)
|
726
|
+
leaders.collect { |leader| expect(leader.keys).to eq([:member]) }
|
727
727
|
|
728
728
|
leaders = @leaderboard.ranked_in_list(['member_1', 'member_25'], :members_only => true)
|
729
|
-
leaders.size.
|
730
|
-
leaders.collect { |leader| leader.keys.
|
729
|
+
expect(leaders.size).to eq(2)
|
730
|
+
leaders.collect { |leader| expect(leader.keys).to eq([:member]) }
|
731
731
|
end
|
732
732
|
|
733
733
|
it 'should allow you to rank a member across multiple leaderboards' do
|
734
734
|
@leaderboard.rank_member_across(['highscores', 'more_highscores'], 'david', 50000, { :member_name => "david" })
|
735
|
-
@leaderboard.leaders_in('highscores', 1).size.
|
736
|
-
@leaderboard.leaders_in('more_highscores', 1).size.
|
735
|
+
expect(@leaderboard.leaders_in('highscores', 1).size).to eql(1)
|
736
|
+
expect(@leaderboard.leaders_in('more_highscores', 1).size).to eql(1)
|
737
737
|
end
|
738
738
|
|
739
739
|
it 'should allow you to set custom keys for member, score, rank and member_data' do
|
@@ -749,14 +749,14 @@ describe 'Leaderboard' do
|
|
749
749
|
rank_members_in_leaderboard(5)
|
750
750
|
leaders = @leaderboard.leaders(1, :with_member_data => true)
|
751
751
|
leaders.each do |leader|
|
752
|
-
leader[:custom_member_key].
|
753
|
-
leader[:member].
|
754
|
-
leader[:custom_score_key].
|
755
|
-
leader[:score].
|
756
|
-
leader[:custom_rank_key].
|
757
|
-
leader[:rank].
|
758
|
-
leader[:custom_member_data_key].
|
759
|
-
leader[:member_data].
|
752
|
+
expect(leader[:custom_member_key]).not_to be_nil
|
753
|
+
expect(leader[:member]).to be_nil
|
754
|
+
expect(leader[:custom_score_key]).not_to be_nil
|
755
|
+
expect(leader[:score]).to be_nil
|
756
|
+
expect(leader[:custom_rank_key]).not_to be_nil
|
757
|
+
expect(leader[:rank]).to be_nil
|
758
|
+
expect(leader[:custom_member_data_key]).not_to be_nil
|
759
|
+
expect(leader[:member_data]).to be_nil
|
760
760
|
end
|
761
761
|
end
|
762
762
|
|
@@ -764,7 +764,7 @@ describe 'Leaderboard' do
|
|
764
764
|
@leaderboard = Leaderboard.new('name', {:member_data_namespace => 'md'}, {:host => "127.0.0.1", :db => 15})
|
765
765
|
rank_members_in_leaderboard
|
766
766
|
|
767
|
-
@redis_connection.exists("name:member_data").
|
768
|
-
@redis_connection.exists("name:md").
|
767
|
+
expect(@redis_connection.exists("name:member_data")).to be_falsey
|
768
|
+
expect(@redis_connection.exists("name:md")).to be_truthy
|
769
769
|
end
|
770
770
|
end
|