Cleanup

quadratichq · Dec 4, 2023 · f2955d4 · f2955d4
1 parent 8660be0
commit f2955d4
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 8 deletions.
diff --git a/quadratic-multiplayer/src/message.rs b/quadratic-multiplayer/src/message.rs
@@ -252,7 +252,7 @@ pub(crate) mod tests {
         let _user_2 = add_new_user_to_room(file_id, state.clone()).await;
         let message = MessageResponse::ChangeSelection {
             user_id: user_1.id.clone(),
-            file_id: file_id,
+            file_id,
             selection: "test".to_string(),
         };
         broadcast(user_1.id.clone(), file_id, state, message).unwrap();
@@ -268,7 +268,7 @@ pub(crate) mod tests {
         let _user_2 = add_new_user_to_room(file_id, state.clone()).await;
         let message = MessageResponse::Transaction {
             user_id: user_1.id.clone(),
-            file_id: file_id,
+            file_id,
             operations: "test".to_string(),
         };
         broadcast(user_1.id.clone(), file_id, state, message).unwrap();

diff --git a/quadratic-multiplayer/src/server.rs b/quadratic-multiplayer/src/server.rs
@@ -189,10 +189,7 @@ pub(crate) mod tests {
         let user_id = user.id.clone();
         let user2 = new_user();
         let file_id = Uuid::new_v4();
-        let request = MessageRequest::LeaveRoom {
-            user_id: user_id,
-            file_id,
-        };
+        let request = MessageRequest::LeaveRoom { user_id, file_id };
         let expected = MessageResponse::Room {
             room: Room {
                 file_id,

diff --git a/quadratic-multiplayer/src/state.rs b/quadratic-multiplayer/src/state.rs
@@ -89,7 +89,7 @@ impl State {
         let num_in_room = get_room!(self, file_id)?.users.len();
 
         tracing::trace!(
-            "User {:?} leaving room {}, {} left",
+            "User {:?} is leaving room {}, {} user(s) left",
             user_id,
             file_id,
             num_in_room
@@ -153,7 +153,7 @@ mod tests {
     use super::*;
 
     #[tokio::test]
-    async fn enters_retrieves_and_leaves_a_room() {
+    async fn enters_retrieves_leaves_and_removes_a_room() {
         let state = State::new();
         let file_id = Uuid::new_v4();
         let user = new_user();