[review] add lots more tests

Author: tommilligan

Diff for: tests/source/skip_macro_invocations/all.rs

@@ -0,0 +1,11 @@
+// rustfmt-skip_macro_invocations: ["*"]
+
+// Should skip this invocation
+items!(
+        const _: u8 = 0;
+);
+
+// Should skip this invocation
+renamed_items!(
+        const _: u8 = 0;
+);

Diff for: tests/source/skip_macro_names_all.rs renamed to tests/source/skip_macro_invocations/all_and_name.rs

@@ -1,16 +1,11 @@
 // rustfmt-skip_macro_invocations: ["*","items"]
 
-macro_rules! items {
-    ($($arg:item)*) => { $($arg)* };
-}
-
 // Should skip this invocation
 items!(
         const _: u8 = 0;
 );
 
 // Should also skip this invocation, as the wildcard covers it
-use self::items as renamed_items;
 renamed_items!(
         const _: u8 = 0;
 );

Diff for: tests/source/skip_macro_invocations/empty.rs

@@ -0,0 +1,11 @@
+// rustfmt-skip_macro_invocations: []
+
+// Should not skip this invocation
+items!(
+        const _: u8 = 0;
+);
+
+// Should not skip this invocation
+renamed_items!(
+        const _: u8 = 0;
+);

Diff for: tests/source/skip_macro_names.rs renamed to tests/source/skip_macro_invocations/name.rs

@@ -1,16 +1,11 @@
 // rustfmt-skip_macro_invocations: ["items"]
 
-macro_rules! items {
-    ($($arg:item)*) => { $($arg)* };
-}
-
 // Should skip this invocation
 items!(
         const _: u8 = 0;
 );
 
 // Should not skip this invocation
-use self::items as renamed_items;
 renamed_items!(
         const _: u8 = 0;
 );

Diff for: tests/source/skip_macro_invocations/name_unknown.rs

@@ -0,0 +1,6 @@
+// rustfmt-skip_macro_invocations: ["unknown"]
+
+// Should not skip this invocation
+items!(
+        const _: u8 = 0;
+);

Diff for: tests/source/skip_macro_invocations/names.rs

@@ -0,0 +1,16 @@
+// rustfmt-skip_macro_invocations: ["foo","bar"]
+
+// Should skip this invocation
+foo!(
+        const _: u8 = 0;
+);
+
+// Should skip this invocation
+bar!(
+        const _: u8 = 0;
+);
+
+// Should not skip this invocation
+baz!(
+        const _: u8 = 0;
+);

Diff for: tests/source/skip_macro_invocations/path_qualified_invocation_mismatch.rs

@@ -0,0 +1,6 @@
+// rustfmt-skip_macro_invocations: ["items"]
+
+// Should not skip this invocation
+self::items!(
+        const _: u8 = 0;
+);

Diff for: tests/source/skip_macro_invocations/path_qualified_match.rs

@@ -0,0 +1,6 @@
+// rustfmt-skip_macro_invocations: ["self::items"]
+
+// Should skip this invocation
+self::items!(
+        const _: u8 = 0;
+);

Diff for: tests/source/skip_macro_invocations/path_qualified_name_mismatch.rs

@@ -0,0 +1,6 @@
+// rustfmt-skip_macro_invocations: ["self::items"]
+
+// Should not skip this invocation
+items!(
+        const _: u8 = 0;
+);

Diff for: tests/target/skip_macro_invocations/all.rs

@@ -0,0 +1,11 @@
+// rustfmt-skip_macro_invocations: ["*"]
+
+// Should skip this invocation
+items!(
+        const _: u8 = 0;
+);
+
+// Should skip this invocation
+renamed_items!(
+        const _: u8 = 0;
+);

Diff for: tests/target/skip_macro_names_all.rs renamed to tests/target/skip_macro_invocations/all_and_name.rs

@@ -1,16 +1,11 @@
 // rustfmt-skip_macro_invocations: ["*","items"]
 
-macro_rules! items {
-    ($($arg:item)*) => { $($arg)* };
-}
-
 // Should skip this invocation
 items!(
         const _: u8 = 0;
 );
 
 // Should also skip this invocation, as the wildcard covers it
-use self::items as renamed_items;
 renamed_items!(
         const _: u8 = 0;
 );

Diff for: tests/target/skip_macro_invocations/empty.rs

@@ -0,0 +1,11 @@
+// rustfmt-skip_macro_invocations: []
+
+// Should not skip this invocation
+items!(
+    const _: u8 = 0;
+);
+
+// Should not skip this invocation
+renamed_items!(
+    const _: u8 = 0;
+);

Diff for: tests/target/skip_macro_names.rs renamed to tests/target/skip_macro_invocations/name.rs

@@ -1,16 +1,11 @@
 // rustfmt-skip_macro_invocations: ["items"]
 
-macro_rules! items {
-    ($($arg:item)*) => { $($arg)* };
-}
-
 // Should skip this invocation
 items!(
         const _: u8 = 0;
 );
 
 // Should not skip this invocation
-use self::items as renamed_items;
 renamed_items!(
     const _: u8 = 0;
 );

Diff for: tests/target/skip_macro_invocations/name_unknown.rs

@@ -0,0 +1,6 @@
+// rustfmt-skip_macro_invocations: ["unknown"]
+
+// Should not skip this invocation
+items!(
+    const _: u8 = 0;
+);

Diff for: tests/target/skip_macro_invocations/names.rs

@@ -0,0 +1,16 @@
+// rustfmt-skip_macro_invocations: ["foo","bar"]
+
+// Should skip this invocation
+foo!(
+        const _: u8 = 0;
+);
+
+// Should skip this invocation
+bar!(
+        const _: u8 = 0;
+);
+
+// Should not skip this invocation
+baz!(
+    const _: u8 = 0;
+);

Diff for: tests/target/skip_macro_invocations/path_qualified_invocation_mismatch.rs

@@ -0,0 +1,6 @@
+// rustfmt-skip_macro_invocations: ["items"]
+
+// Should not skip this invocation
+self::items!(
+    const _: u8 = 0;
+);

Diff for: tests/target/skip_macro_invocations/path_qualified_match.rs

@@ -0,0 +1,6 @@
+// rustfmt-skip_macro_invocations: ["self::items"]
+
+// Should skip this invocation
+self::items!(
+        const _: u8 = 0;
+);

Diff for: tests/target/skip_macro_invocations/path_qualified_name_mismatch.rs

@@ -0,0 +1,6 @@
+// rustfmt-skip_macro_invocations: ["self::items"]
+
+// Should not skip this invocation
+items!(
+    const _: u8 = 0;
+);