From 59857af32e3c161c690e8af34ab7580210140048 Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Apr 28 2015 06:51:23 +0000
Subject: Add unit-tests for the view_commits endpoint when the git repo has no master


---

diff --git a/tests/test_progit_flask_ui_no_master_branch.py b/tests/test_progit_flask_ui_no_master_branch.py
index 99627cd..550bd4e 100644
--- a/tests/test_progit_flask_ui_no_master_branch.py
+++ b/tests/test_progit_flask_ui_no_master_branch.py
@@ -159,6 +159,35 @@ class PagureFlaskNoMasterBranchtests(tests.Modeltests):
         self.assertIn('<h3>Last 1 commits</h3>', output.data)
         self.assertTrue(output.data.count('<a href="/test/branch/'), 1)
 
+    @patch('pagure.lib.notify.send_email')
+    def test_view_commits(self, send_email):
+        """ Test the view_commits endpoint when the git repo has no
+        master branch.
+        """
+        send_email.return_value = True
+
+        tests.create_projects(self.session)
+        # Non-existant git repo
+        output = self.app.get('/test/commits')
+        self.assertEqual(output.status_code, 404)
+
+        self.set_up_git_repo()
+
+        # With git repo
+        output = self.app.get('/test/commits')
+        self.assertEqual(output.status_code, 200)
+        self.assertIn(
+            '<h3>Commits list</h3>\n    <ul>\n    </ul>', output.data)
+        self.assertTrue(output.data.count('<a href="/test/branch/'), 1)
+
+        output = self.app.get('/test/commits/feature')
+        self.assertEqual(output.status_code, 200)
+        self.assertIn('<h3>Commits list</h3>', output.data)
+        self.assertIn('Add sources file for testing', output.data)
+        self.assertNotIn(
+            '<h3>Commits list</h3>\n    <ul>\n    </ul>', output.data)
+        self.assertTrue(output.data.count('<a href="/test/branch/'), 1)
+
 
 if __name__ == '__main__':
     SUITE = unittest.TestLoader().loadTestsFromTestCase(