diff --git a/tests/test_pagure_flask_api_fork.py b/tests/test_pagure_flask_api_fork.py index b793a61..65f1b82 100644 --- a/tests/test_pagure_flask_api_fork.py +++ b/tests/test_pagure_flask_api_fork.py @@ -795,7 +795,7 @@ class PagureFlaskApiForktests(tests.Modeltests): self.assertEqual(req.title, 'test pull-request') # Check comments before - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.comments), 0) @@ -819,7 +819,7 @@ class PagureFlaskApiForktests(tests.Modeltests): ) # No change - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.comments), 0) @@ -839,7 +839,7 @@ class PagureFlaskApiForktests(tests.Modeltests): ) # One comment added - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.comments), 1) @@ -912,7 +912,7 @@ class PagureFlaskApiForktests(tests.Modeltests): self.assertEqual(req.title, 'test pull-request') # Check comments before - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.comments), 0) @@ -936,7 +936,7 @@ class PagureFlaskApiForktests(tests.Modeltests): ) # No change - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.comments), 0) @@ -956,7 +956,7 @@ class PagureFlaskApiForktests(tests.Modeltests): ) # One comment added - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.comments), 1) diff --git a/tests/test_pagure_flask_api_issue.py b/tests/test_pagure_flask_api_issue.py index b192a97..f5a8f97 100644 --- a/tests/test_pagure_flask_api_issue.py +++ b/tests/test_pagure_flask_api_issue.py @@ -2504,7 +2504,7 @@ class PagureFlaskApiIssuetests(tests.SimplePagureTest): ) # One comment added - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') issue = pagure.lib.search_issues(self.session, repo, issueid=1) self.assertEqual(issue.assignee.user, 'pingou') @@ -3016,7 +3016,7 @@ class PagureFlaskApiIssuetests(tests.SimplePagureTest): } ) - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') issue = pagure.lib.search_issues(self.session, repo, issueid=1) self.assertEqual(len(issue.other_fields), 1) @@ -3039,7 +3039,7 @@ class PagureFlaskApiIssuetests(tests.SimplePagureTest): } ) - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') issue = pagure.lib.search_issues(self.session, repo, issueid=1) self.assertEqual(len(issue.other_fields), 0) diff --git a/tests/test_pagure_flask_api_issue_custom_fields.py b/tests/test_pagure_flask_api_issue_custom_fields.py index 8b13fea..d15ccc5 100644 --- a/tests/test_pagure_flask_api_issue_custom_fields.py +++ b/tests/test_pagure_flask_api_issue_custom_fields.py @@ -122,7 +122,7 @@ class PagureFlaskApiCustomFieldIssuetests(tests.Modeltests): } ) - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') issue = pagure.lib.search_issues(self.session, repo, issueid=1) self.assertEqual(len(issue.other_fields), 1) @@ -149,7 +149,7 @@ class PagureFlaskApiCustomFieldIssuetests(tests.Modeltests): } ) - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') issue = pagure.lib.search_issues(self.session, repo, issueid=1) self.assertEqual(len(issue.other_fields), 3) diff --git a/tests/test_pagure_flask_api_pr_flag.py b/tests/test_pagure_flask_api_pr_flag.py index 9f1b47e..3f044b4 100644 --- a/tests/test_pagure_flask_api_pr_flag.py +++ b/tests/test_pagure_flask_api_pr_flag.py @@ -58,7 +58,7 @@ class PagureFlaskApiPRFlagtests(tests.Modeltests): self.assertEqual(req.title, 'test pull-request') # Check flags before - # self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.flags), 0) @@ -159,7 +159,7 @@ class PagureFlaskApiPRFlagtests(tests.Modeltests): ) # No change - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.flags), 0) @@ -215,7 +215,7 @@ class PagureFlaskApiPRFlagtests(tests.Modeltests): ) # One flag added - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.flags), 1) @@ -276,7 +276,7 @@ class PagureFlaskApiPRFlagtests(tests.Modeltests): ) # One flag added - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.flags), 1) @@ -321,7 +321,7 @@ class PagureFlaskApiPRFlagtests(tests.Modeltests): ) # One flag added - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.flags), 1) @@ -371,7 +371,7 @@ class PagureFlaskApiPRFlagtests(tests.Modeltests): ) # One flag added - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.flags), 1) @@ -417,7 +417,7 @@ class PagureFlaskApiPRFlagtests(tests.Modeltests): ) # Two flag added - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.flags), 2) @@ -533,7 +533,7 @@ class PagureFlaskApiPRFlagUserTokentests(tests.Modeltests): self.assertEqual(req.title, 'test pull-request') # Check flags before - # self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.flags), 0) @@ -620,7 +620,7 @@ class PagureFlaskApiPRFlagUserTokentests(tests.Modeltests): ) # No change - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.flags), 0) @@ -653,7 +653,7 @@ class PagureFlaskApiPRFlagUserTokentests(tests.Modeltests): ) # No change - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.flags), 0) @@ -704,7 +704,7 @@ class PagureFlaskApiPRFlagUserTokentests(tests.Modeltests): ) # One flag added - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.flags), 1) @@ -757,7 +757,7 @@ class PagureFlaskApiPRFlagUserTokentests(tests.Modeltests): ) # One flag added - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.flags), 1) @@ -803,7 +803,7 @@ class PagureFlaskApiPRFlagUserTokentests(tests.Modeltests): ) # Still only one flag - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.flags), 1) diff --git a/tests/test_pagure_flask_api_ui_private_repo.py b/tests/test_pagure_flask_api_ui_private_repo.py index 1530504..2d4f200 100644 --- a/tests/test_pagure_flask_api_ui_private_repo.py +++ b/tests/test_pagure_flask_api_ui_private_repo.py @@ -619,7 +619,7 @@ class PagurePrivateRepotest(tests.Modeltests): '', output.get_data(as_text=True)) - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib._get_project(self.session, 'test4') self.assertTrue(repo.private) @@ -640,7 +640,7 @@ class PagurePrivateRepotest(tests.Modeltests): '', output.get_data(as_text=True)) - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib._get_project(self.session, 'test4') self.assertFalse(repo.private) @@ -675,7 +675,7 @@ class PagurePrivateRepotest(tests.Modeltests): '', output.get_data(as_text=True)) - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib._get_project(self.session, 'test4') self.assertFalse(repo.private) @@ -697,7 +697,7 @@ class PagurePrivateRepotest(tests.Modeltests): output.get_data(as_text=True)) # No change since we can't do public -> private - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib._get_project(self.session, 'test4') self.assertFalse(repo.private) @@ -1530,7 +1530,7 @@ class PagurePrivateRepotest(tests.Modeltests): self.assertEqual(req.title, 'test pull-request') # Check comments before - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.comments), 0) @@ -1554,7 +1554,7 @@ class PagurePrivateRepotest(tests.Modeltests): ) # No change - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.comments), 0) @@ -1574,7 +1574,7 @@ class PagurePrivateRepotest(tests.Modeltests): ) # One comment added - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.comments), 1) @@ -1667,7 +1667,7 @@ class PagurePrivateRepotest(tests.Modeltests): self.assertEqual(req.title, 'test pull-request') # Check comments before - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.flags), 0) @@ -1694,7 +1694,7 @@ class PagurePrivateRepotest(tests.Modeltests): ) # No change - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.flags), 0) @@ -1736,7 +1736,7 @@ class PagurePrivateRepotest(tests.Modeltests): ) # One flag added - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.flags), 1) @@ -1780,7 +1780,7 @@ class PagurePrivateRepotest(tests.Modeltests): ) # One flag added - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.flags), 1) @@ -2893,7 +2893,7 @@ class PagurePrivateRepotest(tests.Modeltests): self.assertEqual(msg.title, 'Test issue #1') # Check comments before - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib._get_project(self.session, 'test4') issue = pagure.lib.search_issues(self.session, repo, issueid=1) self.assertEqual(len(issue.comments), 0) @@ -2917,7 +2917,7 @@ class PagurePrivateRepotest(tests.Modeltests): ) # No change - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib._get_project(self.session, 'test4') issue = pagure.lib.search_issues(self.session, repo, issueid=1) self.assertEqual(issue.status, 'Open') @@ -2937,7 +2937,7 @@ class PagurePrivateRepotest(tests.Modeltests): ) # One comment added - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib._get_project(self.session, 'test4') issue = pagure.lib.search_issues(self.session, repo, issueid=1) self.assertEqual(len(issue.comments), 1) diff --git a/tests/test_pagure_flask_api_user.py b/tests/test_pagure_flask_api_user.py index 3208523..d7b99fd 100644 --- a/tests/test_pagure_flask_api_user.py +++ b/tests/test_pagure_flask_api_user.py @@ -288,7 +288,7 @@ class PagureFlaskApiUSertests(tests.Modeltests): self.assertEqual(req.title, 'test pull-request') # Check comments before - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.comments), 0) @@ -308,7 +308,7 @@ class PagureFlaskApiUSertests(tests.Modeltests): ) # One comment added - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(len(request.comments), 1) @@ -324,7 +324,7 @@ class PagureFlaskApiUSertests(tests.Modeltests): ) # PR closed - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() request = pagure.lib.search_pull_requests( self.session, project_id=1, requestid=1) self.assertEqual(request.status, 'Closed') diff --git a/tests/test_pagure_flask_internal.py b/tests/test_pagure_flask_internal.py index b13b250..c654989 100644 --- a/tests/test_pagure_flask_internal.py +++ b/tests/test_pagure_flask_internal.py @@ -116,7 +116,7 @@ class PagureFlaskInternaltests(tests.Modeltests): js_data = json.loads(output.data) self.assertDictEqual(js_data, {'message': 'Comment added'}) - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') request = repo.requests[0] self.assertEqual(len(request.comments), 1) @@ -196,7 +196,7 @@ class PagureFlaskInternaltests(tests.Modeltests): js_data = json.loads(output.data) self.assertDictEqual(js_data, {'message': 'Comment added'}) - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') issue = repo.issues[0] self.assertEqual(len(issue.comments), 1) @@ -286,7 +286,7 @@ class PagureFlaskInternaltests(tests.Modeltests): js_data = json.loads(output.data) self.assertDictEqual(js_data, {'message': 'Comment added'}) - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') issue = repo.issues[0] self.assertEqual(len(issue.comments), 1) diff --git a/tests/test_pagure_flask_ui_app_give_project.py b/tests/test_pagure_flask_ui_app_give_project.py index 0fbe8dd..3e71b22 100644 --- a/tests/test_pagure_flask_ui_app_give_project.py +++ b/tests/test_pagure_flask_ui_app_give_project.py @@ -43,7 +43,7 @@ class PagureFlaskGiveRepotests(tests.SimplePagureTest): tests.create_projects_git(os.path.join(self.path, 'repos'), bare=True) def _check_user(self, user='pingou'): - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() project = pagure.lib.get_authorized_project( self.session, project_name='test') self.assertEqual(project.user.user, user) diff --git a/tests/test_pagure_flask_ui_fork.py b/tests/test_pagure_flask_ui_fork.py index 40665b3..5a02e97 100644 --- a/tests/test_pagure_flask_ui_fork.py +++ b/tests/test_pagure_flask_ui_fork.py @@ -337,7 +337,7 @@ class PagureFlaskForktests(tests.Modeltests): self.assertEqual(output.status_code, 404) # Project w/o pull-request - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') settings = repo.settings settings['pull_requests'] = False @@ -351,7 +351,7 @@ class PagureFlaskForktests(tests.Modeltests): self.assertEqual(output.status_code, 404) # Project w pull-request but only assignee can merge - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') settings['pull_requests'] = True settings['Only_assignee_can_merge_pull-request'] = True @@ -374,7 +374,7 @@ class PagureFlaskForktests(tests.Modeltests): 'assigned to be merged', output.data) # PR assigned but not to this user - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') req = repo.requests[0] req.assignee_id = 2 @@ -393,7 +393,7 @@ class PagureFlaskForktests(tests.Modeltests): 'merge this review', output.data) # Project w/ minimal PR score - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') settings['Only_assignee_can_merge_pull-request'] = False settings['Minimum_score_to_merge_pull-request'] = 2 @@ -414,7 +414,7 @@ class PagureFlaskForktests(tests.Modeltests): output.data) # Merge - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') settings['Minimum_score_to_merge_pull-request'] = -1 repo.settings = settings @@ -675,7 +675,7 @@ class PagureFlaskForktests(tests.Modeltests): ' PR from the feature branch\n', output.data) self.assertTrue( output.data.count('\n Comment updated', output.data) - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') issue = pagure.lib.search_issues(self.session, repo, issueid=1) self.assertEqual(len(issue.comments), 1) @@ -3073,7 +3073,7 @@ class PagureFlaskIssuestests(tests.Modeltests): '\n Comment updated', output.data) - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') issue = pagure.lib.search_issues(self.session, repo, issueid=1) self.assertEqual(len(issue.comments), 1) @@ -3158,7 +3158,7 @@ class PagureFlaskIssuestests(tests.Modeltests): ) # Ticket #1 has one more comment and is still open - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() issue = pagure.lib.search_issues(self.session, repo, issueid=1) self.assertEqual(len(issue.comments), 2) self.assertEqual(issue.status, 'Open') diff --git a/tests/test_pagure_flask_ui_login.py b/tests/test_pagure_flask_ui_login.py index f43d01d..50bfe55 100644 --- a/tests/test_pagure_flask_ui_login.py +++ b/tests/test_pagure_flask_ui_login.py @@ -200,7 +200,7 @@ class PagureFlaskLogintests(tests.SimplePagureTest): 'provided by email?', output.data) # Confirm the user so that we can log in - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() item = pagure.lib.search_user(self.session, username='foouser') self.assertEqual(item.user, 'foouser') self.assertNotEqual(item.token, None) @@ -228,7 +228,7 @@ class PagureFlaskLogintests(tests.SimplePagureTest): # partly fail if hasattr(flask, '__version__'): flask_v = tuple(int(el) for el in flask.__version__.split('.')) - if flask_v <= (0, 12, 0): + if flask_v < (0, 12, 0): self.assertIn( '', output.data) @@ -241,7 +241,7 @@ class PagureFlaskLogintests(tests.SimplePagureTest): 'href="/logout/?next=http://localhost/">', output.data) # Make the password invalid - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() item = pagure.lib.search_user(self.session, username='foouser') self.assertEqual(item.user, 'foouser') self.assertTrue(item.password.startswith('$2$')) @@ -252,7 +252,7 @@ class PagureFlaskLogintests(tests.SimplePagureTest): self.session.commit() # Check the password - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() item = pagure.lib.search_user(self.session, username='foouser') self.assertEqual(item.user, 'foouser') self.assertFalse(item.password.startswith('$2$')) @@ -268,7 +268,7 @@ class PagureFlaskLogintests(tests.SimplePagureTest): self.assertIn('Username or password of invalid format.', output.data) # Check the password is still not of a known version - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() item = pagure.lib.search_user(self.session, username='foouser') self.assertEqual(item.user, 'foouser') self.assertFalse(item.password.startswith('$1$')) @@ -283,7 +283,7 @@ class PagureFlaskLogintests(tests.SimplePagureTest): self.session.commit() # Check the password - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() item = pagure.lib.search_user(self.session, username='foouser') self.assertEqual(item.user, 'foouser') self.assertTrue(item.password.startswith('$1$')) @@ -297,7 +297,7 @@ class PagureFlaskLogintests(tests.SimplePagureTest): self.assertIn('Activity', output.data) # Check the password got upgraded to version 2 - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() item = pagure.lib.search_user(self.session, username='foouser') self.assertEqual(item.user, 'foouser') self.assertTrue(item.password.startswith('$2$')) diff --git a/tests/test_pagure_flask_ui_priorities.py b/tests/test_pagure_flask_ui_priorities.py index d8caff0..8483ec5 100644 --- a/tests/test_pagure_flask_ui_priorities.py +++ b/tests/test_pagure_flask_ui_priorities.py @@ -181,7 +181,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): 'Settings - test - Pagure', output.data) self.assertIn('

