diff --git a/tests/test_progit_flask_api_fork.py b/tests/test_progit_flask_api_fork.py index 592a572..41ea5b9 100644 --- a/tests/test_progit_flask_api_fork.py +++ b/tests/test_progit_flask_api_fork.py @@ -54,7 +54,7 @@ class PagureFlaskApiForktests(tests.Modeltests): # Create a pull-request repo = pagure.lib.get_project(self.session, 'test') forked_repo = pagure.lib.get_project(self.session, 'test') - msg = pagure.lib.new_pull_request( + req = pagure.lib.new_pull_request( session=self.session, repo_from=forked_repo, branch_from='master', @@ -65,7 +65,8 @@ class PagureFlaskApiForktests(tests.Modeltests): requestfolder=None, ) self.session.commit() - self.assertEqual(msg, 'Request created') + self.assertEqual(req.id, 1) + self.assertEqual(req.title, 'test pull-request') # Invalid repo output = self.app.get('/api/0/foo/pull-requests') @@ -177,7 +178,7 @@ class PagureFlaskApiForktests(tests.Modeltests): # Create a pull-request repo = pagure.lib.get_project(self.session, 'test') forked_repo = pagure.lib.get_project(self.session, 'test') - msg = pagure.lib.new_pull_request( + req = pagure.lib.new_pull_request( session=self.session, repo_from=forked_repo, branch_from='master', @@ -188,7 +189,8 @@ class PagureFlaskApiForktests(tests.Modeltests): requestfolder=None, ) self.session.commit() - self.assertEqual(msg, 'Request created') + self.assertEqual(req.id, 1) + self.assertEqual(req.title, 'test pull-request') # Invalid repo output = self.app.get('/api/0/foo/pull-request/1') @@ -304,7 +306,7 @@ class PagureFlaskApiForktests(tests.Modeltests): # Create the pull-request to close repo = pagure.lib.get_project(self.session, 'test') forked_repo = pagure.lib.get_project(self.session, 'test') - msg = pagure.lib.new_pull_request( + req = pagure.lib.new_pull_request( session=self.session, repo_from=forked_repo, branch_from='master', @@ -315,7 +317,8 @@ class PagureFlaskApiForktests(tests.Modeltests): requestfolder=None, ) self.session.commit() - self.assertEqual(msg, 'Request created') + self.assertEqual(req.id, 1) + self.assertEqual(req.title, 'test pull-request') headers = {'Authorization': 'token aaabbbcccddd'} @@ -414,7 +417,7 @@ class PagureFlaskApiForktests(tests.Modeltests): # Create the pull-request to close repo = pagure.lib.get_project(self.session, 'test') forked_repo = pagure.lib.get_project(self.session, 'test') - msg = pagure.lib.new_pull_request( + req = pagure.lib.new_pull_request( session=self.session, repo_from=forked_repo, branch_from='master', @@ -425,7 +428,8 @@ class PagureFlaskApiForktests(tests.Modeltests): requestfolder=None, ) self.session.commit() - self.assertEqual(msg, 'Request created') + self.assertEqual(req.id, 1) + self.assertEqual(req.title, 'test pull-request') headers = {'Authorization': 'token aaabbbcccddd'} @@ -566,7 +570,7 @@ class PagureFlaskApiForktests(tests.Modeltests): # Create a pull-request repo = pagure.lib.get_project(self.session, 'test') forked_repo = pagure.lib.get_project(self.session, 'test') - msg = pagure.lib.new_pull_request( + req = pagure.lib.new_pull_request( session=self.session, repo_from=forked_repo, branch_from='master', @@ -577,7 +581,8 @@ class PagureFlaskApiForktests(tests.Modeltests): requestfolder=None, ) self.session.commit() - self.assertEqual(msg, 'Request created') + self.assertEqual(req.id, 1) + self.assertEqual(req.title, 'test pull-request') # Check comments before request = pagure.lib.search_pull_requests( @@ -680,7 +685,7 @@ class PagureFlaskApiForktests(tests.Modeltests): # Create a pull-request repo = pagure.lib.get_project(self.session, 'test') forked_repo = pagure.lib.get_project(self.session, 'test') - msg = pagure.lib.new_pull_request( + req = pagure.lib.new_pull_request( session=self.session, repo_from=forked_repo, branch_from='master', @@ -691,7 +696,8 @@ class PagureFlaskApiForktests(tests.Modeltests): requestfolder=None, ) self.session.commit() - self.assertEqual(msg, 'Request created') + self.assertEqual(req.id, 1) + self.assertEqual(req.title, 'test pull-request') # Check comments before request = pagure.lib.search_pull_requests( diff --git a/tests/test_progit_flask_internal.py b/tests/test_progit_flask_internal.py index 7300ceb..d812b13 100644 --- a/tests/test_progit_flask_internal.py +++ b/tests/test_progit_flask_internal.py @@ -58,7 +58,7 @@ class PagureFlaskInternaltests(tests.Modeltests): repo = pagure.lib.get_project(self.session, 'test') - msg = pagure.lib.new_pull_request( + req = pagure.lib.new_pull_request( session=self.session, repo_from=repo, branch_from='feature', @@ -69,7 +69,8 @@ class PagureFlaskInternaltests(tests.Modeltests): requestfolder=None, ) self.session.commit() - self.assertEqual(msg, 'Request created') + self.assertEqual(req.id, 1) + self.assertEqual(req.title, 'PR from the feature branch') request = repo.requests[0] self.assertEqual(len(request.comments), 0) @@ -353,7 +354,7 @@ class PagureFlaskInternaltests(tests.Modeltests): # Create a PR for these changes tests.create_projects(self.session) project = pagure.lib.get_project(self.session, 'test') - msg = pagure.lib.new_pull_request( + req = pagure.lib.new_pull_request( session=self.session, repo_from=project, branch_from='feature', @@ -364,7 +365,8 @@ class PagureFlaskInternaltests(tests.Modeltests): requestfolder=None, ) self.session.commit() - self.assertEqual(msg, 'Request created') + self.assertEqual(req.id, 1) + self.assertEqual(req.title, 'PR from the feature branch') # Check if the PR can be merged data = { @@ -474,7 +476,7 @@ class PagureFlaskInternaltests(tests.Modeltests): # Create a PR for these changes tests.create_projects(self.session) project = pagure.lib.get_project(self.session, 'test') - msg = pagure.lib.new_pull_request( + req = pagure.lib.new_pull_request( session=self.session, repo_from=project, branch_from='master', @@ -485,7 +487,8 @@ class PagureFlaskInternaltests(tests.Modeltests): requestfolder=None, ) self.session.commit() - self.assertEqual(msg, 'Request created') + self.assertEqual(req.id, 1) + self.assertEqual(req.title, 'PR from the feature branch') # Check if the PR can be merged data = { @@ -616,7 +619,7 @@ class PagureFlaskInternaltests(tests.Modeltests): # Create a PR for these changes tests.create_projects(self.session) project = pagure.lib.get_project(self.session, 'test') - msg = pagure.lib.new_pull_request( + req = pagure.lib.new_pull_request( session=self.session, repo_from=project, branch_from='feature', @@ -627,7 +630,8 @@ class PagureFlaskInternaltests(tests.Modeltests): requestfolder=None, ) self.session.commit() - self.assertEqual(msg, 'Request created') + self.assertEqual(req.id, 1) + self.assertEqual(req.title, 'PR from the feature branch') # Check if the PR can be merged data = {} @@ -756,7 +760,7 @@ class PagureFlaskInternaltests(tests.Modeltests): # Create a PR for these changes tests.create_projects(self.session) project = pagure.lib.get_project(self.session, 'test') - msg = pagure.lib.new_pull_request( + req = pagure.lib.new_pull_request( session=self.session, repo_from=project, branch_from='feature', @@ -767,7 +771,8 @@ class PagureFlaskInternaltests(tests.Modeltests): requestfolder=None, ) self.session.commit() - self.assertEqual(msg, 'Request created') + self.assertEqual(req.id, 1) + self.assertEqual(req.title, 'PR from the feature branch') # Check if the PR can be merged data = {} diff --git a/tests/test_progit_flask_ui_fork.py b/tests/test_progit_flask_ui_fork.py index da5f035..1068b72 100644 --- a/tests/test_progit_flask_ui_fork.py +++ b/tests/test_progit_flask_ui_fork.py @@ -187,7 +187,7 @@ class PagureFlaskForktests(tests.Modeltests): # Create a PR for these changes project = pagure.lib.get_project(self.session, 'test') - msg = pagure.lib.new_pull_request( + req = pagure.lib.new_pull_request( session=self.session, repo_from=project, branch_from=branch_from, @@ -198,7 +198,8 @@ class PagureFlaskForktests(tests.Modeltests): requestfolder=None, ) self.session.commit() - self.assertEqual(msg, 'Request created') + self.assertEqual(req.id, 1) + self.assertEqual(req.title, 'PR from the %s branch' % branch_from) shutil.rmtree(newpath) @@ -556,7 +557,7 @@ class PagureFlaskForktests(tests.Modeltests): # Create a PR for these changes project = pagure.lib.get_project(self.session, 'test') - msg = pagure.lib.new_pull_request( + req = pagure.lib.new_pull_request( session=self.session, repo_from=item, branch_from='feature', @@ -568,7 +569,8 @@ class PagureFlaskForktests(tests.Modeltests): ) self.session.commit() - self.assertEqual(msg, 'Request created') + self.assertEqual(req.id, 1) + self.assertEqual(req.title, 'PR from the feature branch') output = self.app.get('/test/pull-request/1') self.assertEqual(output.status_code, 200) @@ -613,7 +615,7 @@ class PagureFlaskForktests(tests.Modeltests): # Create a PR for these "changes" (there are none, both repos are # empty) project = pagure.lib.get_project(self.session, 'test') - msg = pagure.lib.new_pull_request( + req = pagure.lib.new_pull_request( session=self.session, repo_from=item, branch_from='feature', @@ -625,7 +627,8 @@ class PagureFlaskForktests(tests.Modeltests): ) self.session.commit() - self.assertEqual(msg, 'Request created') + self.assertEqual(req.id, 1) + self.assertEqual(req.title, 'PR from the feature branch') output = self.app.get('/test/pull-request/1', follow_redirects=True) self.assertEqual(output.status_code, 200) @@ -852,7 +855,7 @@ index 9f44358..2a552bb 100644 # Create a PR for these "changes" (there are none, both repos are # empty) project = pagure.lib.get_project(self.session, 'test') - msg = pagure.lib.new_pull_request( + req = pagure.lib.new_pull_request( session=self.session, repo_from=item, branch_from='feature', @@ -864,7 +867,8 @@ index 9f44358..2a552bb 100644 ) self.session.commit() - self.assertEqual(msg, 'Request created') + self.assertEqual(req.id, 1) + self.assertEqual(req.title, 'PR from the feature branch') output = self.app.get('/test/pull-request/1.patch', follow_redirects=True) self.assertEqual(output.status_code, 200) @@ -939,7 +943,7 @@ index 0000000..2a552bb # Create a PR for these "changes" (there are none, both repos are # empty) project = pagure.lib.get_project(self.session, 'test') - msg = pagure.lib.new_pull_request( + req = pagure.lib.new_pull_request( session=self.session, repo_from=item, branch_from='feature', @@ -951,7 +955,8 @@ index 0000000..2a552bb ) self.session.commit() - self.assertEqual(msg, 'Request created') + self.assertEqual(req.id, 1) + self.assertEqual(req.title, 'PR from the feature branch') output = self.app.get('/test/pull-request/1.patch', follow_redirects=True) self.assertEqual(output.status_code, 200) @@ -1282,7 +1287,8 @@ index 0000000..2a552bb '/test/diff/master..feature', data=data, follow_redirects=True) self.assertEqual(output.status_code, 200) self.assertIn( - '