Skip to content

Commit a664532

Browse files
committed
Remove unused guild role helpers
1 parent 65a2e68 commit a664532

2 files changed

Lines changed: 7 additions & 32 deletions

File tree

src/models/guild/role.rs

Lines changed: 0 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -129,35 +129,10 @@ impl GuildRole {
129129
Self::default()
130130
}
131131

132-
/// Returns true if this role is hoisted (displayed separately).
133-
pub fn is_hoisted(&self) -> bool {
134-
self.hoist != 0
135-
}
136-
137132
/// Converts the role to the numeric hoist value expected by the API.
138133
pub fn hoist_as_u32(&self) -> u32 {
139134
self.hoist_value()
140135
}
141-
142-
/// Gets the role's color as a hex value.
143-
pub fn color_hex(&self) -> Option<String> {
144-
(self.color != 0).then(|| format!("#{:06X}", self.color))
145-
}
146-
147-
/// Gets the number of members with this role.
148-
pub fn member_count(&self) -> u32 {
149-
self.member_count
150-
}
151-
152-
/// Gets the member limit for this role.
153-
pub fn get_member_limit(&self) -> u32 {
154-
self.member_limit
155-
}
156-
157-
/// Returns true if the role has reached its member limit.
158-
pub fn is_at_member_limit(&self) -> bool {
159-
self.member_limit > 0 && self.member_count >= self.member_limit
160-
}
161136
}
162137

163138
impl HasId for GuildRole {

src/models/guild/tests.rs

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -128,8 +128,8 @@ fn test_role_creation() {
128128
let role = GuildRole::new();
129129
assert_eq!(role.id, "");
130130
assert_eq!(role.name, "");
131-
assert!(!role.is_hoisted());
132-
assert_eq!(role.member_count(), 0);
131+
assert_eq!(role.hoist, 0);
132+
assert_eq!(role.member_count, 0);
133133
}
134134

135135
#[test]
@@ -144,11 +144,11 @@ fn test_role_with_data() {
144144

145145
assert_eq!(role.id(), Some(&"role123".to_string()));
146146
assert_eq!(role.name(), "Admin");
147-
assert_eq!(role.color_hex(), Some("#FF0000".to_string()));
148-
assert!(role.is_hoisted());
149-
assert_eq!(role.member_count(), 5);
150-
assert_eq!(role.get_member_limit(), 10);
151-
assert!(!role.is_at_member_limit());
147+
assert_eq!(role.color, 0xFF0000);
148+
assert_eq!(role.hoist, 1);
149+
assert_eq!(role.member_count, 5);
150+
assert_eq!(role.member_limit, 10);
151+
assert!(role.member_count < role.member_limit);
152152
}
153153

154154
#[test]

0 commit comments

Comments
 (0)