From 5952ba2fdbc5079472d7bb958bf1ebb7f3d8f128 Mon Sep 17 00:00:00 2001
From: Erwan Rouchet <rouchet@teklia.com>
Date: Thu, 27 Jan 2022 09:46:20 +0100
Subject: [PATCH] Use a public method to access registered responses
---
tests/test_git.py | 24 ++++++++++++------------
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/tests/test_git.py b/tests/test_git.py
index 11783007..1a9f4388 100644
--- a/tests/test_git.py
+++ b/tests/test_git.py
@@ -225,8 +225,8 @@ def test_merge_request(fake_responses, fake_gitlab_helper_factory, mocker):
)
# the fake_responses are defined in the same order as they are expected to be called
- expected_http_methods = [r.method for r in fake_responses._matches]
- expected_urls = [r.url for r in fake_responses._matches]
+ expected_http_methods = [r.method for r in fake_responses.registered()]
+ expected_urls = [r.url for r in fake_responses.registered()]
gitlab_helper = fake_gitlab_helper_factory()
gitlab_helper._wait_for_rebase_to_finish = mocker.MagicMock()
@@ -261,8 +261,8 @@ def test_merge_request_fail(fake_responses, fake_gitlab_helper_factory, mocker):
)
# the fake_responses are defined in the same order as they are expected to be called
- expected_http_methods = [r.method for r in fake_responses._matches]
- expected_urls = [r.url for r in fake_responses._matches]
+ expected_http_methods = [r.method for r in fake_responses.registered()]
+ expected_urls = [r.url for r in fake_responses.registered()]
gitlab_helper = fake_gitlab_helper_factory()
gitlab_helper._wait_for_rebase_to_finish = mocker.MagicMock()
@@ -349,8 +349,8 @@ def test_merge_request__success_after_errors(
)
# the fake_responses are defined in the same order as they are expected to be called
- expected_http_methods = [r.method for r in fake_responses._matches]
- expected_urls = [r.url for r in fake_responses._matches]
+ expected_http_methods = [r.method for r in fake_responses.registered()]
+ expected_urls = [r.url for r in fake_responses.registered()]
gitlab_helper = fake_gitlab_helper_factory()
@@ -387,8 +387,8 @@ def test_merge_request__fail_bad_request(fake_responses, fake_gitlab_helper_fact
)
# the fake_responses are defined in the same order as they are expected to be called
- expected_http_methods = [r.method for r in fake_responses._matches]
- expected_urls = [r.url for r in fake_responses._matches]
+ expected_http_methods = [r.method for r in fake_responses.registered()]
+ expected_urls = [r.url for r in fake_responses.registered()]
gitlab_helper = fake_gitlab_helper_factory()
@@ -413,8 +413,8 @@ def test_create_merge_request__no_retry_5xx_error(
)
# the fake_responses are defined in the same order as they are expected to be called
- expected_http_methods = [r.method for r in fake_responses._matches]
- expected_urls = [r.url for r in fake_responses._matches]
+ expected_http_methods = [r.method for r in fake_responses.registered()]
+ expected_urls = [r.url for r in fake_responses.registered()]
gitlab_helper = fake_gitlab_helper_factory()
@@ -458,8 +458,8 @@ def test_create_merge_request__retry_5xx_error(
)
# the fake_responses are defined in the same order as they are expected to be called
- expected_http_methods = [r.method for r in fake_responses._matches]
- expected_urls = [r.url for r in fake_responses._matches]
+ expected_http_methods = [r.method for r in fake_responses.registered()]
+ expected_urls = [r.url for r in fake_responses.registered()]
gitlab_helper = fake_gitlab_helper_factory()
--
GitLab