From 21dd347f942d33dd668f91c02f8115d1098d3395 Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Feb 19 2016 22:08:25 +0000
Subject: Add unit-tests checking if the notification is added when a PR is closed/merged


---

diff --git a/tests/test_pagure_flask_ui_fork.py b/tests/test_pagure_flask_ui_fork.py
index f36b735..55c88d4 100644
--- a/tests/test_pagure_flask_ui_fork.py
+++ b/tests/test_pagure_flask_ui_fork.py
@@ -408,6 +408,13 @@ class PagureFlaskForktests(tests.Modeltests):
                 ]
             )
 
+            # Check if the closing notification was added
+            output = self.app.get('/test/pull-request/1')
+            self.assertIn(
+                '<div>\n                '
+                'Pull-Request has been merged by pingou\n                '
+                '<div class="pull-xs-right text-muted">', output.data)
+
     @patch('pagure.lib.notify.send_email')
     def test_merge_request_pull_merge(self, send_email):
         """ Test the merge_request_pull endpoint with a merge PR. """
@@ -442,6 +449,13 @@ class PagureFlaskForktests(tests.Modeltests):
                 '</button>\n                      Changes merged!',
                 output.data)
 
+            # Check if the closing notification was added
+            output = self.app.get('/test/pull-request/1')
+            self.assertIn(
+                '<div>\n                '
+                'Pull-Request has been merged by pingou\n                '
+                '<div class="pull-xs-right text-muted">', output.data)
+
     @patch('pagure.lib.notify.send_email')
     def test_merge_request_pull_conflicts(self, send_email):
         """ Test the merge_request_pull endpoint with a conflicting PR. """
@@ -514,6 +528,13 @@ class PagureFlaskForktests(tests.Modeltests):
                 '</button>\n                      Nothing to do, changes '
                 'were already merged', output.data)
 
+            # Check if the closing notification was added
+            output = self.app.get('/test/pull-request/1')
+            self.assertIn(
+                '<div>\n                '
+                'Pull-Request has been merged by pingou\n                '
+                '<div class="pull-xs-right text-muted">', output.data)
+
     @patch('pagure.lib.notify.send_email')
     def test_request_pull_close(self, send_email):
         """ Test the request_pull endpoint with a closed PR. """
@@ -1792,6 +1813,13 @@ index 0000000..2a552bb
                 'A commit on branch feature',
             ])
 
+            # Check if the closing notification was added
+            output = self.app.get('/test/pull-request/1')
+            self.assertIn(
+                '<div>\n                '
+                'Pull-Request has been merged by pingou\n                '
+                '<div class="pull-xs-right text-muted">', output.data)
+
 
 if __name__ == '__main__':
     SUITE = unittest.TestLoader().loadTestsFromTestCase(PagureFlaskForktests)