Remove the unsafe and mut from the atomic TEST_PORT

RedisLabsModules · May 27, 2024 · 758b11a · 758b11a
1 parent 08110de
commit 758b11a
Showing 1 changed file with 5 additions and 7 deletions.
diff --git a/tests/integration.rs b/tests/integration.rs
@@ -21,17 +21,15 @@ struct TestConnection {
     connection: Connection,
 }
 
-static mut TEST_PORT: AtomicU16 = AtomicU16::new(6479);
+static TEST_PORT: AtomicU16 = AtomicU16::new(6479);
 
 impl TestConnection {
     fn new(module_name: &str) -> Self {
-        unsafe {
-            let port = TEST_PORT.fetch_add(1, std::sync::atomic::Ordering::SeqCst);
+        let port = TEST_PORT.fetch_add(1, std::sync::atomic::Ordering::SeqCst);
 
-            Self {
-                _guards: start_redis(module_name, port).expect("Redis instance started."),
-                connection: get_redis_connection(port).expect("Established connection to server."),
-            }
+        Self {
+            _guards: start_redis(module_name, port).expect("Redis instance started."),
+            connection: get_redis_connection(port).expect("Established connection to server."),
         }
     }
 }