Skip to content

Commit

Permalink
Rename weight tie breaking to points tie breaking
Browse files Browse the repository at this point in the history
  • Loading branch information
maurolacy committed Feb 9, 2022
1 parent d8810c0 commit 059b285
Show file tree
Hide file tree
Showing 6 changed files with 18 additions and 18 deletions.
18 changes: 9 additions & 9 deletions contracts/tg4-engagement/src/contract.rs
Original file line number Diff line number Diff line change
Expand Up @@ -712,8 +712,8 @@ pub fn query(deps: Deps, env: Env, msg: QueryMsg) -> StdResult<Binary> {
ListMembersByPoints { start_after, limit } => {
to_binary(&list_members_by_points(deps, start_after, limit)?)
}
ListMembersByWeightTieBreaking { start_after, limit } => to_binary(
&list_members_by_weight_tie_breaking(deps, start_after, limit)?,
ListMembersByPointsTieBreaking { start_after, limit } => to_binary(
&list_members_by_points_tie_breaking(deps, start_after, limit)?,
),
TotalPoints {} => to_binary(&query_total_points(deps)?),
Admin {} => to_binary(&ADMIN.query_admin(deps)?),
Expand Down Expand Up @@ -878,7 +878,7 @@ fn list_members_by_points(
Ok(MemberListResponse { members: members? })
}

fn list_members_by_weight_tie_breaking(
fn list_members_by_points_tie_breaking(
deps: Deps,
start_after: Option<(Member, u64)>,
limit: Option<u32>,
Expand Down Expand Up @@ -1155,7 +1155,7 @@ mod tests {
let info = mock_info(INIT_ADMIN, &[]);
execute_update_members(deps.as_mut(), env.clone(), info.clone(), add, remove).unwrap();

let members = list_members_by_weight_tie_breaking(deps.as_ref(), None, None)
let members = list_members_by_points_tie_breaking(deps.as_ref(), None, None)
.unwrap()
.members;
assert_eq!(members.len(), 3);
Expand Down Expand Up @@ -1201,7 +1201,7 @@ mod tests {
let env = mock_env_height(20);
execute_update_members(deps.as_mut(), env, info, add, remove).unwrap();

let members = list_members_by_weight_tie_breaking(deps.as_ref(), None, None)
let members = list_members_by_points_tie_breaking(deps.as_ref(), None, None)
.unwrap()
.members;
assert_eq!(members.len(), 3);
Expand Down Expand Up @@ -1234,7 +1234,7 @@ mod tests {
);

// Test pagination / limits work
let members = list_members_by_weight_tie_breaking(deps.as_ref(), None, Some(1))
let members = list_members_by_points_tie_breaking(deps.as_ref(), None, Some(1))
.unwrap()
.members;
assert_eq!(members.len(), 1);
Expand All @@ -1252,7 +1252,7 @@ mod tests {

// Next page
let start_after = Some(members[0].clone());
let members = list_members_by_weight_tie_breaking(deps.as_ref(), start_after, Some(1))
let members = list_members_by_points_tie_breaking(deps.as_ref(), start_after, Some(1))
.unwrap()
.members;
assert_eq!(members.len(), 1);
Expand All @@ -1270,7 +1270,7 @@ mod tests {

// Next page
let start_after = Some(members[0].clone());
let members = list_members_by_weight_tie_breaking(deps.as_ref(), start_after, Some(1))
let members = list_members_by_points_tie_breaking(deps.as_ref(), start_after, Some(1))
.unwrap()
.members;
assert_eq!(members.len(), 1);
Expand All @@ -1288,7 +1288,7 @@ mod tests {

// Assert there's no more
let start_after = Some(members[0].clone());
let members = list_members_by_weight_tie_breaking(deps.as_ref(), start_after, Some(1))
let members = list_members_by_points_tie_breaking(deps.as_ref(), start_after, Some(1))
.unwrap()
.members;
assert_eq!(members.len(), 0);
Expand Down
2 changes: 1 addition & 1 deletion contracts/tg4-engagement/src/msg.rs
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ pub enum QueryMsg {
},
/// Returns MemberListResponse2, sorted by weight descending,
/// breaking ties by start height
ListMembersByWeightTieBreaking {
ListMembersByPointsTieBreaking {
start_after: Option<(Member, u64)>,
limit: Option<u32>,
},
Expand Down
6 changes: 3 additions & 3 deletions contracts/tg4-mixer/src/contract.rs
Original file line number Diff line number Diff line change
Expand Up @@ -360,8 +360,8 @@ pub fn query(deps: Deps, _env: Env, msg: QueryMsg) -> StdResult<Binary> {
ListMembersByPoints { start_after, limit } => {
to_binary(&list_members_by_points(deps, start_after, limit)?)
}
ListMembersByWeightTieBreaking { start_after, limit } => to_binary(
&list_members_by_weight_tie_breaking(deps, start_after, limit)?,
ListMembersByPointsTieBreaking { start_after, limit } => to_binary(
&list_members_by_points_tie_breaking(deps, start_after, limit)?,
),
TotalPoints {} => to_binary(&query_total_points(deps)?),
Groups {} => to_binary(&query_groups(deps)?),
Expand Down Expand Up @@ -465,7 +465,7 @@ fn list_members_by_points(
Ok(MemberListResponse { members: members? })
}

fn list_members_by_weight_tie_breaking(
fn list_members_by_points_tie_breaking(
deps: Deps,
start_after: Option<(Member, u64)>,
limit: Option<u32>,
Expand Down
2 changes: 1 addition & 1 deletion contracts/tg4-mixer/src/msg.rs
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ pub enum QueryMsg {
},
/// Returns MemberListResponse2, sorted by weight descending,
/// breaking ties by start height
ListMembersByWeightTieBreaking {
ListMembersByPointsTieBreaking {
start_after: Option<(Member, u64)>,
limit: Option<u32>,
},
Expand Down
6 changes: 3 additions & 3 deletions contracts/tg4-stake/src/contract.rs
Original file line number Diff line number Diff line change
Expand Up @@ -457,8 +457,8 @@ pub fn query(deps: Deps, _env: Env, msg: QueryMsg) -> StdResult<Binary> {
ListMembersByPoints { start_after, limit } => {
to_binary(&list_members_by_points(deps, start_after, limit)?)
}
ListMembersByWeightTieBreaking { start_after, limit } => to_binary(
&list_members_by_weight_tie_breaking(deps, start_after, limit)?,
ListMembersByPointsTieBreaking { start_after, limit } => to_binary(
&list_members_by_points_tie_breaking(deps, start_after, limit)?,
),
TotalPoints {} => to_binary(&query_total_points(deps)?),
Claims {
Expand Down Expand Up @@ -576,7 +576,7 @@ fn list_members_by_points(
Ok(MemberListResponse { members: members? })
}

fn list_members_by_weight_tie_breaking(
fn list_members_by_points_tie_breaking(
deps: Deps,
start_after: Option<(Member, u64)>,
limit: Option<u32>,
Expand Down
2 changes: 1 addition & 1 deletion contracts/tg4-stake/src/msg.rs
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ pub enum QueryMsg {
},
/// Returns MemberListResponse2, sorted by weight descending,
/// breaking ties by start height
ListMembersByWeightTieBreaking {
ListMembersByPointsTieBreaking {
start_after: Option<(Member, u64)>,
limit: Option<u32>,
},
Expand Down

0 comments on commit 059b285

Please sign in to comment.