Settings for test

', output.data) # Check the result of the action -- Priority recorded - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual(repo.priorities, {u'': u'', u'1': u'High'}) @@ -203,7 +203,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): self.assertTrue( output.data.find('Normal') < output.data.find('Low')) # Check the result of the action -- Priority recorded - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual( repo.priorities, @@ -228,7 +228,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): ' Priorities weights and titles are ' 'not of the same length', output.data) # Check the result of the action -- Priorities un-changed - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual( repo.priorities, @@ -253,7 +253,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): ' Priorities weights must be numbers', output.data) # Check the result of the action -- Priorities un-changed - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual( repo.priorities, @@ -278,7 +278,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): ' Priority weight 2 is present 2 times', output.data) # Check the result of the action -- Priorities un-changed - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual( repo.priorities, @@ -303,7 +303,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): ' Priority Normal is present 2 times', output.data) # Check the result of the action -- Priorities un-changed - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual( repo.priorities, @@ -381,7 +381,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): self.assertIn('

Settings for test

', output.data) # Check the result of the action -- Priority recorded - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib._get_project(self.session, 'test') self.assertEqual( repo.priorities, @@ -436,7 +436,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): self.assertNotIn('', output.data) # Check the result of the action -- Priority reset - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib._get_project(self.session, 'test') self.assertEqual(repo.priorities, {}) @@ -482,7 +482,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): self.assertIn('

