From 89c82e57e4d93435d6c931c080c0a8af5e9675a2 Mon Sep 17 00:00:00 2001 From: Robert Knight Date: Tue, 11 Feb 2025 11:05:06 +0000 Subject: [PATCH] Prefer `mock.ANY` over `Any.string()` See https://github.com/hypothesis/h/pull/9336#discussion_r1950529976 --- tests/unit/h/views/admin/users_test.py | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/tests/unit/h/views/admin/users_test.py b/tests/unit/h/views/admin/users_test.py index d5c5ea33e92..610b44ec889 100644 --- a/tests/unit/h/views/admin/users_test.py +++ b/tests/unit/h/views/admin/users_test.py @@ -2,7 +2,6 @@ from unittest import mock import pytest -from h_matchers import Any from pyramid import httpexceptions from h.models import Annotation @@ -36,7 +35,7 @@ def test_users_index(pyramid_request): "username": None, "authority": None, "user": None, - "username_pattern": Any.string(), + "username_pattern": mock.ANY, "user_meta": {}, "format_date": format_date, } @@ -91,7 +90,7 @@ def test_users_index_no_user_found(models, pyramid_request): "authority": "foo.org", "user": None, "user_meta": {}, - "username_pattern": Any.string(), + "username_pattern": mock.ANY, "format_date": format_date, } @@ -110,7 +109,7 @@ def test_users_index_user_marked_as_deleted(models, pyramid_request, factories): "authority": "foo.org", "user": None, "user_meta": {}, - "username_pattern": Any.string(), + "username_pattern": mock.ANY, "format_date": format_date, } @@ -132,7 +131,7 @@ def test_users_index_user_found( "authority": "foo.org", "user": user, "user_meta": {"annotations_count": 8}, - "username_pattern": Any.string(), + "username_pattern": mock.ANY, "format_date": format_date, }