From d3cdf6ea76830e9a6f7df216d14f59270750f7a1 Mon Sep 17 00:00:00 2001
From: Revertron <r@revertron.com>
Date: Mon, 27 Oct 2025 01:37:38 +0100
Subject: [PATCH] Fixed warnings in some tests.

---
 src/dns/resolve.rs |  4 ++--
 src/dns/server.rs  | 10 +++++-----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/dns/resolve.rs b/src/dns/resolve.rs
index 9acf7bb..394c653 100644
--- a/src/dns/resolve.rs
+++ b/src/dns/resolve.rs
@@ -251,7 +251,7 @@ mod tests {
 
         let mut resolver = context.create_resolver(Arc::clone(&context));
 
-        // First verify that we get a match back
+        // First, verify that we get a match back
         {
             let res = match resolver.resolve("google.com", QueryType::A, true) {
                 Ok(x) => x,
@@ -268,7 +268,7 @@ mod tests {
             }
         };
 
-        // Do the same lookup again, and verify that it's present in the cache
+        // Do the same lookup again and verify that it's present in the cache
         // and that the counter has been updated
         {
             let res = match resolver.resolve("google.com", QueryType::A, true) {
diff --git a/src/dns/server.rs b/src/dns/server.rs
index 05f617d..2c75402 100644
--- a/src/dns/server.rs
+++ b/src/dns/server.rs
@@ -441,7 +441,7 @@ mod tests {
         }));
 
         match Arc::get_mut(&mut context) {
-            Some(mut ctx) => {
+            Some(ctx) => {
                 ctx.resolve_strategy = ResolveStrategy::Forward { upstreams: vec![String::from("127.0.0.1:53")] };
             }
             None => panic!()
@@ -460,7 +460,7 @@ mod tests {
             }
         };
 
-        // A successful resolve, that also resolves a CNAME without recursive lookup
+        // A successful resolve that also resolves a CNAME without recursive lookup
         {
             let res = execute_query(Arc::clone(&context), &build_query("www.facebook.com", QueryType::CNAME));
             assert_eq!(2, res.answers.len());
@@ -480,7 +480,7 @@ mod tests {
             }
         };
 
-        // A successful resolve, that also resolves a CNAME through recursive lookup
+        // A successful resolve that also resolves a CNAME through recursive lookup
         {
             let res = execute_query(Arc::clone(&context), &build_query("www.microsoft.com", QueryType::CNAME));
             dbg!(&res);
@@ -503,7 +503,7 @@ mod tests {
 
         // Disable recursive resolves to generate a failure
         match Arc::get_mut(&mut context) {
-            Some(mut ctx) => {
+            Some(ctx) => {
                 ctx.allow_recursive = false;
             }
             None => panic!()
@@ -531,7 +531,7 @@ mod tests {
         }));
 
         match Arc::get_mut(&mut context2) {
-            Some(mut ctx) => {
+            Some(ctx) => {
                 ctx.resolve_strategy = ResolveStrategy::Forward { upstreams: vec![String::from("127.0.0.1:53")] };
             }
             None => panic!()