diff options
| author | Liu Dingming <liudingming@bytedance.com> | 2023-02-22 21:30:42 +0800 |
|---|---|---|
| committer | Liu Dingming <liudingming@bytedance.com> | 2023-02-22 21:31:21 +0800 |
| commit | 72e773f6a8bca1a4aaec6866ed995eaa0fd05cb2 (patch) | |
| tree | 7063f971641533137237d9846dd6fe8f9fc61245 | |
| parent | 8d08917d3a964f8498e6cc47f0a34770032f5e61 (diff) | |
| download | rust-72e773f6a8bca1a4aaec6866ed995eaa0fd05cb2.tar.gz rust-72e773f6a8bca1a4aaec6866ed995eaa0fd05cb2.zip | |
Fix test function checker in [unwrap_used], [expect_used]
| -rw-r--r-- | clippy_lints/src/methods/expect_used.rs | 4 | ||||
| -rw-r--r-- | clippy_lints/src/methods/unwrap_used.rs | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/clippy_lints/src/methods/expect_used.rs b/clippy_lints/src/methods/expect_used.rs index cce8f797e98..9e370de0a05 100644 --- a/clippy_lints/src/methods/expect_used.rs +++ b/clippy_lints/src/methods/expect_used.rs @@ -1,5 +1,5 @@ use clippy_utils::diagnostics::span_lint_and_help; -use clippy_utils::is_in_cfg_test; +use clippy_utils::{is_in_cfg_test, is_in_test_function}; use clippy_utils::ty::is_type_diagnostic_item; use rustc_hir as hir; use rustc_lint::LateContext; @@ -27,7 +27,7 @@ pub(super) fn check( let method = if is_err { "expect_err" } else { "expect" }; - if allow_expect_in_tests && is_in_cfg_test(cx.tcx, expr.hir_id) { + if allow_expect_in_tests && (is_in_test_function(cx.tcx, expr.hir_id) || is_in_cfg_test(cx.tcx, expr.hir_id)) { return; } diff --git a/clippy_lints/src/methods/unwrap_used.rs b/clippy_lints/src/methods/unwrap_used.rs index 90983f249cd..5e4c3daee64 100644 --- a/clippy_lints/src/methods/unwrap_used.rs +++ b/clippy_lints/src/methods/unwrap_used.rs @@ -1,6 +1,6 @@ use clippy_utils::diagnostics::span_lint_and_help; use clippy_utils::ty::is_type_diagnostic_item; -use clippy_utils::{is_in_cfg_test, is_lint_allowed}; +use clippy_utils::{is_in_cfg_test, is_in_test_function, is_lint_allowed}; use rustc_hir as hir; use rustc_lint::LateContext; use rustc_span::sym; @@ -27,7 +27,7 @@ pub(super) fn check( let method_suffix = if is_err { "_err" } else { "" }; - if allow_unwrap_in_tests && is_in_cfg_test(cx.tcx, expr.hir_id) { + if allow_unwrap_in_tests && (is_in_test_function(cx.tcx, expr.hir_id) || is_in_cfg_test(cx.tcx, expr.hir_id)) { return; } |
