diff --git a/tests/test_progit_flask_ui_fork.py b/tests/test_progit_flask_ui_fork.py index f995e5c..d76c97a 100644 --- a/tests/test_progit_flask_ui_fork.py +++ b/tests/test_progit_flask_ui_fork.py @@ -321,8 +321,8 @@ class PagureFlaskForktests(tests.Modeltests): '

PR#1 ' 'PR from the feature branchThis request must be assigned to be merged', - output.data) + '\n This request must be ' + 'assigned to be merged', output.data) # PR assigned but not to this user repo = pagure.lib.get_project(self.session, 'test') @@ -338,8 +338,8 @@ class PagureFlaskForktests(tests.Modeltests): '

PR#1 ' 'PR from the feature branchOnly the assignee can merge this review', - output.data) + '\n Only the assignee can ' + 'merge this review', output.data) # Project w/ minimal PR score settings['Only_assignee_can_merge_pull-request'] = False @@ -355,8 +355,9 @@ class PagureFlaskForktests(tests.Modeltests): '

PR#1 ' 'PR from the feature branchThis request does not have the minimum ' - 'review score necessary to be merged', output.data) + '\n This request does not ' + 'have the minimum review score necessary to be merged', + output.data) # Merge settings['Minimum_score_to_merge_pull-request'] = -1 @@ -369,7 +370,8 @@ class PagureFlaskForktests(tests.Modeltests): self.assertIn( 'Overview - test - Pagure', output.data) self.assertIn( - '
  • Changes merged!
  • ', output.data) + '\n Changes merged!', + output.data) @patch('pagure.lib.notify.send_email') def test_merge_request_pull_merge(self, send_email): @@ -402,7 +404,8 @@ class PagureFlaskForktests(tests.Modeltests): self.assertIn( 'Overview - test - Pagure', output.data) self.assertIn( - '
  • Changes merged!
  • ', output.data) + '\n Changes merged!', + output.data) @patch('pagure.lib.notify.send_email') def test_merge_request_pull_conflicts(self, send_email): @@ -436,7 +439,8 @@ class PagureFlaskForktests(tests.Modeltests): '

    PR#1 ' 'PR from the feature branchMerge conflicts!', output.data) + '\n Merge conflicts!', + output.data) @patch('pagure.lib.notify.send_email') def test_merge_request_pull_nochange(self, send_email): @@ -470,8 +474,8 @@ class PagureFlaskForktests(tests.Modeltests): '

    PR#1 ' 'PR from the master branch

    ', output.data) self.assertIn( - '
  • Nothing to do, changes were already ' - 'merged
  • ', output.data) + '\n Nothing to do, changes ' + 'were already merged', output.data) @patch('pagure.lib.notify.send_email') def test_request_pull_close(self, send_email): @@ -648,8 +652,8 @@ class PagureFlaskForktests(tests.Modeltests): self.assertIn( 'Overview - test - Pagure', output.data) self.assertIn( - '
  • Fork is empty, there are no commits to ' - 'request pulling
  • ', output.data) + '\n Fork is empty, there are no ' + 'commits to request pulling', output.data) shutil.rmtree(newpath) @@ -981,8 +985,8 @@ index 0000000..2a552bb self.assertIn( 'Overview - test - Pagure', output.data) self.assertIn( - '
  • Fork is empty, there are no commits to ' - 'request pulling
  • ', output.data) + '\n Fork is empty, there are no ' + 'commits to request pulling', output.data) shutil.rmtree(newpath) @@ -1008,7 +1012,8 @@ index 0000000..2a552bb self.assertIn( 'Overview - test - Pagure', output.data) self.assertIn( - '
  • Invalid input submitted
  • ', output.data) + '\n Invalid input submitted', + output.data) output = self.app.get('/test/pull-request/1') self.assertEqual(output.status_code, 200) @@ -1063,7 +1068,7 @@ index 0000000..2a552bb self.assertIn( 'Overview - test - Pagure', output.data) self.assertIn( - '
  • Request pull canceled!
  • ', + '\n Request pull canceled!', output.data) @patch('pagure.lib.notify.send_email') @@ -1097,7 +1102,8 @@ index 0000000..2a552bb '

    PR#1 ' 'PR from the feature branch

    ', output.data) self.assertNotIn( - '
  • Request assigned
  • ', output.data) + '\n Request assigned', + output.data) output = self.app.get('/test/pull-request/1') self.assertEqual(output.status_code, 200) @@ -1121,7 +1127,8 @@ index 0000000..2a552bb '

    PR#1 ' 'PR from the feature branch

    ', output.data) self.assertNotIn( - '
  • Request assigned
  • ', output.data) + '\n Request assigned', + output.data) # Invalid assignee data = { @@ -1140,7 +1147,7 @@ index 0000000..2a552bb '

    PR#1 ' 'PR from the feature branch

    ', output.data) self.assertIn( - '
  • No user "bar" found
  • ', + '\n No user "bar" found', output.data) # Assign the PR @@ -1160,7 +1167,8 @@ index 0000000..2a552bb '

    PR#1 ' 'PR from the feature branch

    ', output.data) self.assertIn( - '
  • Request assigned
  • ', output.data) + '\n Request assigned', + output.data) # Pull-Request closed repo = pagure.lib.get_project(self.session, 'test') @@ -1223,8 +1231,8 @@ index 0000000..2a552bb '/do_fork/test', data=data, follow_redirects=True) self.assertEqual(output.status_code, 200) self.assertIn( - '
  • You may not fork your own repo
  • ', - output.data) + '\n You may not fork your ' + 'own repo', output.data) user.username = 'foo' with tests.user_set(pagure.APP, user): @@ -1239,8 +1247,8 @@ index 0000000..2a552bb '/do_fork/test', data=data, follow_redirects=True) self.assertEqual(output.status_code, 200) self.assertIn( - '
  • Repo "test" cloned to ' - '"foo/test"
  • ', output.data) + '\n Repo "test" ' + 'cloned to "foo/test"', output.data) @patch('pagure.lib.notify.send_email') def test_new_request_pull(self, send_email): @@ -1313,7 +1321,8 @@ index 0000000..2a552bb 'PR#2: foo bar PR - test\n - Pagure', output.data) self.assertIn( - '
  • Request created
  • ', output.data) + '\n Request created', + output.data) @patch('pagure.lib.notify.send_email') def test_new_request_pull_empty_repo(self, send_email): @@ -1347,8 +1356,8 @@ index 0000000..2a552bb self.assertIn( 'Overview - test - Pagure', output.data) self.assertIn( - '
  • Fork is empty, there are no commits to ' - 'request pulling
  • ', output.data) + '\n Fork is empty, there are ' + 'no commits to request pulling', output.data) output = self.app.get('/test/new_issue') csrf_token = output.data.split( @@ -1365,8 +1374,8 @@ index 0000000..2a552bb self.assertIn( 'Overview - test - Pagure', output.data) self.assertIn( - '
  • Fork is empty, there are no commits to ' - 'request pulling
  • ', output.data) + '\n Fork is empty, there are ' + 'no commits to request pulling', output.data) shutil.rmtree(newpath) @@ -1400,8 +1409,8 @@ index 0000000..2a552bb self.assertIn( 'Overview - test - Pagure', output.data) self.assertIn( - '
  • Fork is empty, there are no commits to ' - 'request pulling
  • ', output.data) + '\n Fork is empty, there are ' + 'no commits to request pulling', output.data) shutil.rmtree(newpath) @@ -1441,7 +1450,8 @@ index 0000000..2a552bb 'PR#1: PR from the feature branch - test\n - ' 'Pagure', output.data) self.assertIn( - '
  • Comment added
  • ', output.data) + '\n Comment added', + output.data) # Project w/o pull-request repo = pagure.lib.get_project(self.session, 'test') @@ -1520,7 +1530,8 @@ index 0000000..2a552bb '

    PR#1 ' 'PR from the feature branchComment removed', output.data) + '\n Comment removed', + output.data) # Project w/o pull-request repo = pagure.lib.get_project(self.session, 'test') @@ -1573,9 +1584,11 @@ index 0000000..2a552bb self.assertIn( '

    PR#1 ' - 'PR from the feature branchComment added', output.data) + '\n Comment added', + output.data) # Check if the comment is there self.assertIn( '

    This look alright but we can do better

    ', output.data) @@ -1608,7 +1621,7 @@ index 0000000..2a552bb 'Edited just now by pingou ', output.data) self.assertIn( - '
  • Comment updated
  • ', output.data) + '\n Comment updated', output.data) # Project w/o pull-request repo = pagure.lib.get_project(self.session, 'test') diff --git a/tests/test_progit_flask_ui_slash_branch_name.py b/tests/test_progit_flask_ui_slash_branch_name.py index 97d6dbf..9e53717 100644 --- a/tests/test_progit_flask_ui_slash_branch_name.py +++ b/tests/test_progit_flask_ui_slash_branch_name.py @@ -140,9 +140,11 @@ class PagureFlaskSlashInBranchtests(tests.Modeltests): # With git repo output = self.app.get('/test') self.assertEqual(output.status_code, 200) - self.assertIn('

    Last 1 commits

    ', output.data) - self.assertEqual( - output.data.count('

    Admins

    ', output.data) + self.assertEqual(output.data.count( + '

    Admins

    ', output.data) + self.assertEqual(output.data.count( + '
    master/sources

    ', + '
  •   master
  • ' + '
  • ' + '  sources
  • ', output.data) output = self.app.get('/test/blob/master/f/.gitignore') @@ -219,8 +225,12 @@ class PagureFlaskSlashInBranchtests(tests.Modeltests): output = self.app.get('/test/blob/maxamilion/feature/f/.gitignore') self.assertEqual(output.status_code, 200) self.assertIn( - 'maxamilion/feature' - '/.gitignore

    ', output.data) + '
  • ' + '' + '  maxamilion/feature
  • ' + '
  • ' + '  .gitignore
  • ', + output.data) self.assertIn('
    *~
    ', output.data) @patch('pagure.lib.notify.send_email') @@ -277,34 +287,47 @@ class PagureFlaskSlashInBranchtests(tests.Modeltests): output = self.app.get('/test/tree/master') self.assertEqual(output.status_code, 200) self.assertIn('', output.data) - self.assertEqual(output.data.count('
  • '), 1) + self.assertEqual( + output.data.count(''), 1) output = self.app.get('/test/tree/master/sources') self.assertEqual(output.status_code, 200) self.assertIn('', output.data) - self.assertEqual(output.data.count('
  • '), 1) + self.assertEqual( + output.data.count(''), 1) output = self.app.get('/test/tree/feature') self.assertEqual(output.status_code, 200) self.assertIn('', output.data) - self.assertIn('9f4435', output.data) - self.assertEqual(output.data.count('
  • '), 1) + self.assertIn( + '\n' + ' 9f4435', output.data) + self.assertEqual( + output.data.count(''), 1) output = self.app.get('/test/tree/maxamilion/feature') self.assertEqual(output.status_code, 200) self.assertIn( '', output.data) - self.assertIn('9f4435', output.data) - self.assertIn('e4e5f6', output.data) - self.assertEqual(output.data.count('
  • '), 2) + self.assertIn( + '\n' + ' 9f4435', output.data) + self.assertIn( + '\n' + ' e4e5f6', output.data) + self.assertEqual( + output.data.count(''), 2) # Wrong identifier, back onto master output = self.app.get('/test/tree/maxamilion/feature/f/.gitignore') self.assertEqual(output.status_code, 200) self.assertIn('', output.data) - self.assertIn('9f4435', output.data) - self.assertEqual(output.data.count('
  • '), 1) + self.assertIn( + '\n' + ' 9f4435', output.data) + self.assertEqual( + output.data.count(''), 1) @patch('pagure.lib.notify.send_email') def test_new_request_pull(self, send_email):