Changeset View
Changeset View
Standalone View
Standalone View
swh/web/tests/browse/views/test_directory.py
| # Copyright (C) 2017-2019 The Software Heritage developers | # Copyright (C) 2017-2019 The Software Heritage developers | ||||
| # See the AUTHORS file at the top-level directory of this distribution | # See the AUTHORS file at the top-level directory of this distribution | ||||
| # License: GNU Affero General Public License version 3, or any later version | # License: GNU Affero General Public License version 3, or any later version | ||||
| # See top-level LICENSE file for more information | # See top-level LICENSE file for more information | ||||
| import random | import random | ||||
| from hypothesis import given | from hypothesis import given | ||||
| from swh.web.common.utils import reverse, get_swh_persistent_id | from swh.web.common.utils import reverse, get_swh_persistent_id | ||||
| from swh.web.common.utils import gen_path_info | from swh.web.common.utils import gen_path_info | ||||
| from swh.web.tests.django_asserts import assert_contains, assert_template_used | |||||
| from swh.web.tests.strategies import ( | from swh.web.tests.strategies import ( | ||||
| directory, directory_with_subdirs, invalid_sha1, | directory, directory_with_subdirs, invalid_sha1, | ||||
| unknown_directory | unknown_directory | ||||
| ) | ) | ||||
| from swh.web.tests.testcase import WebTestCase | |||||
| class SwhBrowseDirectoryTest(WebTestCase): | @given(directory()) | ||||
| def test_root_directory_view(client, archive_data, directory): | |||||
| _directory_view(client, directory, archive_data.directory_ls(directory)) | |||||
| @given(directory_with_subdirs()) | |||||
| def test_sub_directory_view(client, archive_data, directory): | |||||
| dir_content = archive_data.directory_ls(directory) | |||||
| subdir = random.choice([e for e in dir_content if e['type'] == 'dir']) | |||||
| subdir_content = archive_data.directory_ls(subdir['target']) | |||||
| _directory_view(client, directory, subdir_content, subdir['name']) | |||||
| @given(invalid_sha1(), unknown_directory()) | |||||
| def test_directory_request_errors(client, invalid_sha1, unknown_directory): | |||||
| dir_url = reverse('browse-directory', | |||||
| url_args={'sha1_git': invalid_sha1}) | |||||
| resp = client.get(dir_url) | |||||
| assert resp.status_code == 400 | |||||
| assert_template_used('browse/error.html') | |||||
| dir_url = reverse('browse-directory', | |||||
| url_args={'sha1_git': unknown_directory}) | |||||
| resp = client.get(dir_url) | |||||
| assert resp.status_code == 404 | |||||
| assert_template_used('browse/error.html') | |||||
| @given(directory()) | |||||
| def test_directory_uppercase(client, directory): | |||||
| url = reverse('browse-directory-uppercase-checksum', | |||||
| url_args={'sha1_git': directory.upper()}) | |||||
| resp = client.get(url) | |||||
| assert resp.status_code == 302 | |||||
| redirect_url = reverse('browse-directory', | |||||
| url_args={'sha1_git': directory}) | |||||
| assert resp['location'] == redirect_url | |||||
| def directory_view(self, root_directory_sha1, directory_entries, | |||||
| def _directory_view(client, root_directory_sha1, directory_entries, | |||||
| path=None): | path=None): | ||||
| dirs = [e for e in directory_entries if e['type'] in ('dir', 'rev')] | dirs = [e for e in directory_entries if e['type'] in ('dir', 'rev')] | ||||
| files = [e for e in directory_entries if e['type'] == 'file'] | files = [e for e in directory_entries if e['type'] == 'file'] | ||||
| url_args = {'sha1_git': root_directory_sha1} | url_args = {'sha1_git': root_directory_sha1} | ||||
| if path: | if path: | ||||
| url_args['path'] = path | url_args['path'] = path | ||||
| url = reverse('browse-directory', | url = reverse('browse-directory', | ||||
| url_args=url_args) | url_args=url_args) | ||||
| root_dir_url = reverse('browse-directory', | root_dir_url = reverse('browse-directory', | ||||
| url_args={'sha1_git': root_directory_sha1}) | url_args={'sha1_git': root_directory_sha1}) | ||||
| resp = self.client.get(url) | resp = client.get(url) | ||||
| self.assertEqual(resp.status_code, 200) | assert resp.status_code == 200 | ||||
| self.assertTemplateUsed('browse/directory.html') | assert_template_used('browse/directory.html') | ||||
| self.assertContains(resp, '<a href="' + root_dir_url + '">' + | assert_contains(resp, '<a href="' + root_dir_url + '">' + | ||||
| root_directory_sha1[:7] + '</a>') | root_directory_sha1[:7] + '</a>') | ||||
| self.assertContains(resp, '<td class="swh-directory">', | assert_contains(resp, '<td class="swh-directory">', | ||||
| count=len(dirs)) | count=len(dirs)) | ||||
| self.assertContains(resp, '<td class="swh-content">', | assert_contains(resp, '<td class="swh-content">', | ||||
| count=len(files)) | count=len(files)) | ||||
| for d in dirs: | for d in dirs: | ||||
| if d['type'] == 'rev': | if d['type'] == 'rev': | ||||
| dir_url = reverse('browse-revision', | dir_url = reverse('browse-revision', | ||||
| url_args={'sha1_git': d['target']}) | url_args={'sha1_git': d['target']}) | ||||
| else: | else: | ||||
| dir_path = d['name'] | dir_path = d['name'] | ||||
| if path: | if path: | ||||
| dir_path = "%s/%s" % (path, d['name']) | dir_path = "%s/%s" % (path, d['name']) | ||||
| dir_url = reverse('browse-directory', | dir_url = reverse('browse-directory', | ||||
| url_args={'sha1_git': root_directory_sha1, | url_args={'sha1_git': root_directory_sha1, | ||||
| 'path': dir_path}) | 'path': dir_path}) | ||||
| self.assertContains(resp, dir_url) | assert_contains(resp, dir_url) | ||||
| for f in files: | for f in files: | ||||
| file_path = "%s/%s" % (root_directory_sha1, f['name']) | file_path = "%s/%s" % (root_directory_sha1, f['name']) | ||||
| if path: | if path: | ||||
| file_path = "%s/%s/%s" % (root_directory_sha1, path, f['name']) | file_path = "%s/%s/%s" % (root_directory_sha1, path, f['name']) | ||||
| query_string = 'sha1_git:' + f['target'] | query_string = 'sha1_git:' + f['target'] | ||||
| file_url = reverse('browse-content', | file_url = reverse('browse-content', | ||||
| url_args={'query_string': query_string}, | url_args={'query_string': query_string}, | ||||
| query_params={'path': file_path}) | query_params={'path': file_path}) | ||||
| self.assertContains(resp, file_url) | assert_contains(resp, file_url) | ||||
| path_info = gen_path_info(path) | path_info = gen_path_info(path) | ||||
| self.assertContains(resp, '<li class="swh-path">', | assert_contains(resp, '<li class="swh-path">', | ||||
| count=len(path_info)+1) | count=len(path_info)+1) | ||||
| self.assertContains(resp, '<a href="%s">%s</a>' % | assert_contains(resp, '<a href="%s">%s</a>' % | ||||
| (root_dir_url, root_directory_sha1[:7])) | (root_dir_url, root_directory_sha1[:7])) | ||||
| for p in path_info: | for p in path_info: | ||||
| dir_url = reverse('browse-directory', | dir_url = reverse('browse-directory', | ||||
| url_args={'sha1_git': root_directory_sha1, | url_args={'sha1_git': root_directory_sha1, | ||||
| 'path': p['path']}) | 'path': p['path']}) | ||||
| self.assertContains(resp, '<a href="%s">%s</a>' % | assert_contains(resp, '<a href="%s">%s</a>' % | ||||
| (dir_url, p['name'])) | (dir_url, p['name'])) | ||||
| self.assertContains(resp, 'vault-cook-directory') | assert_contains(resp, 'vault-cook-directory') | ||||
| swh_dir_id = get_swh_persistent_id('directory', directory_entries[0]['dir_id']) # noqa | swh_dir_id = get_swh_persistent_id( | ||||
| 'directory', directory_entries[0]['dir_id']) | |||||
| swh_dir_id_url = reverse('browse-swh-id', | swh_dir_id_url = reverse('browse-swh-id', | ||||
| url_args={'swh_id': swh_dir_id}) | url_args={'swh_id': swh_dir_id}) | ||||
| self.assertContains(resp, swh_dir_id) | assert_contains(resp, swh_dir_id) | ||||
| self.assertContains(resp, swh_dir_id_url) | assert_contains(resp, swh_dir_id_url) | ||||
| @given(directory()) | |||||
| def test_root_directory_view(self, directory): | |||||
| self.directory_view(directory, self.directory_ls(directory)) | |||||
| @given(directory_with_subdirs()) | |||||
| def test_sub_directory_view(self, directory): | |||||
| dir_content = self.directory_ls(directory) | |||||
| subdir = random.choice([e for e in dir_content if e['type'] == 'dir']) | |||||
| subdir_content = self.directory_ls(subdir['target']) | |||||
| self.directory_view(directory, subdir_content, subdir['name']) | |||||
| @given(invalid_sha1(), unknown_directory()) | |||||
| def test_directory_request_errors(self, invalid_sha1, unknown_directory): | |||||
| dir_url = reverse('browse-directory', | |||||
| url_args={'sha1_git': invalid_sha1}) | |||||
| resp = self.client.get(dir_url) | |||||
| self.assertEqual(resp.status_code, 400) | |||||
| self.assertTemplateUsed('browse/error.html') | |||||
| dir_url = reverse('browse-directory', | |||||
| url_args={'sha1_git': unknown_directory}) | |||||
| resp = self.client.get(dir_url) | |||||
| self.assertEqual(resp.status_code, 404) | |||||
| self.assertTemplateUsed('browse/error.html') | |||||
| @given(directory()) | |||||
| def test_directory_uppercase(self, directory): | |||||
| url = reverse('browse-directory-uppercase-checksum', | |||||
| url_args={'sha1_git': directory.upper()}) | |||||
| resp = self.client.get(url) | |||||
| self.assertEqual(resp.status_code, 302) | |||||
| redirect_url = reverse('browse-directory', | |||||
| url_args={'sha1_git': directory}) | |||||
| self.assertEqual(resp['location'], redirect_url) | |||||
vlorentz: what happened to these tests? | |||||
Done Inline ActionsThey are still here, I simply moved private helper functions to the end of the file and put test functions at the beginning. anlambert: They are still here, I simply moved private helper functions to the end of the file and put… | |||||
what happened to these tests?