100% test coverage.

libpathod/pathoc.py

@@ -65,8 +65,10 @@ class Pathoc(tcp.TCPClient):
                 return
             except tcp.NetLibTimeout:
                 print >> fp, "<<", "Timeout"
-            except tcp.NetLibDisconnect:
+                return
+            except tcp.NetLibDisconnect: # pragma: nocover
                 print >> fp, "<<", "Disconnect"
+                return
             else:
                 if respdump:
                     print_full(fp, *resp)

libpathod/pathod.py

@@ -9,9 +9,6 @@ class PathodError(Exception): pass
 class PathodHandler(tcp.BaseHandler):
     wbufsize = 0
     sni = None
-    def debug(self, s):
-        logging.debug("%s:%s: %s"%(self.client_address[0], self.client_address[1], str(s)))
-
     def info(self, s):
         logging.info("%s:%s: %s"%(self.client_address[0], self.client_address[1], str(s)))
 

test/test_pathoc.py

@@ -57,3 +57,4 @@ class TestDaemon:
         d = tutils.test_data.path("data/request")
         assert "foo" in self.tval(["+%s"%d])
         assert "File" in self.tval(["+/nonexistent"])
+

test/test_pathod.py

@@ -107,6 +107,12 @@ class CommonTests(tutils.DaemonTests):
         assert l["type"] == "error"
         assert "Invalid" in l["msg"]
 
+    def test_invalid_headers(self):
+        tutils.raises(http.HttpError, self.pathoc, "get:/:h'\t'='foo'")
+        l = self.d.log()[0]
+        assert l["type"] == "error"
+        assert "Invalid headers" in l["msg"]
+
     def test_access_denied(self):
         rsp = self.get("=nonexistent")
         assert rsp.status_code == 800