Skip to content
This repository was archived by the owner on Apr 8, 2026. It is now read-only.

Commit 3d49cb0

Browse files
GraylinKimczarneckid
authored andcommitted
Fix TieRankingLeaderboard.rank_member_across (#65)
1 parent 2b832a1 commit 3d49cb0

2 files changed

Lines changed: 7 additions & 1 deletion

File tree

leaderboard/tie_ranking_leaderboard.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -120,7 +120,7 @@ def rank_member_across(
120120
@param member_data [String] Optional member data.
121121
'''
122122
for leaderboard_name in leaderboards:
123-
self.rank_member_in(leaderboard, member, score, member_data)
123+
self.rank_member_in(leaderboard_name, member, score, member_data)
124124

125125
def rank_members_in(self, leaderboard_name, members_and_scores):
126126
'''

test/leaderboard/tie_ranking_leaderboard_test.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -230,6 +230,12 @@ def test_it_should_output_the_correct_rank_when_initial_score_is_0_and_then_late
230230
self.leaderboard.rank_for('member_1').should.equal(2)
231231
self.leaderboard.rank_for('member_2').should.equal(2)
232232

233+
def test_rank_member_across(self):
234+
self.leaderboard.rank_member_across(
235+
['highscores', 'more_highscores'], 'david', 50000, {'member_name': 'david'})
236+
len(self.leaderboard.leaders_in('highscores', 1)).should.equal(1)
237+
len(self.leaderboard.leaders_in('more_highscores', 1)).should.equal(1)
238+
233239
def test_it_should_correctly_pop_ties_namespace_from_options(self):
234240
self.leaderboard = TieRankingLeaderboard('ties', ties_namespace='ties_namespace')
235241
self.__rank_members_in_leaderboard(26)

0 commit comments

Comments
 (0)