feat: test error spans and messages

compiler/noirc_frontend/src/ast/statement.rs

@@ -505,7 +505,11 @@ impl PathSegment {
     ///
     /// Returns an empty span at the end of `foo` if there's no turbofish.
     pub fn turbofish_span(&self) -> Span {
-        Span::from(self.ident.span().end()..self.location.span.end())
+        if self.ident.location().file == self.location.file {
+            Span::from(self.ident.span().end()..self.location.span.end())
+        } else {
+            self.location.span
+        }
     }
 
     pub fn turbofish_location(&self) -> Location {

compiler/noirc_frontend/src/hir/resolution/errors.rs

@@ -309,7 +309,7 @@ impl<'a> From<&'a ResolverError> for Diagnostic {
 
                 let mut diagnostic = Diagnostic::simple_warning(
                     format!("unused variable {name}"),
-                    "unused variable ".to_string(),
+                    "unused variable".to_string(),
                     ident.location(),
                 );
                 diagnostic.unnecessary = true;

compiler/noirc_frontend/src/hir/resolution/import.rs

@@ -115,7 +115,8 @@ impl<'a> From<&'a PathResolutionError> for CustomDiagnostic {
                 CustomDiagnostic::simple_warning(error.to_string(), String::new(), ident.location())
             }
             PathResolutionError::UnresolvedWithPossibleTraitsToImport { ident, traits } => {
-                let traits = vecmap(traits, |trait_name| format!("`{}`", trait_name));
+                let mut traits = vecmap(traits, |trait_name| format!("`{}`", trait_name));
+                traits.sort();
                 CustomDiagnostic::simple_error(
                     error.to_string(),
                     format!(
@@ -126,7 +127,8 @@ impl<'a> From<&'a PathResolutionError> for CustomDiagnostic {
                 )
             }
             PathResolutionError::MultipleTraitsInScope { ident, traits } => {
-                let traits = vecmap(traits, |trait_name| format!("`{}`", trait_name));
+                let mut traits = vecmap(traits, |trait_name| format!("`{}`", trait_name));
+                traits.sort();
                 CustomDiagnostic::simple_error(
                     error.to_string(),
                     format!(

compiler/noirc_frontend/src/parser/parser/path.rs

@@ -125,7 +125,11 @@ impl Parser<'_> {
                     None
                 };
 
-                segments.push(PathSegment { ident, generics, location });
+                segments.push(PathSegment {
+                    ident,
+                    generics,
+                    location: self.location_since(location),
+                });
 
                 if self.at(Token::DoubleColon)
                     && matches!(self.next_token.token(), Token::Ident(..))