Settings for test

', output.data) # Check the result of the action -- Priority recorded - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib._get_project(self.session, 'test') self.assertEqual( repo.priorities, @@ -538,7 +538,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): self.assertNotIn('', output.data) # Check the result of the action -- Priority recorded - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib._get_project(self.session, 'test') self.assertEqual(repo.priorities, {}) @@ -583,7 +583,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): self.assertIn('

Settings for test

', output.data) # Check the result of the action -- Priority recorded - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib._get_project(self.session, 'test') self.assertEqual( repo.priorities, @@ -707,7 +707,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): self.assertIn('

Settings for test

', output.data) # Check the result of the action -- Priority recorded - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib._get_project(self.session, 'test') self.assertEqual( repo.priorities, @@ -810,7 +810,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): self.assertIn('

Settings for test

', output.data) # Check the result of the action -- Priority recorded - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib._get_project(self.session, 'test') self.assertEqual( repo.priorities, @@ -908,7 +908,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): self.assertTrue( output.data.find('Normal') < output.data.find('Low')) # Check the result of the action -- Priority recorded - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual( repo.priorities, @@ -926,7 +926,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): 'Settings - test - Pagure', output.data) self.assertIn('

Settings for test

', output.data) # Check the result of the action -- default_priority no change - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual(repo.default_priority, None) @@ -944,7 +944,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): '\n Default priority set ' 'to High', output.data) # Check the result of the action -- default_priority no change - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual(repo.default_priority, 'High') @@ -959,7 +959,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): 'Settings - test - Pagure', output.data) self.assertIn('

