[pre-commit.ci] pre-commit autoupdate (#36)

* [pre-commit.ci] pre-commit autoupdate

updates:
- [github.com/psf/black: 22.1.0 → 22.3.0](psf/black@22.1.0...22.3.0)

* Update CHANGELOG + update hook version

* Fix failing tests

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Damien Nguyen <[email protected]>

Author: pre-commit-ci[bot]

.pre-commit-config.yaml

@@ -3,7 +3,7 @@ ci:
 
 repos:
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.1.0
+    rev: v4.2.0
     hooks:
       - id: check-added-large-files
       - id: check-case-conflict
@@ -38,7 +38,7 @@ repos:
         name: isort (python)
 
   - repo: https://github.com/psf/black
-    rev: 22.1.0
+    rev: 22.3.0
     hooks:
       - id: black
         language_version: python3

CHANGELOG.md

@@ -10,14 +10,15 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Fixed
 
 -   Fixed uses of of `pylint.testutils.MessageTest` instead of `pylint.testutils.Message` for Pylint >= 2.12
+-   Fixed failing tests due to missing `ignore_position` argument to `assertAddsMessages()`
 
 ### Repository
 
--   Update `black` hook to v22.1.0
+-   Update `black` hook to v22.3.0
 -   Update `check-manifest` hook to v0.48
 -   Update `isort` hook to v5.10.1
 -   Update `flake8` hook to v4.0.1
--   Update `pre-commit/pre-commit-hooks` to v4.1.0
+-   Update `pre-commit/pre-commit-hooks` to v4.2.0
 -   Update `Lucas-C/pre-commit-hooks` hook to v1.1.13
 -   Update `dangoslen/changelog-enforcer` GitHub action to v3
 -   Update `thomaseizinger/create-pull-request` GitHub action to v1.2.2

tests/assert_test.py

@@ -49,5 +49,5 @@ def test_assert_ok(self):
     )
     def test_assert_not_ok(self, s):
         node = astroid.extract_node(s + ' #@')
-        with self.assertAddsMessages(MessageTest(msg_id='avoid-assert', node=node)):
+        with self.assertAddsMessages(MessageTest(msg_id='avoid-assert', node=node), ignore_position=True):
             self.checker.visit_assert(node)

tests/builtin_open_test.py

@@ -81,7 +81,7 @@ def test_builtin_open_call(self, s, os_open_mode):
         node = astroid.extract_node(s + ' #@')
         self.checker.set_os_open_allowed_modes(str(os_open_mode))
         if os_open_mode:
-            with self.assertAddsMessages(MessageTest(msg_id='replace-builtin-open', node=node)):
+            with self.assertAddsMessages(MessageTest(msg_id='replace-builtin-open', node=node), ignore_position=True):
                 self.checker.visit_call(node)
         else:
             with self.assertNoMessages():
@@ -93,7 +93,7 @@ def test_builtin_open_with(self, s, os_open_mode):
         node = astroid.extract_node(s + ' #@')
         self.checker.set_os_open_allowed_modes(str(os_open_mode))
         if os_open_mode:
-            with self.assertAddsMessages(MessageTest(msg_id='replace-builtin-open', node=node)):
+            with self.assertAddsMessages(MessageTest(msg_id='replace-builtin-open', node=node), ignore_position=True):
                 self.checker.visit_with(node)
         else:
             with self.assertNoMessages():

tests/eval_exec_test.py

@@ -51,5 +51,5 @@ def test_eval_exec_ok(self):
     )
     def test_eval_exec_call(self, s):
         node = astroid.extract_node(s + ' #@')
-        with self.assertAddsMessages(MessageTest(msg_id='avoid-eval-exec', node=node)):
+        with self.assertAddsMessages(MessageTest(msg_id='avoid-eval-exec', node=node), ignore_position=True):
             self.checker.visit_call(node)

tests/jsonpickle_decode_test.py

@@ -48,5 +48,5 @@ def test_jsonpickle_decode_ok(self):
     )
     def test_jsonpickle_decode_not_ok(self, s):
         node = astroid.extract_node(s + ' #@')
-        with self.assertAddsMessages(MessageTest(msg_id='avoid-jsonpickle-decode', node=node)):
+        with self.assertAddsMessages(MessageTest(msg_id='avoid-jsonpickle-decode', node=node), ignore_position=True):
             self.checker.visit_call(node)

