Skip to content

Commit

Permalink
start resource proxy test for status code 200
Browse files Browse the repository at this point in the history
  • Loading branch information
domoritz committed Nov 11, 2012
1 parent 1975050 commit 5369644
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 6 deletions.
1 change: 1 addition & 0 deletions ckan/lib/helpers.py
Expand Up @@ -1326,6 +1326,7 @@ def resource_preview(resource, pkg_id):
directly = False
url = ''

# todo: add package
data_dict = {'resource': resource, 'package': None}

if not resource['url']:
Expand Down
12 changes: 6 additions & 6 deletions ckanext/resourceproxy/tests/test_proxy.py
Expand Up @@ -72,11 +72,11 @@ def teardown_class(cls):

def test_resource_proxy(self):
url = self.data_dict['resource']['url']
result = self.app.get(url, status=200)
assert result.status == 200, result.status
assert "yes, I'm proxied" in result.body, result.body
result = requests.get(url)
assert result.status_code == 200, result.status_code
assert "yes, I'm proxied" in result.content, result.content

proxied_url = proxy.get_proxified_resource_url(self.data_dict)
result = self.app.get(proxied_url).follow()
assert result.status == 200, result.status
assert "yes, I'm proxied" in result.body, result.body
result = requests.get(proxied_url)
assert result.status_code == 200, result.status_code
assert "yes, I'm proxied" in result.content, result.content

0 comments on commit 5369644

Please sign in to comment.