summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMarc Mueller <30130371+cdce8p@users.noreply.github.com>2022-05-13 18:26:30 +0200
committerGitHub <noreply@github.com>2022-05-13 18:26:30 +0200
commit0c37d4c6875e0c1c81a51cd1a02ee270d50c53df (patch)
treede6890dba3567dc814af5bd18f2cab46b0995e8f
parent30042ece14dd32d6569e04c8984ece95f00ecc21 (diff)
downloadpylint-git-0c37d4c6875e0c1c81a51cd1a02ee270d50c53df.tar.gz
Update ranges for error messages (#6606)
* using-constant-test * missing-parentheses-for-call-in-test
-rw-r--r--ChangeLog3
-rw-r--r--doc/whatsnew/2.14.rst3
-rw-r--r--pylint/checkers/base/basic_checker.py6
-rw-r--r--tests/functional/ext/docparams/raise/missing_raises_doc_Google.txt2
-rw-r--r--tests/functional/m/missing/missing_parentheses_for_call_in_test.txt30
-rw-r--r--tests/functional/u/using_constant_test.txt52
6 files changed, 51 insertions, 45 deletions
diff --git a/ChangeLog b/ChangeLog
index f6521cd8b..fd6a991ea 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -318,6 +318,9 @@ Release date: TBA
Ref #6536
+* Update ranges for ``using-constant-test`` and ``missing-parentheses-for-call-in-test``
+ error messages.
+
..
Insert your changelog randomly, it will reduce merge conflicts
(Ie. not necessarily at the end)
diff --git a/doc/whatsnew/2.14.rst b/doc/whatsnew/2.14.rst
index 2a4cd3e7f..e780a7f47 100644
--- a/doc/whatsnew/2.14.rst
+++ b/doc/whatsnew/2.14.rst
@@ -263,6 +263,9 @@ Other Changes
Refs #6595
+* Update ranges for ``using-constant-test`` and ``missing-parentheses-for-call-in-test``
+ error messages.
+
Deprecations
============
diff --git a/pylint/checkers/base/basic_checker.py b/pylint/checkers/base/basic_checker.py
index e4526cac6..312e6c519 100644
--- a/pylint/checkers/base/basic_checker.py
+++ b/pylint/checkers/base/basic_checker.py
@@ -312,7 +312,7 @@ class BasicChecker(_BasicChecker):
inferred = utils.safe_infer(test)
if emit:
- self.add_message("using-constant-test", node=node)
+ self.add_message("using-constant-test", node=test)
elif isinstance(inferred, const_nodes):
# If the constant node is a FunctionDef or Lambda then
# it may be an illicit function call due to missing parentheses
@@ -329,12 +329,12 @@ class BasicChecker(_BasicChecker):
for inf_call in call_inferred:
if inf_call != astroid.Uninferable:
self.add_message(
- "missing-parentheses-for-call-in-test", node=node
+ "missing-parentheses-for-call-in-test", node=test
)
break
except astroid.InferenceError:
pass
- self.add_message("using-constant-test", node=node)
+ self.add_message("using-constant-test", node=test)
def visit_module(self, _: nodes.Module) -> None:
"""Check module name, docstring and required arguments."""
diff --git a/tests/functional/ext/docparams/raise/missing_raises_doc_Google.txt b/tests/functional/ext/docparams/raise/missing_raises_doc_Google.txt
index 6e98f05d9..0c6022c32 100644
--- a/tests/functional/ext/docparams/raise/missing_raises_doc_Google.txt
+++ b/tests/functional/ext/docparams/raise/missing_raises_doc_Google.txt
@@ -11,4 +11,4 @@ missing-raises-doc:148:4:148:18:Foo.foo_method:"""AttributeError"" not documente
unreachable:158:8:158:17:Foo.foo_method:Unreachable code:UNDEFINED
unreachable:180:8:180:17:Foo.foo_method:Unreachable code:UNDEFINED
missing-raises-doc:183:4:183:18:Foo.foo_method:"""AttributeError"" not documented as being raised":UNDEFINED
-using-constant-test:190:8:191:34:Foo.foo_method:Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:190:11:190:15:Foo.foo_method:Using a conditional statement with a constant value:UNDEFINED
diff --git a/tests/functional/m/missing/missing_parentheses_for_call_in_test.txt b/tests/functional/m/missing/missing_parentheses_for_call_in_test.txt
index 8c96f6e18..f782a9704 100644
--- a/tests/functional/m/missing/missing_parentheses_for_call_in_test.txt
+++ b/tests/functional/m/missing/missing_parentheses_for_call_in_test.txt
@@ -1,15 +1,15 @@
-missing-parentheses-for-call-in-test:29:0:30:8::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
-missing-parentheses-for-call-in-test:35:0:36:8::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
-missing-parentheses-for-call-in-test:43:0:44:8::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
-missing-parentheses-for-call-in-test:51:0:52:8::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
-missing-parentheses-for-call-in-test:56:0:57:8::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
-missing-parentheses-for-call-in-test:64:0:65:8::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
-missing-parentheses-for-call-in-test:70:11:70:38::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
-missing-parentheses-for-call-in-test:72:17:72:42::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
-missing-parentheses-for-call-in-test:73:18:73:46::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
-missing-parentheses-for-call-in-test:75:0:None:None::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
-missing-parentheses-for-call-in-test:76:0:None:None::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
-missing-parentheses-for-call-in-test:79:0:None:None::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
-missing-parentheses-for-call-in-test:80:0:None:None::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
-missing-parentheses-for-call-in-test:85:0:86:8::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
-missing-parentheses-for-call-in-test:91:0:92:8::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
+missing-parentheses-for-call-in-test:29:3:29:16::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
+missing-parentheses-for-call-in-test:35:3:35:19::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
+missing-parentheses-for-call-in-test:43:3:43:23::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
+missing-parentheses-for-call-in-test:51:5:51:25::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
+missing-parentheses-for-call-in-test:56:3:56:14::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
+missing-parentheses-for-call-in-test:64:3:64:17::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
+missing-parentheses-for-call-in-test:70:17:70:30::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
+missing-parentheses-for-call-in-test:72:23:72:34::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
+missing-parentheses-for-call-in-test:73:24:73:38::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
+missing-parentheses-for-call-in-test:75:26:75:39::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
+missing-parentheses-for-call-in-test:76:26:76:37::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
+missing-parentheses-for-call-in-test:79:26:79:40::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
+missing-parentheses-for-call-in-test:80:26:80:42::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
+missing-parentheses-for-call-in-test:85:3:85:26::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
+missing-parentheses-for-call-in-test:91:3:91:20::Using a conditional statement with potentially wrong function or method call due to missing parentheses:UNDEFINED
diff --git a/tests/functional/u/using_constant_test.txt b/tests/functional/u/using_constant_test.txt
index 3bf09b95b..e311cae55 100644
--- a/tests/functional/u/using_constant_test.txt
+++ b/tests/functional/u/using_constant_test.txt
@@ -1,27 +1,27 @@
-using-constant-test:22:0:23:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:26:0:27:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:29:0:30:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:32:0:33:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:35:0:36:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:38:0:39:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:41:0:42:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:44:0:45:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:47:0:48:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:50:0:51:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:53:0:54:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:56:0:57:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:59:0:60:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:62:0:63:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:65:0:66:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:68:0:69:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:73:0:74:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:76:3:76:16::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:80:0:None:None:test_comprehensions:Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:81:0:None:None:test_comprehensions:Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:82:0:None:None:test_comprehensions:Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:83:0:None:None:test_comprehensions:Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:84:0:None:None:test_comprehensions:Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:85:0:None:None:test_comprehensions:Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:89:0:90:8::Using a conditional statement with a constant value:UNDEFINED
-using-constant-test:93:0:94:8::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:22:3:22:14::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:26:3:26:31::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:29:3:29:15:<lambda>:Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:32:3:32:11::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:35:3:35:8::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:38:3:38:4::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:41:3:41:7::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:44:3:44:5::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:47:3:47:6::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:50:3:50:6::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:53:3:53:5::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:56:3:56:12::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:59:3:59:12::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:62:3:62:5::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:65:3:65:12::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:68:3:68:5::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:73:3:73:12::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:76:8:76:9::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:80:36:80:39:test_comprehensions:Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:81:36:81:37:test_comprehensions:Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:82:36:82:39:test_comprehensions:Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:83:36:83:37:test_comprehensions:Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:84:36:84:39:test_comprehensions:Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:85:39:85:42:test_comprehensions:Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:89:3:89:15::Using a conditional statement with a constant value:UNDEFINED
+using-constant-test:93:3:93:18::Using a conditional statement with a constant value:UNDEFINED
comparison-of-constants:117:3:117:8::"Comparison between constants: '2 < 3' has a constant value":HIGH