tests/marshal_load_test.py

@@ -52,7 +52,7 @@ def test_marshal_load_ok(self):
     )
     def test_marshal_load_not_ok(self, s):
         node = astroid.extract_node(s + ' #@')
-        with self.assertAddsMessages(MessageTest(msg_id='avoid-marshal-load', node=node)):
+        with self.assertAddsMessages(MessageTest(msg_id='avoid-marshal-load', node=node), ignore_position=True):
             self.checker.visit_call(node)
 
     @pytest.mark.parametrize(
@@ -65,5 +65,5 @@ def test_marshal_load_not_ok(self, s):
     )
     def test_marshal_open_importfrom(self, s):
         node = astroid.extract_node(s + ' #@')
-        with self.assertAddsMessages(MessageTest(msg_id='avoid-marshal-load', node=node)):
+        with self.assertAddsMessages(MessageTest(msg_id='avoid-marshal-load', node=node), ignore_position=True):
             self.checker.visit_importfrom(node)

tests/os_chmod_test.py

@@ -130,7 +130,9 @@ def test_chmod(self, mocker, platform, enabled_platform, fname, arg_type, forbid
         print(code)
         node = astroid.extract_node(code)
         if enabled_platform and forbidden != 'S_IRGRP':
-            with self.assertAddsMessages(MessageTest(msg_id='os-chmod-unsafe-permissions', node=node)):
+            with self.assertAddsMessages(
+                MessageTest(msg_id='os-chmod-unsafe-permissions', node=node), ignore_position=True
+            ):
                 self.checker.visit_call(node)
         else:
             with self.assertNoMessages():

tests/os_mkdir_mkfifo_mknod_test.py

@@ -127,7 +127,8 @@ def test_os_function_call(self, mocker, platform, enabled_platform, function, op
                     msg_id=f'os-{function}-unsafe-permissions',
                     node=node,
                     args=(getattr(self.checker, f'_os_{function}_msg_arg'),),
-                )
+                ),
+                ignore_position=True,
             ):
                 self.checker.visit_call(node)
         else:

tests/os_open_test.py

@@ -135,7 +135,8 @@ def test_os_open_call_default_modes(self, mode, arg, expected_warning):
             with self.assertAddsMessages(
                 MessageTest(
                     msg_id='os-open-unsafe-permissions', node=node, args=(getattr(self.checker, '_os_open_msg_arg'),)
-                )
+                ),
+                ignore_position=True,
             ):
                 self.checker.visit_call(node)
         else:
@@ -168,7 +169,8 @@ def test_os_open_call(self, mode, call_mode, arg, expected_warning):
             with self.assertAddsMessages(
                 MessageTest(
                     msg_id='os-open-unsafe-permissions', node=node, args=(getattr(self.checker, '_os_open_msg_arg'),)
-                )
+                ),
+                ignore_position=True,
             ):
                 self.checker.visit_call(node)
         else:
@@ -197,7 +199,9 @@ def test_os_open_with_default_modes(self, mode, call_mode, arg, expected_warning
         node = astroid.extract_node(call_mode.format(mode))
         self.checker.set_os_open_allowed_modes(arg)
         if expected_warning:
-            with self.assertAddsMessages(MessageTest(msg_id='os-open-unsafe-permissions', node=node)):
+            with self.assertAddsMessages(
+                MessageTest(msg_id='os-open-unsafe-permissions', node=node), ignore_position=True
+            ):
                 self.checker.visit_with(node)
         else:
             with self.assertNoMessages():
@@ -217,7 +221,9 @@ def test_os_open_with(self, mode, arg, expected_warning):
         node = astroid.extract_node(f'with os.open("file.txt", os.O_WRONLY, 0o{mode:o}) as fd: fd.read() #@')
         self.checker.set_os_open_allowed_modes(arg)
         if expected_warning and mode != 0o755:
-            with self.assertAddsMessages(MessageTest(msg_id='os-open-unsafe-permissions', node=node)):
+            with self.assertAddsMessages(
+                MessageTest(msg_id='os-open-unsafe-permissions', node=node), ignore_position=True
+            ):
                 self.checker.visit_with(node)
         else:
             with self.assertNoMessages():