fix some clippy errors in tests

cloudflare · Jul 21, 2023 · 5c05436 · 5c05436
1 parent 34bc920
commit 5c05436
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 19 deletions.
diff --git a/quiche/src/h3/mod.rs b/quiche/src/h3/mod.rs
@@ -3475,21 +3475,21 @@ mod tests {
 
         let (_, ev) = s.poll_server().unwrap();
         let ev_headers = Event::Headers {
-            list: reqs[(0 / 4) as usize].clone(),
+            list: reqs[0].clone(),
             has_body: true,
         };
         assert_eq!(ev, ev_headers);
 
         let (_, ev) = s.poll_server().unwrap();
         let ev_headers = Event::Headers {
-            list: reqs[(4 / 4) as usize].clone(),
+            list: reqs[1].clone(),
             has_body: true,
         };
         assert_eq!(ev, ev_headers);
 
         let (_, ev) = s.poll_server().unwrap();
         let ev_headers = Event::Headers {
-            list: reqs[(8 / 4) as usize].clone(),
+            list: reqs[2].clone(),
             has_body: true,
         };
         assert_eq!(ev, ev_headers);

diff --git a/quiche/src/stream.rs b/quiche/src/stream.rs
@@ -3558,10 +3558,11 @@ mod tests {
     #[test]
     fn writable_prioritized_default_priority() {
         let local_tp = crate::TransportParams::default();
-        let mut peer_tp = crate::TransportParams::default();
-
-        peer_tp.initial_max_stream_data_bidi_local = 100;
-        peer_tp.initial_max_stream_data_uni = 100;
+        let peer_tp = crate::TransportParams {
+            initial_max_stream_data_bidi_local: 100,
+            initial_max_stream_data_uni: 100,
+            ..Default::default()
+        };
 
         let mut streams = StreamMap::new(100, 100, 100);
 
@@ -3593,10 +3594,11 @@ mod tests {
     #[test]
     fn writable_prioritized_insert_order() {
         let local_tp = crate::TransportParams::default();
-        let mut peer_tp = crate::TransportParams::default();
-
-        peer_tp.initial_max_stream_data_bidi_local = 100;
-        peer_tp.initial_max_stream_data_uni = 100;
+        let peer_tp = crate::TransportParams {
+            initial_max_stream_data_bidi_local: 100,
+            initial_max_stream_data_uni: 100,
+            ..Default::default()
+        };
 
         let mut streams = StreamMap::new(100, 100, 100);
 
@@ -3627,10 +3629,11 @@ mod tests {
     #[test]
     fn writable_prioritized_mixed_urgency() {
         let local_tp = crate::TransportParams::default();
-        let mut peer_tp = crate::TransportParams::default();
-
-        peer_tp.initial_max_stream_data_bidi_local = 100;
-        peer_tp.initial_max_stream_data_uni = 100;
+        let peer_tp = crate::TransportParams {
+            initial_max_stream_data_bidi_local: 100,
+            initial_max_stream_data_uni: 100,
+            ..Default::default()
+        };
 
         let mut streams = StreamMap::new(100, 100, 100);
 
@@ -3729,10 +3732,11 @@ mod tests {
     #[test]
     fn writable_prioritized_mixed_urgencies_incrementals() {
         let local_tp = crate::TransportParams::default();
-        let mut peer_tp = crate::TransportParams::default();
-
-        peer_tp.initial_max_stream_data_bidi_local = 100;
-        peer_tp.initial_max_stream_data_uni = 100;
+        let peer_tp = crate::TransportParams {
+            initial_max_stream_data_bidi_local: 100,
+            initial_max_stream_data_uni: 100,
+            ..Default::default()
+        };
 
         let mut streams = StreamMap::new(100, 100, 100);