From 5ae446fbd290273eb4eb1f52abec0d6a04c214ab Mon Sep 17 00:00:00 2001 From: david-sarah Date: Mon, 10 Oct 2011 17:22:27 -0700 Subject: [PATCH] test/common.py: in shouldFail and shouldHTTPError, when the raised exception does not include the expected substring (or, for shouldHTTPError, when the status code is wrong), mention which test that happened in. --- src/allmydata/test/common.py | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/allmydata/test/common.py b/src/allmydata/test/common.py index 42890d81..5f379003 100644 --- a/src/allmydata/test/common.py +++ b/src/allmydata/test/common.py @@ -1069,8 +1069,8 @@ class ShouldFailMixin: if substring: message = repr(res.value.args[0]) self.failUnless(substring in message, - "substring '%s' not in '%s'" - % (substring, message)) + "%s: substring '%s' not in '%s'" + % (which, substring, message)) else: self.fail("%s was supposed to raise %s, not get '%s'" % (which, expected_failure, res)) @@ -1103,17 +1103,17 @@ class WebErrorMixin: assert callable def _validate(f): if code is not None: - self.failUnlessEqual(f.value.status, str(code)) + self.failUnlessEqual(f.value.status, str(code), which) if substring: code_string = str(f) self.failUnless(substring in code_string, - "substring '%s' not in '%s'" - % (substring, code_string)) + "%s: substring '%s' not in '%s'" + % (which, substring, code_string)) response_body = f.value.response if response_substring: self.failUnless(response_substring in response_body, - "response substring '%s' not in '%s'" - % (response_substring, response_body)) + "%s: response substring '%s' not in '%s'" + % (which, response_substring, response_body)) return response_body d = defer.maybeDeferred(callable, *args, **kwargs) d.addBoth(self._shouldHTTPError, which, _validate) -- 2.45.2