diff options
Diffstat (limited to 'crates/ra_assists/src/handlers/fill_match_arms.rs')
-rw-r--r-- | crates/ra_assists/src/handlers/fill_match_arms.rs | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/crates/ra_assists/src/handlers/fill_match_arms.rs b/crates/ra_assists/src/handlers/fill_match_arms.rs index b57ff75ae..bbdcdc626 100644 --- a/crates/ra_assists/src/handlers/fill_match_arms.rs +++ b/crates/ra_assists/src/handlers/fill_match_arms.rs | |||
@@ -4,7 +4,7 @@ use hir::{Adt, HasSource, ModuleDef, Semantics}; | |||
4 | use itertools::Itertools; | 4 | use itertools::Itertools; |
5 | use ra_ide_db::RootDatabase; | 5 | use ra_ide_db::RootDatabase; |
6 | use ra_syntax::ast::{self, make, AstNode, MatchArm, NameOwner, Pat}; | 6 | use ra_syntax::ast::{self, make, AstNode, MatchArm, NameOwner, Pat}; |
7 | use test_utils::tested_by; | 7 | use test_utils::mark; |
8 | 8 | ||
9 | use crate::{utils::FamousDefs, AssistContext, AssistId, Assists}; | 9 | use crate::{utils::FamousDefs, AssistContext, AssistId, Assists}; |
10 | 10 | ||
@@ -58,7 +58,7 @@ pub(crate) fn fill_match_arms(acc: &mut Assists, ctx: &AssistContext) -> Option< | |||
58 | .collect::<Vec<_>>(); | 58 | .collect::<Vec<_>>(); |
59 | if Some(enum_def) == FamousDefs(&ctx.sema, module.krate()).core_option_Option() { | 59 | if Some(enum_def) == FamousDefs(&ctx.sema, module.krate()).core_option_Option() { |
60 | // Match `Some` variant first. | 60 | // Match `Some` variant first. |
61 | tested_by!(option_order); | 61 | mark::hit!(option_order); |
62 | variants.reverse() | 62 | variants.reverse() |
63 | } | 63 | } |
64 | variants | 64 | variants |
@@ -174,13 +174,14 @@ fn build_pat(db: &RootDatabase, module: hir::Module, var: hir::EnumVariant) -> O | |||
174 | 174 | ||
175 | #[cfg(test)] | 175 | #[cfg(test)] |
176 | mod tests { | 176 | mod tests { |
177 | use test_utils::mark; | ||
178 | |||
177 | use crate::{ | 179 | use crate::{ |
178 | tests::{check_assist, check_assist_not_applicable, check_assist_target}, | 180 | tests::{check_assist, check_assist_not_applicable, check_assist_target}, |
179 | utils::FamousDefs, | 181 | utils::FamousDefs, |
180 | }; | 182 | }; |
181 | 183 | ||
182 | use super::fill_match_arms; | 184 | use super::fill_match_arms; |
183 | use test_utils::covers; | ||
184 | 185 | ||
185 | #[test] | 186 | #[test] |
186 | fn all_match_arms_provided() { | 187 | fn all_match_arms_provided() { |
@@ -750,7 +751,7 @@ mod tests { | |||
750 | 751 | ||
751 | #[test] | 752 | #[test] |
752 | fn option_order() { | 753 | fn option_order() { |
753 | covers!(option_order); | 754 | mark::check!(option_order); |
754 | let before = r#" | 755 | let before = r#" |
755 | fn foo(opt: Option<i32>) { | 756 | fn foo(opt: Option<i32>) { |
756 | match opt<|> { | 757 | match opt<|> { |