From 317b0c2e6df8b82ed6d5ff73e8cf70862cb8deac Mon Sep 17 00:00:00 2001
From: Lukas Wirth <lukastw97@gmail.com>
Date: Thu, 15 Jul 2021 16:27:01 +0200
Subject: [PATCH] Move attribute completion tests

---
 .../src/completions/attribute.rs              | 556 +-----------
 .../src/completions/attribute/derive.rs       |  90 --
 .../src/completions/attribute/lint.rs         |  41 -
 .../src/completions/attribute/repr.rs         | 129 ---
 .../src/completions/unqualified_path.rs       |  12 -
 crates/ide_completion/src/tests.rs            |   8 +-
 crates/ide_completion/src/tests/attribute.rs  | 809 ++++++++++++++++++
 .../src/tests/{items.rs => item.rs}           |   2 +-
 crates/ide_completion/src/tests/item_list.rs  |   1 +
 crates/ide_completion/src/tests/pattern.rs    |   2 +-
 crates/ide_completion/src/tests/type_pos.rs   |   2 +-
 crates/ide_completion/src/tests/use_tree.rs   |   2 +-
 12 files changed, 831 insertions(+), 823 deletions(-)
 create mode 100644 crates/ide_completion/src/tests/attribute.rs
 rename crates/ide_completion/src/tests/{items.rs => item.rs} (97%)

diff --git a/crates/ide_completion/src/completions/attribute.rs b/crates/ide_completion/src/completions/attribute.rs
index 9ee7f2bce6b..53cee3f4e25 100644
--- a/crates/ide_completion/src/completions/attribute.rs
+++ b/crates/ide_completion/src/completions/attribute.rs
@@ -322,550 +322,18 @@ fn parse_comma_sep_input(derive_input: ast::TokenTree) -> Option<FxHashSet<Strin
     Some(input_derives)
 }
 
