language-servers/ruff
1
0
Fork 0
mirror of https://github.com/astral-sh/ruff.git synced 2025-09-29 21:35:58 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

linter: Enable test-rules for test build (#11201)

Browse source
This commit is contained in:
Micha Reiser 2024-04-30 08:06:47 +02:00 • committed by GitHub
parent c391c8b6cb
commit 5561d445d7
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
5 changed files with 20 additions and 20 deletions
Download patch file Download diff file Expand all files Collapse all files

26
crates/ruff_linter/src/codes.rs
View file

@ -971,32 +971,32 @@ pub fn code_to_rule(linter: Linter, code: &str) -> Option<(RuleGroup, Rule)> {
(Ruff, "100") => (RuleGroup::Stable, rules::ruff::rules::UnusedNOQA), (Ruff, "100") => (RuleGroup::Stable, rules::ruff::rules::UnusedNOQA),
(Ruff, "101") => (RuleGroup::Preview, rules::ruff::rules::RedirectedNOQA), (Ruff, "101") => (RuleGroup::Preview, rules::ruff::rules::RedirectedNOQA),
(Ruff, "200") => (RuleGroup::Stable, rules::ruff::rules::InvalidPyprojectToml), (Ruff, "200") => (RuleGroup::Stable, rules::ruff::rules::InvalidPyprojectToml),
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
(Ruff, "900") => (RuleGroup::Stable, rules::ruff::rules::StableTestRule), (Ruff, "900") => (RuleGroup::Stable, rules::ruff::rules::StableTestRule),
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
(Ruff, "901") => (RuleGroup::Stable, rules::ruff::rules::StableTestRuleSafeFix), (Ruff, "901") => (RuleGroup::Stable, rules::ruff::rules::StableTestRuleSafeFix),
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
(Ruff, "902") => (RuleGroup::Stable, rules::ruff::rules::StableTestRuleUnsafeFix), (Ruff, "902") => (RuleGroup::Stable, rules::ruff::rules::StableTestRuleUnsafeFix),
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
(Ruff, "903") => (RuleGroup::Stable, rules::ruff::rules::StableTestRuleDisplayOnlyFix), (Ruff, "903") => (RuleGroup::Stable, rules::ruff::rules::StableTestRuleDisplayOnlyFix),
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
(Ruff, "911") => (RuleGroup::Preview, rules::ruff::rules::PreviewTestRule), (Ruff, "911") => (RuleGroup::Preview, rules::ruff::rules::PreviewTestRule),
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
#[allow(deprecated)] #[allow(deprecated)]
(Ruff, "912") => (RuleGroup::Nursery, rules::ruff::rules::NurseryTestRule), (Ruff, "912") => (RuleGroup::Nursery, rules::ruff::rules::NurseryTestRule),
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
(Ruff, "920") => (RuleGroup::Deprecated, rules::ruff::rules::DeprecatedTestRule), (Ruff, "920") => (RuleGroup::Deprecated, rules::ruff::rules::DeprecatedTestRule),
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
(Ruff, "921") => (RuleGroup::Deprecated, rules::ruff::rules::AnotherDeprecatedTestRule), (Ruff, "921") => (RuleGroup::Deprecated, rules::ruff::rules::AnotherDeprecatedTestRule),
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
(Ruff, "930") => (RuleGroup::Removed, rules::ruff::rules::RemovedTestRule), (Ruff, "930") => (RuleGroup::Removed, rules::ruff::rules::RemovedTestRule),
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
(Ruff, "931") => (RuleGroup::Removed, rules::ruff::rules::AnotherRemovedTestRule), (Ruff, "931") => (RuleGroup::Removed, rules::ruff::rules::AnotherRemovedTestRule),
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
(Ruff, "940") => (RuleGroup::Removed, rules::ruff::rules::RedirectedFromTestRule), (Ruff, "940") => (RuleGroup::Removed, rules::ruff::rules::RedirectedFromTestRule),
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
(Ruff, "950") => (RuleGroup::Stable, rules::ruff::rules::RedirectedToTestRule), (Ruff, "950") => (RuleGroup::Stable, rules::ruff::rules::RedirectedToTestRule),
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
(Ruff, "960") => (RuleGroup::Removed, rules::ruff::rules::RedirectedFromPrefixTestRule), (Ruff, "960") => (RuleGroup::Removed, rules::ruff::rules::RedirectedFromPrefixTestRule),

4
crates/ruff_linter/src/linter.rs
View file

@ -33,7 +33,7 @@ use crate::message::Message;
use crate::noqa::add_noqa; use crate::noqa::add_noqa;
use crate::registry::{AsRule, Rule, RuleSet}; use crate::registry::{AsRule, Rule, RuleSet};
use crate::rules::pycodestyle; use crate::rules::pycodestyle;
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
use crate::rules::ruff::rules::test_rules::{self, TestRule, TEST_RULES}; use crate::rules::ruff::rules::test_rules::{self, TestRule, TEST_RULES};
use crate::settings::types::UnsafeFixes; use crate::settings::types::UnsafeFixes;
use crate::settings::{flags, LinterSettings}; use crate::settings::{flags, LinterSettings};
@ -218,7 +218,7 @@ pub fn check_path(
} }
// Raise violations for internal test rules // Raise violations for internal test rules
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
{ {
for test_rule in TEST_RULES { for test_rule in TEST_RULES {
if !settings.rules.enabled(*test_rule) { if !settings.rules.enabled(*test_rule) {

4
crates/ruff_linter/src/rule_redirects.rs
View file

@ -104,10 +104,10 @@ static REDIRECTS: Lazy<HashMap<&'static str, &'static str>> = Lazy::new(|| {
("PGH001", "S307"), ("PGH001", "S307"),
("PGH002", "G010"), ("PGH002", "G010"),
// Test redirect by exact code // Test redirect by exact code
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
("RUF940", "RUF950"), ("RUF940", "RUF950"),
// Test redirect by prefix // Test redirect by prefix
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
("RUF96", "RUF95"), ("RUF96", "RUF95"),
// See: https://github.com/astral-sh/ruff/issues/10791 // See: https://github.com/astral-sh/ruff/issues/10791
("PLW0117", "PLW0177"), ("PLW0117", "PLW0177"),

2
crates/ruff_linter/src/rule_selector.rs
View file

@ -323,7 +323,7 @@ mod schema {
}) })
.filter(|_rule| { .filter(|_rule| {
// Filter out all test-only rules // Filter out all test-only rules
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
#[allow(clippy::used_underscore_binding)] #[allow(clippy::used_underscore_binding)]
if _rule.starts_with("RUF9") { if _rule.starts_with("RUF9") {
return false; return false;

4
crates/ruff_linter/src/rules/ruff/rules/mod.rs
View file

@ -21,7 +21,7 @@ pub(crate) use redirected_noqa::*;
pub(crate) use sort_dunder_all::*; pub(crate) use sort_dunder_all::*;
pub(crate) use sort_dunder_slots::*; pub(crate) use sort_dunder_slots::*;
pub(crate) use static_key_dict_comprehension::*; pub(crate) use static_key_dict_comprehension::*;
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
pub(crate) use test_rules::*; pub(crate) use test_rules::*;
pub(crate) use unnecessary_dict_comprehension_for_iterable::*; pub(crate) use unnecessary_dict_comprehension_for_iterable::*;
pub(crate) use unnecessary_iterable_allocation_for_first_element::*; pub(crate) use unnecessary_iterable_allocation_for_first_element::*;
@ -56,7 +56,7 @@ mod sort_dunder_all;
mod sort_dunder_slots; mod sort_dunder_slots;
mod static_key_dict_comprehension; mod static_key_dict_comprehension;
mod suppression_comment_visitor; mod suppression_comment_visitor;
#[cfg(feature = "test-rules")] #[cfg(any(feature = "test-rules", test))]
pub(crate) mod test_rules; pub(crate) mod test_rules;
mod unnecessary_dict_comprehension_for_iterable; mod unnecessary_dict_comprehension_for_iterable;
mod unnecessary_iterable_allocation_for_first_element; mod unnecessary_iterable_allocation_for_first_element;