about summary refs log tree commit diff
path: root/src
diff options
context:
space:
mode:
authorOblarg <emichaelbrnett@gmail.com>2025-09-29 11:04:24 -0400
committerOblarg <emichaelbrnett@gmail.com>2025-09-29 11:18:49 -0400
commitf4a865fea72cbbd1478cada7af19c1bcf0b9d588 (patch)
tree631edbc6e6f8e22dca678f34345b26ce243c1f35 /src
parente0f22df8b9376c86496eb7facaadd7b27ef6e4a8 (diff)
downloadrust-f4a865fea72cbbd1478cada7af19c1bcf0b9d588.tar.gz
rust-f4a865fea72cbbd1478cada7af19c1bcf0b9d588.zip
move test per review feedback
Diffstat (limited to 'src')
-rw-r--r--src/tools/rust-analyzer/crates/ide/src/hover/tests.rs42
-rw-r--r--src/tools/rust-analyzer/crates/mbe/src/tests.rs97
2 files changed, 42 insertions, 97 deletions
diff --git a/src/tools/rust-analyzer/crates/ide/src/hover/tests.rs b/src/tools/rust-analyzer/crates/ide/src/hover/tests.rs
index 1ea11a215f8..8bc0b3f6ab3 100644
--- a/src/tools/rust-analyzer/crates/ide/src/hover/tests.rs
+++ b/src/tools/rust-analyzer/crates/ide/src/hover/tests.rs
@@ -4797,6 +4797,48 @@ fn main() {
 }
 
 #[test]
