Make modules case insenstive on windows

src/agent/coverage/src/allowlist.rs

@@ -2,7 +2,7 @@
 // Licensed under the MIT License.
 
 use anyhow::Result;
-use regex::{Regex, RegexSet};
+use regex::{Regex, RegexBuilder, RegexSet};
 use std::path::Path;
 
 #[derive(Clone, Debug)]
@@ -144,7 +144,13 @@ fn glob_to_regex(expr: &str) -> Result<Regex> {
     // Anchor to line start and end.
     let expr = format!("^{expr}$");
 
-    Ok(Regex::new(&expr)?)
+    let mut rb = RegexBuilder::new(&expr);
+
+    if cfg!(windows) {
+        rb.case_insensitive(true);
+    }
+
+    Ok(rb.build()?)
 }
 
 #[cfg(test)]

src/agent/coverage/tests/snapshot.rs

@@ -43,7 +43,8 @@ fn windows_snapshot_tests() {
         };
 
         // filter to just the input test file:
-        let source_allowlist = AllowList::parse(&input_path.to_string_lossy()).unwrap();
+        let source_allowlist =
+            AllowList::parse(&input_path.to_string_lossy().to_ascii_lowercase()).unwrap();
 
         let exe_cmd = std::process::Command::new(&exe_name);
         let recorded = coverage::CoverageRecorder::new(exe_cmd)
@@ -52,6 +53,13 @@ fn windows_snapshot_tests() {
             .record()
             .unwrap();
 
+        // Windows modules should be case insensitive
+        recorded
+            .coverage
+            .modules
+            .keys()
+            .for_each(|k| assert_eq!(k.to_string().to_ascii_lowercase(), k.to_string()));
+
         // generate source-line coverage info
         let source =
             coverage::source::binary_to_source_coverage(&recorded.coverage, &source_allowlist)

src/agent/coverage/tests/snapshots/snapshot__windows_snapshot_tests.snap

@@ -1,7 +1,7 @@
 ---
 source: coverage/tests/snapshot.rs
 expression: result
-input_file: coverage/tests/windows/inlinee.cpp
+input_file: coverage/tests/windows/Inlinee.cpp
 ---
 [ ] #include <iostream>
 [ ] 

src/agent/debugger/src/module.rs

@@ -47,6 +47,8 @@ impl Module {
             "???".into()
         });
 
+        let path = PathBuf::from(path.as_os_str().to_ascii_lowercase());
+
         let image_details = get_image_details(&path)?;
 
         Ok(Module {

src/agent/onefuzz/src/expand.rs

@@ -128,7 +128,8 @@ impl<'a> Expand<'a> {
 
     fn input_file_sha256(&self) -> Result<ExpandedValue<'a>> {
         let Some(val) = self.values.get(PlaceHolder::Input.get_string()) else {
-            bail!("no value found for {}, unable to evaluate {}",
+            bail!(
+                "no value found for {}, unable to evaluate {}",
                 PlaceHolder::Input.get_string(),
                 PlaceHolder::InputFileSha256.get_string(),
             )
@@ -149,7 +150,8 @@ impl<'a> Expand<'a> {
 
     fn extract_file_name_no_ext(&self) -> Result<ExpandedValue<'a>> {
         let Some(val) = self.values.get(PlaceHolder::Input.get_string()) else {
-            bail!("no value found for {}, unable to evaluate {}",
+            bail!(
+                "no value found for {}, unable to evaluate {}",
                 PlaceHolder::Input.get_string(),
                 PlaceHolder::InputFileNameNoExt.get_string(),
             )
@@ -173,7 +175,8 @@ impl<'a> Expand<'a> {
 
     fn extract_file_name(&self) -> Result<ExpandedValue<'a>> {
         let Some(val) = self.values.get(PlaceHolder::Input.get_string()) else {
-            bail!("no value found for {}, unable to evaluate {}",
+            bail!(
+                "no value found for {}, unable to evaluate {}",
                 PlaceHolder::Input.get_string(),
                 PlaceHolder::InputFileName.get_string(),
             )