#1',
- output_text)
- self.assertIn(
- '\n', output_text)
- self.assertIn(
- '
\n', output_text)
- self.assertNotIn(
- '
\n', output_text)
+ with patch(
+ 'pagure.forms.RemoteRequestPullForm.git_repo.args',
+ MagicMock(return_value=(
+ u'Git Repo address', [wtforms.validators.Required()]))):
+
+ output = self.app.post(
+ '/test/diff/remote', data=data, follow_redirects=True)
+ self.assertEqual(output.status_code, 200)
+
+ data['confirm'] = 1
+ output = self.app.post(
+ '/test/diff/remote', data=data, follow_redirects=True)
+ self.assertEqual(output.status_code, 200)
+ output_text = output.get_data(as_text=True)
+ self.assertIn(
+ '
#1',
+ output_text)
+ self.assertIn(
+ '\n', output_text)
+ self.assertIn(
+ '
\n', output_text)
+ self.assertNotIn(
+ '
\n', output_text)
# Remote PR Created
self.session = pagure.lib.create_session(self.dbpath)
@@ -503,24 +519,32 @@ class PagureRemotePRtests(tests.Modeltests):
'branch_to': 'master',
'git_repo': os.path.join(self.newpath, 'test'),
}
- output = self.app.post(
- '/test/diff/remote', data=data, follow_redirects=True)
- self.assertEqual(output.status_code, 200)
-
- data['confirm'] = 1
- output = self.app.post(
- '/test/diff/remote', data=data, follow_redirects=True)
- self.assertEqual(output.status_code, 200)
- output_text = output.get_data(as_text=True)
- self.assertIn(
- '
#1',
- output_text)
- self.assertIn(
- '\n', output_text)
- self.assertIn(
- '
\n', output_text)
- self.assertNotIn(
- '
\n', output_text)
+ # Disables checking the URL pattern for git_repo
+ with patch(
+ 'pagure.forms.RemoteRequestPullForm.git_repo.args',
+ MagicMock(return_value=(
+ u'Git Repo address', [wtforms.validators.Required()]))):
+
+ # Do the preview, triggers the cache & all
+ output = self.app.post(
+ '/test/diff/remote', data=data, follow_redirects=True)
+ self.assertEqual(output.status_code, 200)
+
+ # Confirm the PR creation
+ data['confirm'] = 1
+ output = self.app.post(
+ '/test/diff/remote', data=data, follow_redirects=True)
+ self.assertEqual(output.status_code, 200)
+ output_text = output.get_data(as_text=True)
+ self.assertIn(
+ '
#1',
+ output_text)
+ self.assertIn(
+ '\n', output_text)
+ self.assertIn(
+ '
\n', output_text)
+ self.assertNotIn(
+ '
\n', output_text)
# Remote PR Created
self.session = pagure.lib.create_session(self.dbpath)