diff options
author | bbangert <devnull@localhost> | 2006-06-03 00:31:41 +0000 |
---|---|---|
committer | bbangert <devnull@localhost> | 2006-06-03 00:31:41 +0000 |
commit | 5b02061b69a69cbe10ff6cb717d0f49d50269af7 (patch) | |
tree | 2e4f3d6f2d09b2892d88d1670107ef4153193f54 /paste/fixture.py | |
parent | 00ac4e80a50220f69578a81948d40575c3eeb2ea (diff) | |
download | paste-5b02061b69a69cbe10ff6cb717d0f49d50269af7.tar.gz |
Fixing default {}, since it will always be retained. Creating a {} for each call where its not passed.
Diffstat (limited to 'paste/fixture.py')
-rw-r--r-- | paste/fixture.py | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/paste/fixture.py b/paste/fixture.py index c2a694b..9cfebfa 100644 --- a/paste/fixture.py +++ b/paste/fixture.py @@ -139,7 +139,7 @@ class TestApp(object): environ['paste.throw_errors'] = True return environ - def get(self, url, params=None, headers={}, extra_environ={}, + def get(self, url, params=None, headers=None, extra_environ=None, status=None, expect_errors=False): """ Get the given url (well, actually a path like @@ -171,6 +171,10 @@ class TestApp(object): Returns a `response object <class-paste.fixture.TestResponse.html>`_ """ + if headers is None: + headers = {} + if extra_environ is None: + extra_environ = {} # Hide from py.test: __tracebackhide__ = True if params: @@ -193,7 +197,7 @@ class TestApp(object): req = TestRequest(url, environ, expect_errors) return self.do_request(req, status=status) - def post(self, url, params='', headers={}, extra_environ={}, + def post(self, url, params='', headers=None, extra_environ=None, status=None, upload_files=None, expect_errors=False): """ Do a POST request. Very like the ``.get()`` method. @@ -207,6 +211,10 @@ class TestApp(object): Returns a `response object <class-paste.fixture.TestResponse.html>`_ """ + if headers is None: + headers = {} + if extra_environ is None: + extra_environ = {} environ = self._make_environ() # @@: Should this be all non-strings? if params and isinstance(params, (list, tuple, dict)): |