From c0c45c051a6eadf34d38f1b0d3d96c6a9196f733 Mon Sep 17 00:00:00 2001 From: Aldo Cortesi Date: Sun, 5 Jun 2016 11:47:52 +1200 Subject: Fix test that may fail due to binary header data This is just inherently not a determinisitc test. We don't use the log HTTP interface any more, so it can just go. A more radical "solution" is inbound shortly. Fixes #1207 --- test/pathod/test_app.py | 3 --- test/pathod/test_pathoc.py | 2 +- test/pathod/tutils.py | 4 +++- 3 files changed, 4 insertions(+), 5 deletions(-) (limited to 'test/pathod') diff --git a/test/pathod/test_app.py b/test/pathod/test_app.py index fbaa773c..19888c75 100644 --- a/test/pathod/test_app.py +++ b/test/pathod/test_app.py @@ -32,9 +32,6 @@ class TestApp(tutils.DaemonTests): assert self.getpath("/log/%s" % id).status_code == 200 assert self.getpath("/log/9999999").status_code == 404 - def test_log_binary(self): - assert self.get("200:h@10b=@10b:da") - def test_response_preview(self): r = self.getpath("/response_preview", params=dict(spec="200")) assert r.status_code == 200 diff --git a/test/pathod/test_pathoc.py b/test/pathod/test_pathoc.py index 6e36c4bf..18d7f672 100644 --- a/test/pathod/test_pathoc.py +++ b/test/pathod/test_pathoc.py @@ -187,7 +187,7 @@ class TestDaemon(_TestDaemon): def test_showresp_httperr(self): v = self.tval(["get:'/p/200:d20'"], showresp=True, showsummary=True) - assert "Invalid headers" in v + assert "Invalid header" in v assert "HTTP/" in v def test_explain(self): diff --git a/test/pathod/tutils.py b/test/pathod/tutils.py index b9f38d86..1a883c93 100644 --- a/test/pathod/tutils.py +++ b/test/pathod/tutils.py @@ -102,7 +102,9 @@ class DaemonTests(object): fp=logfp, ) with c.connect(): - resp = c.request("get:/p/%s" % urllib.quote(spec).encode("string_escape")) + resp = c.request( + "get:/p/%s" % urllib.quote(spec).encode("string_escape") + ) return resp def pathoc( -- cgit v1.2.3