diff options
Diffstat (limited to 'debian/patches/improve-unknown-interaction-mock.patch')
-rw-r--r-- | debian/patches/improve-unknown-interaction-mock.patch | 56 |
1 files changed, 0 insertions, 56 deletions
diff --git a/debian/patches/improve-unknown-interaction-mock.patch b/debian/patches/improve-unknown-interaction-mock.patch deleted file mode 100644 index b09b3a2..0000000 --- a/debian/patches/improve-unknown-interaction-mock.patch +++ /dev/null @@ -1,56 +0,0 @@ -From: Colin Watson <cjwatson@debian.org> -Date: Fri, 9 Feb 2018 21:54:21 +0000 -Subject: Improve mock setup for 407-then-unknown test - -`test_407_then_unknown_interaction_methods` causes the client to fetch -the possible methods supported by the discharger (because it's told that -it only supports a non-window method). This is currently unmocked, -which causes the client to actually contact `http://example.com/visit`. -This fails in Launchpad builds because they run with a restrictive -network setup that doesn't even expose DNS lookups for non-permitted -hosts. - -There isn't really a good way to simulate this without setting up a -similar stunt DNS server (though perhaps installing an -`httmock.all_requests` fallback mock that raises an exception would be a -good idea?), but this seems to be the only failure at the moment. - -Forwarded: https://github.com/go-macaroon-bakery/py-macaroon-bakery/pull/45 -Last-Update: 2018-02-09 - -Patch-Name: improve-unknown-interaction-mock.patch ---- - macaroonbakery/tests/test_bakery.py | 13 ++++++++++++- - 1 file changed, 12 insertions(+), 1 deletion(-) - -diff --git a/macaroonbakery/tests/test_bakery.py b/macaroonbakery/tests/test_bakery.py -index 72a6928..1883987 100644 ---- a/macaroonbakery/tests/test_bakery.py -+++ b/macaroonbakery/tests/test_bakery.py -@@ -146,6 +146,16 @@ def discharge_401(url, request): - } - - -+@urlmatch(path='.*/visit') -+def visit_200(url, request): -+ return { -+ 'status_code': 200, -+ 'content': { -+ 'interactive': '/visit' -+ } -+ } -+ -+ - @urlmatch(path='.*/wait') - def wait_after_401(url, request): - if request.url != 'http://example.com/wait': -@@ -245,7 +255,8 @@ class TestBakery(TestCase): - def kind(self): - return 'unknown' - client = httpbakery.Client(interaction_methods=[UnknownInteractor()]) -- with HTTMock(first_407_then_200), HTTMock(discharge_401): -+ with HTTMock(first_407_then_200), HTTMock(discharge_401),\ -+ HTTMock(visit_200): - with self.assertRaises(httpbakery.InteractionError) as exc: - requests.get( - ID_PATH, |