Settings for test

', output.data) # Check the result of the action -- default_priority no change - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual(repo.default_priority, 'High') @@ -977,7 +977,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): '\n Default priority reset', output.data) # Check the result of the action -- default_priority no change - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual(repo.default_priority, None) @@ -1049,7 +1049,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): self.assertTrue( output.data.find('Normal') < output.data.find('Low')) # Check the result of the action -- Priority recorded - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual( repo.priorities, @@ -1070,7 +1070,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): '\n Default priority set ' 'to High', output.data) # Check the result of the action -- default_priority no change - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual(repo.default_priority, 'High') @@ -1098,7 +1098,7 @@ class PagureFlaskPrioritiestests(tests.Modeltests): self.assertTrue( output.data.find('Normal') < output.data.find('Low')) # Check the result of the action -- Priority recorded - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual( repo.priorities, diff --git a/tests/test_pagure_flask_ui_quick_reply.py b/tests/test_pagure_flask_ui_quick_reply.py index 8260fd3..7cc1e1a 100644 --- a/tests/test_pagure_flask_ui_quick_reply.py +++ b/tests/test_pagure_flask_ui_quick_reply.py @@ -77,7 +77,7 @@ class PagureFlaskQuickReplytest(tests.Modeltests): self.assertIn(notice, output.data) def assertQuickReplies(self, quick_replies, project='test'): - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib._get_project(self.session, project) self.assertEqual(repo.quick_replies, quick_replies) diff --git a/tests/test_pagure_flask_ui_repo.py b/tests/test_pagure_flask_ui_repo.py index 0cf7737..efaac6d 100644 --- a/tests/test_pagure_flask_ui_repo.py +++ b/tests/test_pagure_flask_ui_repo.py @@ -716,7 +716,7 @@ class PagureFlaskRepotests(tests.Modeltests): '\n User removed', output.data) self.assertNotIn('action="/test/dropuser/2">', output.data) - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual(len(repo.users), 0) @@ -761,7 +761,7 @@ class PagureFlaskRepotests(tests.Modeltests): self.assertIn( u'\n User removed', output.data) - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual(len(repo.users), 0) @@ -937,7 +937,7 @@ class PagureFlaskRepotests(tests.Modeltests): output.data) self.assertNotIn('action="/test/dropgroup/1">', output.data) - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual(len(repo.groups), 0) @@ -3720,7 +3720,7 @@ index 0000000..fb7093d output.data) pagure.config.config['WEBHOOK'] = False - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertNotEqual(repo.hook_token, 'aaabbbccc') @@ -4423,7 +4423,7 @@ index 0000000..fb7093d output.data) # Existing token has been expired - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual( repo.tokens[0].expiration.date(), @@ -4824,7 +4824,7 @@ index 0000000..fb7093d self.assertIn( '\n List of reports updated', output.data) - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() project = pagure.lib.get_authorized_project(self.session, project_name='test') self.assertEqual(project.reports, {}) @@ -4884,7 +4884,7 @@ index 0000000..fb7093d output.data) # Create a report - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() project = pagure.lib.get_authorized_project( self.session, project_name='test', namespace='foo') self.assertEqual(project.reports, {}) @@ -4935,7 +4935,7 @@ index 0000000..fb7093d '\n List of reports updated', output.data) - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() project = pagure.lib.get_authorized_project( self.session, project_name='test', namespace='foo') self.assertEqual(project.reports, {}) diff --git a/tests/test_pagure_flask_ui_roadmap.py b/tests/test_pagure_flask_ui_roadmap.py index d7e9cb1..798503b 100644 --- a/tests/test_pagure_flask_ui_roadmap.py +++ b/tests/test_pagure_flask_ui_roadmap.py @@ -194,7 +194,7 @@ class PagureFlaskRoadmaptests(tests.Modeltests): self.assertIn(u'

