Remove clippy lint ignores (#725)

01539475b49f4afd14bbf4dea77304be89c4aace · by Nico Burns · 2024-10-22T07:50:31Z (2 years ago) · parent 7f8ec509b · browse the tree at this commit

Semantic delta · 1

scripts/gentest/src/main.rs

body changed fn main

1 files · +1 −6

scripts/gentest/src/main.rs +1 −6

diff --git a/scripts/gentest/src/main.rs b/scripts/gentest/src/main.rs
index 588cb54b7..77529b812 100644
--- a/scripts/gentest/src/main.rs
+++ b/scripts/gentest/src/main.rs
@@ -1,6 +1,3 @@
-// This is a spurious warning. See https://github.com/rust-lang/rust-clippy/issues/13535
-#![allow(clippy::needless_return)]
-
 use std::fs::{self, OpenOptions};
 use std::io::Write;
 use std::path::{Path, PathBuf};
@@ -80,9 +77,6 @@ async fn main() {
 
     info!("starting webdriver instance");
     let webdriver_url = "http://localhost:4444";
-
-    // TODO: call `.wait()` at an appropriate time (probably at the end of the process?)
-    #[allow(clippy::zombie_processes)]
     let mut webdriver_handle = Command::new("chromedriver")
         .arg("--port=4444")
         .spawn()
@@ -107,6 +101,7 @@ async fn main() {
 
     info!("killing webdriver instance...");
     webdriver_handle.kill().unwrap();
+    webdriver_handle.wait().unwrap();
 
     info!("generating test sources and concatenating...");