From dec404675205040f616f7a02f1f4116989c04f52 Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Sep 07 2016 12:30:24 +0000 Subject: Adjust unit-tests for the change in check of the git repo --- diff --git a/tests/test_pagure_flask_ui_issues.py b/tests/test_pagure_flask_ui_issues.py index 8ab1e3f..bd9b21c 100644 --- a/tests/test_pagure_flask_ui_issues.py +++ b/tests/test_pagure_flask_ui_issues.py @@ -63,8 +63,9 @@ class PagureFlaskIssuestests(tests.Modeltests): p_send_email.return_value = True p_ugt.return_value = True + # No Git repo output = self.app.get('/foo/new_issue') - self.assertEqual(output.status_code, 302) + self.assertEqual(output.status_code, 404) user = tests.FakeUser() with tests.user_set(pagure.APP, user): @@ -124,6 +125,10 @@ class PagureFlaskIssuestests(tests.Modeltests): '

No such user found in the database: username

', output.data) + # User not logged in + output = self.app.get('/test/new_issue') + self.assertEqual(output.status_code, 302) + user.username = 'pingou' with tests.user_set(pagure.APP, user): output = self.app.post( @@ -1228,8 +1233,9 @@ class PagureFlaskIssuestests(tests.Modeltests): p_send_email.return_value = True p_ugt.return_value = True + # No Git repo output = self.app.get('/foo/issue/1/edit') - self.assertEqual(output.status_code, 302) + self.assertEqual(output.status_code, 404) user = tests.FakeUser() with tests.user_set(pagure.APP, user): @@ -1243,6 +1249,10 @@ class PagureFlaskIssuestests(tests.Modeltests): output = self.app.get('/test/issue/1/edit') self.assertEqual(output.status_code, 404) + # User not logged in + output = self.app.get('/foo/issue/1/edit') + self.assertEqual(output.status_code, 404) + user.username = 'pingou' with tests.user_set(pagure.APP, user): output = self.app.get('/test/issue/1/edit') @@ -1342,8 +1352,9 @@ class PagureFlaskIssuestests(tests.Modeltests): p_send_email.return_value = True p_ugt.return_value = True + # No Git repo output = self.app.get('/foo/tag/foo/edit') - self.assertEqual(output.status_code, 302) + self.assertEqual(output.status_code, 404) user = tests.FakeUser() with tests.user_set(pagure.APP, user): @@ -1355,6 +1366,10 @@ class PagureFlaskIssuestests(tests.Modeltests): output = self.app.get('/test/tag/foo/edit') self.assertEqual(output.status_code, 403) + # User not logged in + output = self.app.get('/test/tag/foo/edit') + self.assertEqual(output.status_code, 302) + # Create issues to play with repo = pagure.lib.get_project(self.session, 'test') msg = pagure.lib.new_issue( @@ -1431,8 +1446,9 @@ class PagureFlaskIssuestests(tests.Modeltests): p_send_email.return_value = True p_ugt.return_value = True + # No Git repo output = self.app.post('/foo/droptag/') - self.assertEqual(output.status_code, 302) + self.assertEqual(output.status_code, 404) user = tests.FakeUser() with tests.user_set(pagure.APP, user): @@ -1444,6 +1460,10 @@ class PagureFlaskIssuestests(tests.Modeltests): output = self.app.post('/test/droptag/') self.assertEqual(output.status_code, 403) + # User not logged in + output = self.app.post('/test/droptag/') + self.assertEqual(output.status_code, 302) + # Create issues to play with repo = pagure.lib.get_project(self.session, 'test') msg = pagure.lib.new_issue(