Settings for test

', output.data) self.assertIn(u'Milestones updated', output.data) # Check the result of the action -- Milestones recorded - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual(repo.milestones, {u'1': {u'active': False, u'date': None}}) @@ -213,7 +213,7 @@ class PagureFlaskRoadmaptests(tests.Modeltests): self.assertIn(u'

Settings for test

', output.data) self.assertIn(u'Milestones updated', output.data) # Check the result of the action -- Milestones recorded - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual( repo.milestones, @@ -239,7 +239,7 @@ class PagureFlaskRoadmaptests(tests.Modeltests): u'Settings - test - Pagure', output.data) self.assertIn(u'

Settings for test

', output.data) # Check the result of the action -- Milestones un-changed - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual( repo.milestones, @@ -269,7 +269,7 @@ class PagureFlaskRoadmaptests(tests.Modeltests): ' Milestone v2.0 is present 2 times', output.data) # Check the result of the action -- Milestones un-changed - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual( repo.milestones, @@ -299,7 +299,7 @@ class PagureFlaskRoadmaptests(tests.Modeltests): ' Milestones updated', output.data) # Check the result of the action -- Milestones updated - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual( repo.milestones, @@ -368,7 +368,7 @@ class PagureFlaskRoadmaptests(tests.Modeltests): self.assertIn(u'

