fix: refactor nested if statements to satisfy clippy::collapsible_if

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: copyleftdev

crates/zgrok-integration-tests/src/tunnel.rs

@@ -128,23 +128,24 @@ mod tests {
             let mut writer = FramedWrite::new(write, FrameCodec::new());
 
             if let Some(Ok(frame)) = reader.next().await {
-                if frame.header.frame_type == FrameType::Auth {
-                    let auth_req: AuthRequest = Message::from_json_bytes(&frame.payload).unwrap();
-
-                    let tunnel_id = Uuid::new_v4();
-                    let response = AuthResponse::new(
-                        tunnel_id,
-                        auth_req
-                            .requested_subdomain
-                            .unwrap_or_else(|| "random".to_string()),
-                        "1.0.0",
-                        3600,
-                    );
-                    let response_payload = Bytes::from(response.to_json_bytes().unwrap());
-                    let response_frame = Frame::auth(response_payload).unwrap();
-                    writer.send(response_frame).await.unwrap();
-                    return true;
+                if frame.header.frame_type != FrameType::Auth {
+                    return false;
                 }
+                let auth_req: AuthRequest = Message::from_json_bytes(&frame.payload).unwrap();
+
+                let tunnel_id = Uuid::new_v4();
+                let response = AuthResponse::new(
+                    tunnel_id,
+                    auth_req
+                        .requested_subdomain
+                        .unwrap_or_else(|| "random".to_string()),
+                    "1.0.0",
+                    3600,
+                );
+                let response_payload = Bytes::from(response.to_json_bytes().unwrap());
+                let response_frame = Frame::auth(response_payload).unwrap();
+                writer.send(response_frame).await.unwrap();
+                return true;
             }
             false
         });
@@ -162,10 +163,11 @@ mod tests {
             writer.send(auth_frame).await.unwrap();
 
             if let Some(Ok(frame)) = reader.next().await {
-                if frame.header.frame_type == FrameType::Auth {
-                    let resp: AuthResponse = Message::from_json_bytes(&frame.payload).unwrap();
-                    return resp.assigned_subdomain == "myapp";
+                if frame.header.frame_type != FrameType::Auth {
+                    return false;
                 }
+                let resp: AuthResponse = Message::from_json_bytes(&frame.payload).unwrap();
+                return resp.assigned_subdomain == "myapp";
             }
             false
         });
@@ -332,11 +334,12 @@ mod tests {
             writer.send(settings_frame).await.unwrap();
 
             if let Some(Ok(frame)) = reader.next().await {
-                if frame.header.frame_type == FrameType::Settings {
-                    let client_settings: Settings =
-                        Message::from_json_bytes(&frame.payload).unwrap();
-                    return Some(client_settings);
+                if frame.header.frame_type != FrameType::Settings {
+                    return None;
                 }
+                let client_settings: Settings =
+                    Message::from_json_bytes(&frame.payload).unwrap();
+                return Some(client_settings);
             }
             None
         });
@@ -347,17 +350,18 @@ mod tests {
             let mut writer = FramedWrite::new(write, FrameCodec::new());
 
             if let Some(Ok(frame)) = reader.next().await {
-                if frame.header.frame_type == FrameType::Settings {
-                    let server_settings: Settings =
-                        Message::from_json_bytes(&frame.payload).unwrap();
+                if frame.header.frame_type != FrameType::Settings {
+                    return None;
+                }
+                let server_settings: Settings =
+                    Message::from_json_bytes(&frame.payload).unwrap();
 
-                    let client_settings = Settings::new(50, 30, 32768);
-                    let settings_payload = Bytes::from(client_settings.to_json_bytes().unwrap());
-                    let settings_frame = Frame::settings(settings_payload).unwrap();
-                    writer.send(settings_frame).await.unwrap();
+                let client_settings = Settings::new(50, 30, 32768);
+                let settings_payload = Bytes::from(client_settings.to_json_bytes().unwrap());
+                let settings_frame = Frame::settings(settings_payload).unwrap();
+                writer.send(settings_frame).await.unwrap();
 
-                    return Some(server_settings);
-                }
+                return Some(server_settings);
             }
             None
         });