From 06fab2dd03a9765d231d0b11e6212c89cc5e6255 Mon Sep 17 00:00:00 2001 From: Carlo Costino Date: Thu, 7 Sep 2023 17:22:39 -0400 Subject: [PATCH] Ignoring two specific warnings about wrong values type The test makes use of list item unpacking and expects the parameter data structure to allow for that. It does not make sense to modify the code in this case just to make test format linting pass. Signed-off-by: Carlo Costino --- tests/app/utils/test_user.py | 32 ++++++++++++++++---------------- 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/tests/app/utils/test_user.py b/tests/app/utils/test_user.py index d1fd1b844..788813fc3 100644 --- a/tests/app/utils/test_user.py +++ b/tests/app/utils/test_user.py @@ -5,27 +5,27 @@ from werkzeug.exceptions import Forbidden from app.utils.user import user_has_permissions -@pytest.mark.parametrize( +@pytest.mark.parametrize( # noqa: PT007 # Ignoring wrong values type because of the list unpacking in the test. "permissions", - [ - ( + ( + [ # Route has one of the permissions which the user has "manage_service" - ), - ( + ], + [ # Route has more than one of the permissions which the user has "manage_templates", "manage_service", - ), - ( + ], + [ # Route has one of the permissions which the user has, and one they do not "manage_service", "send_messages", - ), - ( + ], + [ # Route has no specific permissions required - ), - ], + ], + ), ) def test_permissions( client_request, @@ -48,14 +48,14 @@ def test_permissions( index() -@pytest.mark.parametrize( +@pytest.mark.parametrize( # noqa: PT007 # Ignoring wrong values type because of the list unpacking in the test. "permissions", - [ - ( + ( + [ # Route has a permission which the user doesn’t have "send_messages" - ), - ], + ], + ), ) def test_permissions_forbidden( client_request,