misc: Remove redundant pub keyword of helper functions

Functions marked as `pub` in test module need to be documented. Since these are literally helper functions and there is no need to be `pub`, remove these `pub` keywords. Signed-off-by: Ruoqing He <[email protected]>
UiPath · Oct 18, 2024 · b708db3 · b708db3
1 parent 416fe5e
commit b708db3
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/rate_limiter/src/group.rs b/rate_limiter/src/group.rs
@@ -309,12 +309,12 @@ pub(crate) mod tests {
     use crate::{TokenBucket, TokenType, REFILL_TIMER_INTERVAL_MS};
 
     impl RateLimiterGroupHandle {
-        pub fn bandwidth(&self) -> Option<TokenBucket> {
+        fn bandwidth(&self) -> Option<TokenBucket> {
             let guard = self.inner.rate_limiter.inner.lock().unwrap();
             guard.bandwidth.clone()
         }
 
-        pub fn ops(&self) -> Option<TokenBucket> {
+        fn ops(&self) -> Option<TokenBucket> {
             let guard = self.inner.rate_limiter.inner.lock().unwrap();
             guard.ops.clone()
         }

diff --git a/rate_limiter/src/lib.rs b/rate_limiter/src/lib.rs
@@ -544,7 +544,8 @@ pub(crate) mod tests {
         }
 
         // After a restore, we cannot be certain that the last_update field has the same value.
-        pub fn partial_eq(&self, other: &TokenBucket) -> bool {
+        #[allow(dead_code)]
+        fn partial_eq(&self, other: &TokenBucket) -> bool {
             (other.capacity() == self.capacity())
                 && (other.one_time_burst() == self.one_time_burst())
                 && (other.refill_time_ms() == self.refill_time_ms())
@@ -553,12 +554,12 @@ pub(crate) mod tests {
     }
 
     impl RateLimiter {
-        pub fn bandwidth(&self) -> Option<TokenBucket> {
+        fn bandwidth(&self) -> Option<TokenBucket> {
             let guard = self.inner.lock().unwrap();
             guard.bandwidth.clone()
         }
 
-        pub fn ops(&self) -> Option<TokenBucket> {
+        fn ops(&self) -> Option<TokenBucket> {
             let guard = self.inner.lock().unwrap();
             guard.ops.clone()
         }