+fn const_generic_negative_literal_macro_expansion() {
+    // Test that negative literals work correctly in const generics
+    // when used through macro expansion. This ensures the transcriber
+    // doesn't wrap negative literals in parentheses, which would create
+    // invalid syntax like Foo::<(-1)> instead of Foo::<-1>.
+    check(
+        r#"
+struct Foo<const I: i16> {
+    pub value: i16,
+}
+
+impl<const I: i16> Foo<I> {
+    pub fn new(value: i16) -> Self {
+        Self { value }
+    }
+}
+
+macro_rules! create_foo {
+    ($val:expr) => {
+        Foo::<$val>::new($val)
+    };
+}
+
+fn main() {
+    let v$0alue = create_foo!(-1);
+}
+"#,
+        expect![[r#"
+            *value*
+
+            ```rust
+            let value: Foo<-1>
+            ```
+
+            ---
+
+            size = 2, align = 2, no Drop
+        "#]],
+    );
+}
+
+#[test]
 fn hover_self_param_shows_type() {
     check(
         r#"
diff --git a/src/tools/rust-analyzer/crates/mbe/src/tests.rs b/src/tools/rust-analyzer/crates/mbe/src/tests.rs
index 589e169d30a..56034516ef3 100644
--- a/src/tools/rust-analyzer/crates/mbe/src/tests.rs
+++ b/src/tools/rust-analyzer/crates/mbe/src/tests.rs
@@ -476,100 +476,3 @@ fn minus_belongs_to_literal() {
             --"#]],
     );
 }
-
-#[test]
-fn negative_literals_in_const_generics() {
-    // Test that negative literals work correctly in declarative macros
-    // when used as const generic arguments. The issue was that expressions
-    // like -1 would be wrapped in parentheses, creating invalid syntax
-    // Foo::<(-1)> instead of the correct Foo::<-1>.
-    let decl = r#"
-($val:expr) => {
-    struct Foo<const I: i16> {
-        pub value: i16,
-    }
-
-    impl<const I: i16> Foo<I> {
-        pub fn new(value: i16) -> Self {
-            Self { value }
-        }
-    }
-
-    Foo::<$val>::new($val)
-};
-"#;
-    let check = |args, expect| check(Edition::CURRENT, Edition::CURRENT, decl, args, expect);
-
-    // Test negative integer literal - should produce Foo::<-1>, not Foo::<(-1)>
-    check(
-        "-1",
-        expect![[r#"
-            SUBTREE $$ 1:Root[0000, 0]@0..2#ROOT2024 1:Root[0000, 0]@0..2#ROOT2024
-              IDENT   struct 0:Root[0000, 0]@22..28#ROOT2024
-              IDENT   Foo 0:Root[0000, 0]@29..32#ROOT2024
-              PUNCH   < [alone] 0:Root[0000, 0]@32..33#ROOT2024
-              IDENT   const 0:Root[0000, 0]@33..38#ROOT2024
-              IDENT   I 0:Root[0000, 0]@39..40#ROOT2024
-              PUNCH   : [alone] 0:Root[0000, 0]@40..41#ROOT2024
-              IDENT   i16 0:Root[0000, 0]@42..45#ROOT2024
-              PUNCH   > [alone] 0:Root[0000, 0]@45..46#ROOT2024
-              SUBTREE {} 0:Root[0000, 0]@47..48#ROOT2024 0:Root[0000, 0]@77..78#ROOT2024
-                IDENT   pub 0:Root[0000, 0]@57..60#ROOT2024
-                IDENT   value 0:Root[0000, 0]@61..66#ROOT2024
-                PUNCH   : [alone] 0:Root[0000, 0]@66..67#ROOT2024
-                IDENT   i16 0:Root[0000, 0]@68..71#ROOT2024
-                PUNCH   , [alone] 0:Root[0000, 0]@71..72#ROOT2024
-              IDENT   impl 0:Root[0000, 0]@84..88#ROOT2024
-              PUNCH   < [alone] 0:Root[0000, 0]@88..89#ROOT2024
-              IDENT   const 0:Root[0000, 0]@89..94#ROOT2024
-              IDENT   I 0:Root[0000, 0]@95..96#ROOT2024
-              PUNCH   : [alone] 0:Root[0000, 0]@96..97#ROOT2024
-              IDENT   i16 0:Root[0000, 0]@98..101#ROOT2024
-              PUNCH   > [alone] 0:Root[0000, 0]@101..102#ROOT2024
-              IDENT   Foo 0:Root[0000, 0]@103..106#ROOT2024
-              PUNCH   < [alone] 0:Root[0000, 0]@106..107#ROOT2024
-              IDENT   I 0:Root[0000, 0]@107..108#ROOT2024
-              PUNCH   > [alone] 0:Root[0000, 0]@108..109#ROOT2024
-              SUBTREE {} 0:Root[0000, 0]@110..111#ROOT2024 0:Root[0000, 0]@194..195#ROOT2024
-                IDENT   pub 0:Root[0000, 0]@120..123#ROOT2024
-                IDENT   fn 0:Root[0000, 0]@124..126#ROOT2024
-                IDENT   new 0:Root[0000, 0]@127..130#ROOT2024
-                SUBTREE () 0:Root[0000, 0]@130..131#ROOT2024 0:Root[0000, 0]@141..142#ROOT2024
-                  IDENT   value 0:Root[0000, 0]@131..136#ROOT2024
-                  PUNCH   : [alone] 0:Root[0000, 0]@136..137#ROOT2024
-                  IDENT   i16 0:Root[0000, 0]@138..141#ROOT2024
-                PUNCH   - [joint] 0:Root[0000, 0]@143..144#ROOT2024
-                PUNCH   > [alone] 0:Root[0000, 0]@144..145#ROOT2024
-                IDENT   Self 0:Root[0000, 0]@146..150#ROOT2024
-                SUBTREE {} 0:Root[0000, 0]@151..152#ROOT2024 0:Root[0000, 0]@188..189#ROOT2024
-                  IDENT   Self 0:Root[0000, 0]@165..169#ROOT2024
-                  SUBTREE {} 0:Root[0000, 0]@170..171#ROOT2024 0:Root[0000, 0]@178..179#ROOT2024
-                    IDENT   value 0:Root[0000, 0]@172..177#ROOT2024
-              IDENT   Foo 0:Root[0000, 0]@201..204#ROOT2024
-              PUNCH   : [joint] 0:Root[0000, 0]@204..205#ROOT2024
-              PUNCH   : [joint] 0:Root[0000, 0]@205..206#ROOT2024
-              PUNCH   < [joint] 0:Root[0000, 0]@206..207#ROOT2024
-              PUNCH   - [alone] 1:Root[0000, 0]@0..1#ROOT2024
-              LITERAL Integer 1 1:Root[0000, 0]@1..2#ROOT2024
-              PUNCH   > [joint] 0:Root[0000, 0]@211..212#ROOT2024
-              PUNCH   : [joint] 0:Root[0000, 0]@212..213#ROOT2024
-              PUNCH   : [alone] 0:Root[0000, 0]@213..214#ROOT2024
-              IDENT   new 0:Root[0000, 0]@214..217#ROOT2024
-              SUBTREE () 0:Root[0000, 0]@217..218#ROOT2024 0:Root[0000, 0]@222..223#ROOT2024
-                PUNCH   - [alone] 1:Root[0000, 0]@0..1#ROOT2024
-                LITERAL Integer 1 1:Root[0000, 0]@1..2#ROOT2024
-
-            struct Foo<const I:i16>{
-                pub value:i16,
-            }
-            impl <const I:i16>Foo<I>{
-                pub fn new(value:i16) -> Self {
-                    Self {
-                        value
-                    }
-                }
-
-                }
-            Foo::<-1>::new(-1)"#]],
-    );
-}