Settings for test

', output.data) self.assertIn(u'Milestones updated', output.data) # Check the result of the action -- Milestones recorded - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'test') self.assertEqual( repo.milestones, @@ -421,7 +421,7 @@ class PagureFlaskRoadmaptests(tests.Modeltests): self.assertIn(u'

Settings for test

', output.data) self.assertIn(u'Milestones updated', output.data) # Check the result of the action -- Milestones recorded - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib._get_project(self.session, 'test') self.assertEqual( repo.milestones, diff --git a/tests/test_pagure_lib.py b/tests/test_pagure_lib.py index 6f5e53b..a396c06 100644 --- a/tests/test_pagure_lib.py +++ b/tests/test_pagure_lib.py @@ -1638,7 +1638,7 @@ class PagureLibtests(tests.Modeltests): ) # Now test that creation fails if ignore_existing_repo is False - self.session = pagure.lib.create_session(self.dbpath) + self.session.commit() repo = pagure.lib.get_authorized_project(self.session, 'testproject') self.assertEqual(repo.path, 'testproject.git') @@ -1697,7 +1697,6 @@ class PagureLibtests(tests.Modeltests): repo = pagure.lib._get_project(self.session, 'testproject') self.session.delete(repo) self.session.commit() - self.session = pagure.lib.create_session(self.dbpath) task = pagure.lib.new_project( session=self.session,