-#[cfg(test)]
-mod tests {
-    use super::*;
+#[test]
+fn attributes_are_sorted() {
+    let mut attrs = ATTRIBUTES.iter().map(|attr| attr.key());
+    let mut prev = attrs.next().unwrap();
 
-    use expect_test::{expect, Expect};
-
-    use crate::tests::completion_list;
-
-    #[test]
-    fn attributes_are_sorted() {
-        let mut attrs = ATTRIBUTES.iter().map(|attr| attr.key());
-        let mut prev = attrs.next().unwrap();
-
-        attrs.for_each(|next| {
-            assert!(
-                prev < next,
-                r#"ATTRIBUTES array is not sorted, "{}" should come after "{}""#,
-                prev,
-                next
-            );
-            prev = next;
-        });
-    }
-
-    fn check(ra_fixture: &str, expect: Expect) {
-        let actual = completion_list(ra_fixture);
-        expect.assert_eq(&actual);
-    }
-
-    #[test]
-    fn test_attribute_completion_inside_nested_attr() {
-        check(r#"#[cfg($0)]"#, expect![[]])
-    }
-
-    #[test]
-    fn test_attribute_completion_with_existing_attr() {
-        check(
-            r#"#[no_mangle] #[$0] mcall!();"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-            "#]],
-        )
-    }
-
-    #[test]
-    fn complete_attribute_on_source_file() {
-        check(
-            r#"#![$0]"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-                at crate_name = ""
-                at feature(…)
-                at no_implicit_prelude
-                at no_main
-                at no_std
-                at recursion_limit = …
-                at type_length_limit = …
-                at windows_subsystem = "…"
-            "#]],
+    attrs.for_each(|next| {
+        assert!(
+            prev < next,
+            r#"ATTRIBUTES array is not sorted, "{}" should come after "{}""#,
+            prev,
+            next
         );
-    }
-
-    #[test]
-    fn complete_attribute_on_module() {
-        check(
-            r#"#[$0] mod foo;"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-                at macro_use
-                at path = "…"
-            "#]],
-        );
-        check(
-            r#"mod foo {#![$0]}"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-                at no_implicit_prelude
-            "#]],
-        );
-    }
-
-    #[test]
-    fn complete_attribute_on_macro_rules() {
-        check(
-            r#"#[$0] macro_rules! foo {}"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-                at macro_export
-                at macro_use
-            "#]],
-        );
-    }
-
-    #[test]
-    fn complete_attribute_on_macro_def() {
-        check(
-            r#"#[$0] macro foo {}"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-            "#]],
-        );
-    }
-
-    #[test]
-    fn complete_attribute_on_extern_crate() {
-        check(
-            r#"#[$0] extern crate foo;"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-                at macro_use
-            "#]],
-        );
-    }
-
-    #[test]
-    fn complete_attribute_on_use() {
-        check(
-            r#"#[$0] use foo;"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-            "#]],
-        );
-    }
-
-    #[test]
-    fn complete_attribute_on_type_alias() {
-        check(
-            r#"#[$0] type foo = ();"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-            "#]],
-        );
-    }
-
-    #[test]
-    fn complete_attribute_on_struct() {
-        check(
-            r#"#[$0] struct Foo;"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-                at derive(…)
-                at repr(…)
-                at non_exhaustive
-            "#]],
-        );
-    }
-
-    #[test]
-    fn complete_attribute_on_enum() {
-        check(
-            r#"#[$0] enum Foo {}"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-                at derive(…)
-                at repr(…)
-                at non_exhaustive
-            "#]],
-        );
-    }
-
-    #[test]
-    fn complete_attribute_on_const() {
-        check(
-            r#"#[$0] const FOO: () = ();"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-            "#]],
-        );
-    }
-
-    #[test]
-    fn complete_attribute_on_static() {
-        check(
-            r#"#[$0] static FOO: () = ()"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-                at export_name = "…"
-                at link_name = "…"
-                at link_section = "…"
-                at global_allocator
-                at used
-            "#]],
-        );
-    }
-
-    #[test]
-    fn complete_attribute_on_trait() {
-        check(
-            r#"#[$0] trait Foo {}"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-                at must_use
-            "#]],
-        );
-    }
-
-    #[test]
-    fn complete_attribute_on_impl() {
-        check(
-            r#"#[$0] impl () {}"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-                at automatically_derived
-            "#]],
-        );
-        check(
-            r#"impl () {#![$0]}"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-            "#]],
-        );
-    }
-
-    #[test]
-    fn complete_attribute_on_extern_block() {
-        check(
-            r#"#[$0] extern {}"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-                at link
-            "#]],
-        );
-        check(
-            r#"extern {#![$0]}"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-                at link
-            "#]],
-        );
-    }
-
-    #[test]
-    fn complete_attribute_on_variant() {
-        check(
-            r#"enum Foo { #[$0] Bar }"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at non_exhaustive
-            "#]],
-        );
-    }
-
-    #[test]
-    fn complete_attribute_on_fn() {
-        check(
-            r#"#[$0] fn main() {}"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-                at deprecated
-                at doc = "…"
-                at doc(hidden)
-                at doc(alias = "…")
-                at must_use
-                at no_mangle
-                at export_name = "…"
-                at link_name = "…"
-                at link_section = "…"
-                at cold
-                at ignore = "…"
-                at inline
-                at must_use
-                at panic_handler
-                at proc_macro
-                at proc_macro_derive(…)
-                at proc_macro_attribute
-                at should_panic
-                at target_feature = "…"
-                at test
-                at track_caller
-            "#]],
-        );
-    }
-
-    #[test]
-    fn complete_attribute_on_expr() {
-        cov_mark::check!(no_keyword_completion_in_attr_of_expr);
-        check(
-            r#"fn main() { #[$0] foo() }"#,
-            expect![[r#"
-                at allow(…)
-                at cfg(…)
-                at cfg_attr(…)
-                at deny(…)
-                at forbid(…)
-                at warn(…)
-            "#]],
-        );
-    }
-
-    #[test]
-    fn complete_attribute_in_source_file_end() {
-        check(
-            r#"#[$0]"#,
-            expect![[r#"
-                at allow(…)
-                at automatically_derived
-                at cfg(…)
-                at cfg_attr(…)
-                at cold
-                at deny(…)
-                at deprecated
-                at derive(…)
-                at doc = "…"
-                at doc(alias = "…")
-                at doc(hidden)
-                at export_name = "…"
-                at forbid(…)
-                at global_allocator
-                at ignore = "…"
-                at inline
-                at link
-                at link_name = "…"
-                at link_section = "…"
-                at macro_export
-                at macro_use
-                at must_use
-                at no_mangle
-                at non_exhaustive
-                at panic_handler
-                at path = "…"
-                at proc_macro
-                at proc_macro_attribute
-                at proc_macro_derive(…)
-                at repr(…)
-                at should_panic
-                at target_feature = "…"
-                at test
-                at track_caller
-                at used
-                at warn(…)
-            "#]],
-        );
-    }
-
-    #[test]
-    fn test_cfg() {
-        check(
-            r#"#[cfg(target_endian = $0"#,
-            expect![[r#"
-                at little
-                at big
-"#]],
-        );
-    }
+        prev = next;
+    });
 }
diff --git a/crates/ide_completion/src/completions/attribute/derive.rs b/crates/ide_completion/src/completions/attribute/derive.rs
index 6fe41e0d6d1..f482d44646d 100644
--- a/crates/ide_completion/src/completions/attribute/derive.rs
+++ b/crates/ide_completion/src/completions/attribute/derive.rs
@@ -77,93 +77,3 @@ const DEFAULT_DERIVE_COMPLETIONS: &[DeriveDependencies] = &[
     DeriveDependencies { label: "Ord", dependencies: &["PartialOrd", "Eq", "PartialEq"] },
     DeriveDependencies { label: "PartialOrd", dependencies: &["PartialEq"] },
 ];
-
-#[cfg(test)]
-mod tests {
-    use expect_test::{expect, Expect};
-
-    use crate::tests::completion_list;
-
-    fn check(ra_fixture: &str, expect: Expect) {
-        let builtin_derives = r#"
-#[rustc_builtin_macro]
-pub macro Clone {}
-#[rustc_builtin_macro]
-pub macro Copy {}
-#[rustc_builtin_macro]
-pub macro Default {}
-#[rustc_builtin_macro]
-pub macro Debug {}
-#[rustc_builtin_macro]
-pub macro Hash {}
-#[rustc_builtin_macro]
-pub macro PartialEq {}
-#[rustc_builtin_macro]
-pub macro Eq {}
-#[rustc_builtin_macro]
-pub macro PartialOrd {}
-#[rustc_builtin_macro]
-pub macro Ord {}
-
-"#;
-        let actual = completion_list(&format!("{} {}", builtin_derives, ra_fixture));
-        expect.assert_eq(&actual);
-    }
-
-    #[test]
-    fn no_completion_for_incorrect_derive() {
-        check(r#"#[derive{$0)] struct Test;"#, expect![[]])
-    }
-
-    #[test]
-    fn empty_derive() {
-        check(
-            r#"#[derive($0)] struct Test;"#,
-            expect![[r#"
-            at PartialEq
-            at Default
-            at PartialEq, Eq
-            at PartialEq, Eq, PartialOrd, Ord
-            at Clone, Copy
-            at Debug
-            at Clone
-            at Hash
-            at PartialEq, PartialOrd
-        "#]],
-        );
-    }
-
-    #[test]
-    fn derive_with_input() {
-        check(
-            r#"#[derive(serde::Serialize, PartialEq, $0)] struct Test;"#,
-            expect![[r#"
-                at Default
-                at Eq
-                at Eq, PartialOrd, Ord
-                at Clone, Copy
-                at Debug
-                at Clone
-                at Hash
-                at PartialOrd
-            "#]],
-        )
-    }
-
-    #[test]
-    fn derive_with_input2() {
-        check(
-            r#"#[derive($0 serde::Serialize, PartialEq)] struct Test;"#,
-            expect![[r#"
-                at Default
-                at Eq
-                at Eq, PartialOrd, Ord
-                at Clone, Copy
-                at Debug
-                at Clone
-                at Hash
-                at PartialOrd
-            "#]],
-        )
-    }
-}
diff --git a/crates/ide_completion/src/completions/attribute/lint.rs b/crates/ide_completion/src/completions/attribute/lint.rs
index 1ddc389868b..bb7f4c8d3b3 100644
--- a/crates/ide_completion/src/completions/attribute/lint.rs
+++ b/crates/ide_completion/src/completions/attribute/lint.rs
@@ -30,44 +30,3 @@ pub(super) fn complete_lint(
         }
     }
 }
-
-#[cfg(test)]
-mod tests {
-    use crate::tests::check_edit;
-
-    #[test]
-    fn check_empty() {
-        check_edit(
-            "deprecated",
-            r#"#[allow($0)] struct Test;"#,
-            r#"#[allow(deprecated)] struct Test;"#,
-        )
-    }
-
-    #[test]
-    fn check_with_existing() {
-        check_edit(
-            "deprecated",
-            r#"#[allow(keyword_idents, $0)] struct Test;"#,
-            r#"#[allow(keyword_idents, deprecated)] struct Test;"#,
-        )
-    }
-
-    #[test]
-    fn check_qualified() {
-        check_edit(
-            "deprecated",
-            r#"#[allow(keyword_idents, $0)] struct Test;"#,
-            r#"#[allow(keyword_idents, deprecated)] struct Test;"#,
-        )
-    }
-
-    #[test]
-    fn check_feature() {
-        check_edit(
-            "box_syntax",
-            r#"#[feature(box_$0)] struct Test;"#,
-            r#"#[feature(box_syntax)] struct Test;"#,
-        )
-    }
-}
diff --git a/crates/ide_completion/src/completions/attribute/repr.rs b/crates/ide_completion/src/completions/attribute/repr.rs
index 92a262a43f9..3df04826d44 100644
--- a/crates/ide_completion/src/completions/attribute/repr.rs
+++ b/crates/ide_completion/src/completions/attribute/repr.rs
@@ -68,132 +68,3 @@ const REPR_COMPLETIONS: &[ReprCompletion] = &[
     attr("i28",    &["transparent", "u8", "u16", "u32", "u64", "u128", "usize", "i8", "i16", "i32", "i64", "isize"]),
     attr("isize",  &["transparent", "u8", "u16", "u32", "u64", "u128", "usize", "i8", "i16", "i32", "i64", "i128"]),
 ];
-
-#[cfg(test)]
-mod tests {
-    use expect_test::{expect, Expect};
-
-    use crate::tests::completion_list;
-
-    fn check(ra_fixture: &str, expect: Expect) {
-        let actual = completion_list(ra_fixture);
-        expect.assert_eq(&actual);
-    }
-
-    #[test]
-    fn no_completion_for_incorrect_repr() {
-        check(r#"#[repr{$0)] struct Test;"#, expect![[]])
-    }
-
-    #[test]
-    fn empty() {
-        check(
-            r#"#[repr($0)] struct Test;"#,
-            expect![[r#"
-                at align($0)
-                at packed
-                at transparent
-                at C
-                at u8
-                at u16
-                at u32
-                at u64
-                at u128
-                at usize
-                at i8
-                at i16
-                at i32
-                at i64
-                at i28
-                at isize
-            "#]],
-        );
-    }
-
-    #[test]
-    fn transparent() {
-        check(r#"#[repr(transparent, $0)] struct Test;"#, expect![[r#""#]]);
-    }
-
-    #[test]
-    fn align() {
-        check(
-            r#"#[repr(align(1), $0)] struct Test;"#,
-            expect![[r#"
-                at align($0)
-                at transparent
-                at C
-                at u8
-                at u16
-                at u32
-                at u64
-                at u128
-                at usize
-                at i8
-                at i16
-                at i32
-                at i64
-                at i28
-                at isize
-            "#]],
-        );
-    }
-
-    #[test]
-    fn packed() {
-        check(
-            r#"#[repr(packed, $0)] struct Test;"#,
-            expect![[r#"
-                at transparent
-                at C
-                at u8
-                at u16
-                at u32
-                at u64
-                at u128
-                at usize
-                at i8
-                at i16
-                at i32
-                at i64
-                at i28
-                at isize
-            "#]],
-        );
-    }
-
-    #[test]
-    fn c() {
-        check(
-            r#"#[repr(C, $0)] struct Test;"#,
-            expect![[r#"
-                at align($0)
-                at packed
-                at u8
-                at u16
-                at u32
-                at u64
-                at u128
-                at usize
-                at i8
-                at i16
-                at i32
-                at i64
-                at i28
-                at isize
-            "#]],
-        );
-    }
-
-    #[test]
-    fn prim() {
-        check(
-            r#"#[repr(usize, $0)] struct Test;"#,
-            expect![[r#"
-                at align($0)
-                at packed
-                at C
-            "#]],
-        );
-    }
-}
diff --git a/crates/ide_completion/src/completions/unqualified_path.rs b/crates/ide_completion/src/completions/unqualified_path.rs
index 3ee50957419..facfe005186 100644
--- a/crates/ide_completion/src/completions/unqualified_path.rs
+++ b/crates/ide_completion/src/completions/unqualified_path.rs
@@ -601,16 +601,4 @@ fn f() -> m::E { V$0 }
             "#]],
         )
     }
-
-    #[test]
-    fn dont_complete_attr() {
-        check(
-            r#"
-struct Foo;
-#[$0]
-fn f() {}
-"#,
-            expect![[""]],
-        )
-    }
 }
diff --git a/crates/ide_completion/src/tests.rs b/crates/ide_completion/src/tests.rs
index 454ef914c1b..20f7d360c6d 100644
--- a/crates/ide_completion/src/tests.rs
+++ b/crates/ide_completion/src/tests.rs
@@ -4,12 +4,14 @@
 //! `attributes` or `lifetimes` where the completed concept is a distinct thing.
 //! Notable examples for completions that are being tested in this module's submodule are paths.
 
+mod attribute;
 mod item_list;
-mod use_tree;
-mod items;
+mod item;
 mod pattern;
-mod type_pos;
 mod predicate;
+mod type_pos;
+mod use_tree;
+
 mod sourcegen;
 
 use std::mem;
diff --git a/crates/ide_completion/src/tests/attribute.rs b/crates/ide_completion/src/tests/attribute.rs
new file mode 100644
index 00000000000..012071a8173
--- /dev/null
+++ b/crates/ide_completion/src/tests/attribute.rs
@@ -0,0 +1,809 @@
+//! Completion tests for attributes.
+use expect_test::{expect, Expect};
+
+use crate::tests::{check_edit, completion_list};
+
+fn check(ra_fixture: &str, expect: Expect) {
+    let actual = completion_list(ra_fixture);
+    expect.assert_eq(&actual);
+}
+
+#[test]
+fn doesnt_complete_items() {
+    check(
+        r#"
+struct Foo;
+#[$0]
+use self as this;
+"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+        "#]],
+    )
+}
+
+#[test]
+fn inside_nested_attr() {
+    check(r#"#[cfg($0)]"#, expect![[]])
+}
+
+#[test]
+fn with_existing_attr() {
+    check(
+        r#"#[no_mangle] #[$0] mcall!();"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+        "#]],
+    )
+}
+
+#[test]
+fn attr_on_source_file() {
+    check(
+        r#"#![$0]"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+            at crate_name = ""
+            at feature(…)
+            at no_implicit_prelude
+            at no_main
+            at no_std
+            at recursion_limit = …
+            at type_length_limit = …
+            at windows_subsystem = "…"
+        "#]],
+    );
+}
+
+#[test]
+fn attr_on_module() {
+    check(
+        r#"#[$0] mod foo;"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+            at macro_use
+            at path = "…"
+        "#]],
+    );
+    check(
+        r#"mod foo {#![$0]}"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+            at no_implicit_prelude
+        "#]],
+    );
+}
+
+#[test]
+fn attr_on_macro_rules() {
+    check(
+        r#"#[$0] macro_rules! foo {}"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+            at macro_export
+            at macro_use
+        "#]],
+    );
+}
+
+#[test]
+fn attr_on_macro_def() {
+    check(
+        r#"#[$0] macro foo {}"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+        "#]],
+    );
+}
+
+#[test]
+fn attr_on_extern_crate() {
+    check(
+        r#"#[$0] extern crate foo;"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+            at macro_use
+        "#]],
+    );
+}
+
+#[test]
+fn attr_on_use() {
+    check(
+        r#"#[$0] use foo;"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+        "#]],
+    );
+}
+
+#[test]
+fn attr_on_type_alias() {
+    check(
+        r#"#[$0] type foo = ();"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+        "#]],
+    );
+}
+
+#[test]
+fn attr_on_struct() {
+    check(
+        r#"#[$0] struct Foo;"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+            at derive(…)
+            at repr(…)
+            at non_exhaustive
+        "#]],
+    );
+}
+
+#[test]
+fn attr_on_enum() {
+    check(
+        r#"#[$0] enum Foo {}"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+            at derive(…)
+            at repr(…)
+            at non_exhaustive
+        "#]],
+    );
+}
+
+#[test]
+fn attr_on_const() {
+    check(
+        r#"#[$0] const FOO: () = ();"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+        "#]],
+    );
+}
+
+#[test]
+fn attr_on_static() {
+    check(
+        r#"#[$0] static FOO: () = ()"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+            at export_name = "…"
+            at link_name = "…"
+            at link_section = "…"
+            at global_allocator
+            at used
+        "#]],
+    );
+}
+
+#[test]
+fn attr_on_trait() {
+    check(
+        r#"#[$0] trait Foo {}"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+            at must_use
+        "#]],
+    );
+}
+
+#[test]
+fn attr_on_impl() {
+    check(
+        r#"#[$0] impl () {}"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+            at automatically_derived
+        "#]],
+    );
+    check(
+        r#"impl () {#![$0]}"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+        "#]],
+    );
+}
+
+#[test]
+fn attr_on_extern_block() {
+    check(
+        r#"#[$0] extern {}"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+            at link
+        "#]],
+    );
+    check(
+        r#"extern {#![$0]}"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+            at link
+        "#]],
+    );
+}
+
+#[test]
+fn attr_on_variant() {
+    check(
+        r#"enum Foo { #[$0] Bar }"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at non_exhaustive
+        "#]],
+    );
+}
+
+#[test]
+fn attr_on_fn() {
+    check(
+        r#"#[$0] fn main() {}"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+            at deprecated
+            at doc = "…"
+            at doc(hidden)
+            at doc(alias = "…")
+            at must_use
+            at no_mangle
+            at export_name = "…"
+            at link_name = "…"
+            at link_section = "…"
+            at cold
+            at ignore = "…"
+            at inline
+            at must_use
+            at panic_handler
+            at proc_macro
+            at proc_macro_derive(…)
+            at proc_macro_attribute
+            at should_panic
+            at target_feature = "…"
+            at test
+            at track_caller
+        "#]],
+    );
+}
+
+#[test]
+fn attr_on_expr() {
+    cov_mark::check!(no_keyword_completion_in_attr_of_expr);
+    check(
+        r#"fn main() { #[$0] foo() }"#,
+        expect![[r#"
+            at allow(…)
+            at cfg(…)
+            at cfg_attr(…)
+            at deny(…)
+            at forbid(…)
+            at warn(…)
+        "#]],
+    );
+}
+
+#[test]
+fn attr_in_source_file_end() {
+    check(
+        r#"#[$0]"#,
+        expect![[r#"
+            at allow(…)
+            at automatically_derived
+            at cfg(…)
+            at cfg_attr(…)
+            at cold
+            at deny(…)
+            at deprecated
+            at derive(…)
+            at doc = "…"
+            at doc(alias = "…")
+            at doc(hidden)
+            at export_name = "…"
+            at forbid(…)
+            at global_allocator
+            at ignore = "…"
+            at inline
+            at link
+            at link_name = "…"
+            at link_section = "…"
+            at macro_export
+            at macro_use
+            at must_use
+            at no_mangle
+            at non_exhaustive
+            at panic_handler
+            at path = "…"
+            at proc_macro
+            at proc_macro_attribute
+            at proc_macro_derive(…)
+            at repr(…)
+            at should_panic
+            at target_feature = "…"
+            at test
+            at track_caller
+            at used
+            at warn(…)
+        "#]],
+    );
+}
+
+mod cfg {
+    use super::*;
+
+    #[test]
+    fn cfg_target_endian() {
+        check(
+            r#"#[cfg(target_endian = $0"#,
+            expect![[r#"
+            at little
+            at big
+"#]],
+        );
+    }
+}
+
+mod derive {
+    use super::*;
+
+    fn check_derive(ra_fixture: &str, expect: Expect) {
+        let builtin_derives = r#"
+    #[rustc_builtin_macro]
+    pub macro Clone {}
+    #[rustc_builtin_macro]
+    pub macro Copy {}
+    #[rustc_builtin_macro]
+    pub macro Default {}
+    #[rustc_builtin_macro]
+    pub macro Debug {}
+    #[rustc_builtin_macro]
+    pub macro Hash {}
+    #[rustc_builtin_macro]
+    pub macro PartialEq {}
+    #[rustc_builtin_macro]
+    pub macro Eq {}
+    #[rustc_builtin_macro]
+    pub macro PartialOrd {}
+    #[rustc_builtin_macro]
+    pub macro Ord {}
+
+    "#;
+        let actual = completion_list(&format!("{} {}", builtin_derives, ra_fixture));
+        expect.assert_eq(&actual);
+    }
+
+    #[test]
+    fn no_completion_for_incorrect_derive() {
+        check_derive(r#"#[derive{$0)] struct Test;"#, expect![[]])
+    }
+
+    #[test]
+    fn empty_derive() {
+        check_derive(
+            r#"#[derive($0)] struct Test;"#,
+            expect![[r#"
+        at PartialEq
+        at Default
+        at PartialEq, Eq
+        at PartialEq, Eq, PartialOrd, Ord
+        at Clone, Copy
+        at Debug
+        at Clone
+        at Hash
+        at PartialEq, PartialOrd
+    "#]],
+        );
+    }
+
+    #[test]
+    fn derive_with_input_before() {
+        check_derive(
+            r#"#[derive(serde::Serialize, PartialEq, $0)] struct Test;"#,
+            expect![[r#"
+            at Default
+            at Eq
+            at Eq, PartialOrd, Ord
+            at Clone, Copy
+            at Debug
+            at Clone
+            at Hash
+            at PartialOrd
+        "#]],
+        )
+    }
+
+    #[test]
+    fn derive_with_input_after() {
+        check_derive(
+            r#"#[derive($0 serde::Serialize, PartialEq)] struct Test;"#,
+            expect![[r#"
+            at Default
+            at Eq
+            at Eq, PartialOrd, Ord
+            at Clone, Copy
+            at Debug
+            at Clone
+            at Hash
+            at PartialOrd
+        "#]],
+        )
+    }
+}
+
+mod lint {
+    use super::*;
+
+    #[test]
+    fn lint_empty() {
+        check_edit(
+            "deprecated",
+            r#"#[allow($0)] struct Test;"#,
+            r#"#[allow(deprecated)] struct Test;"#,
+        )
+    }
+
+    #[test]
+    fn lint_with_existing() {
+        check_edit(
+            "deprecated",
+            r#"#[allow(keyword_idents, $0)] struct Test;"#,
+            r#"#[allow(keyword_idents, deprecated)] struct Test;"#,
+        )
+    }
+
+    #[test]
+    fn lint_qualified() {
+        check_edit(
+            "deprecated",
+            r#"#[allow(keyword_idents, $0)] struct Test;"#,
+            r#"#[allow(keyword_idents, deprecated)] struct Test;"#,
+        )
+    }
+
+    #[test]
+    fn lint_feature() {
+        check_edit(
+            "box_syntax",
+            r#"#[feature(box_$0)] struct Test;"#,
+            r#"#[feature(box_syntax)] struct Test;"#,
+        )
+    }
+}
+
+mod repr {
+    use super::*;
+
+    fn check_repr(ra_fixture: &str, expect: Expect) {
+        let actual = completion_list(ra_fixture);
+        expect.assert_eq(&actual);
+    }
+
+    #[test]
+    fn no_completion_for_incorrect_repr() {
+        check_repr(r#"#[repr{$0)] struct Test;"#, expect![[]])
+    }
+
+    #[test]
+    fn empty() {
+        check_repr(
+            r#"#[repr($0)] struct Test;"#,
+            expect![[r#"
+            at align($0)
+            at packed
+            at transparent
+            at C
+            at u8
+            at u16
+            at u32
+            at u64
+            at u128
+            at usize
+            at i8
+            at i16
+            at i32
+            at i64
+            at i28
+            at isize
+        "#]],
+        );
+    }
+
+    #[test]
+    fn transparent() {
+        check_repr(r#"#[repr(transparent, $0)] struct Test;"#, expect![[r#""#]]);
+    }
+
+    #[test]
+    fn align() {
+        check_repr(
+            r#"#[repr(align(1), $0)] struct Test;"#,
+            expect![[r#"
+            at align($0)
+            at transparent
+            at C
+            at u8
+            at u16
+            at u32
+            at u64
+            at u128
+            at usize
+            at i8
+            at i16
+            at i32
+            at i64
+            at i28
+            at isize
+        "#]],
+        );
+    }
+
+    #[test]
+    fn packed() {
+        check_repr(
+            r#"#[repr(packed, $0)] struct Test;"#,
+            expect![[r#"
+            at transparent
+            at C
+            at u8
+            at u16
+            at u32
+            at u64
+            at u128
+            at usize
+            at i8
+            at i16
+            at i32
+            at i64
+            at i28
+            at isize
+        "#]],
+        );
+    }
+
+    #[test]
+    fn c() {
+        check_repr(
+            r#"#[repr(C, $0)] struct Test;"#,
+            expect![[r#"
+            at align($0)
+            at packed
+            at u8
+            at u16
+            at u32
+            at u64
+            at u128
+            at usize
+            at i8
+            at i16
+            at i32
+            at i64
+            at i28
+            at isize
+        "#]],
+        );
+    }
+
+    #[test]
+    fn prim() {
+        check_repr(
+            r#"#[repr(usize, $0)] struct Test;"#,
+            expect![[r#"
+            at align($0)
+            at packed
+            at C
+        "#]],
+        );
+    }
+}
diff --git a/crates/ide_completion/src/tests/items.rs b/crates/ide_completion/src/tests/item.rs
similarity index 97%
rename from crates/ide_completion/src/tests/items.rs
rename to crates/ide_completion/src/tests/item.rs
index 7df200b486a..c9629c9d131 100644
--- a/crates/ide_completion/src/tests/items.rs
+++ b/crates/ide_completion/src/tests/item.rs
@@ -1,4 +1,4 @@
-//! Completions tests for item specifics overall.
+//! Completion tests for item specifics overall.
 //!
 //! Except for use items which are tested in [super::use_tree] and mod declarations with are tested
 //! in [crate::completions::mod_].
diff --git a/crates/ide_completion/src/tests/item_list.rs b/crates/ide_completion/src/tests/item_list.rs
index a76881f2097..129c9e51728 100644
--- a/crates/ide_completion/src/tests/item_list.rs
+++ b/crates/ide_completion/src/tests/item_list.rs
@@ -1,3 +1,4 @@
+//! Completion tests for item list position.
 use expect_test::{expect, Expect};
 
 use crate::tests::{completion_list, BASE_FIXTURE};
diff --git a/crates/ide_completion/src/tests/pattern.rs b/crates/ide_completion/src/tests/pattern.rs
index 9d70aca3c95..b5c2fef85e1 100644
--- a/crates/ide_completion/src/tests/pattern.rs
+++ b/crates/ide_completion/src/tests/pattern.rs
@@ -1,4 +1,4 @@
-//! Completions tests for pattern position.
+//! Completion tests for pattern position.
 use expect_test::{expect, Expect};
 
 use crate::tests::{completion_list, BASE_FIXTURE};
diff --git a/crates/ide_completion/src/tests/type_pos.rs b/crates/ide_completion/src/tests/type_pos.rs
index 40571f45b83..844e2095050 100644
--- a/crates/ide_completion/src/tests/type_pos.rs
+++ b/crates/ide_completion/src/tests/type_pos.rs
@@ -1,4 +1,4 @@
-//! Completions tests for type position.
+//! Completion tests for type position.
 use expect_test::{expect, Expect};
 
 use crate::tests::completion_list;
diff --git a/crates/ide_completion/src/tests/use_tree.rs b/crates/ide_completion/src/tests/use_tree.rs
index 71c618e6690..cb626046f44 100644
--- a/crates/ide_completion/src/tests/use_tree.rs
+++ b/crates/ide_completion/src/tests/use_tree.rs
@@ -1,4 +1,4 @@
-//! Completions tests for use trees.
+//! Completion tests for use trees.
 use expect_test::{expect, Expect};
 
 use crate::tests::completion_list;