tests: rw: Add bug.{close,addcomment,setstatus} tests

crobinso · crobinso · commit 2e4c0a0427b6 · 2016-05-24T10:29:35.000-04:00
diff --git a/bugzilla/bug.py b/bugzilla/bug.py
@@ -205,6 +205,7 @@ def close(self, resolution, dupeid=None, fixedin=None,
         You can optionally add a comment while closing the bug. Set 'isprivate'
           to True if you want that comment to be private.
         '''
+        # Note: fedora bodhi uses this function
         ignore = private_in_it
         ignore = nomail
 
@@ -283,6 +284,7 @@ def addcomment(self, comment, private=False,
         Add the given comment to this bug. Set private to True to mark this
         comment as private.
         '''
+        # Note: fedora bodhi uses this function
         ignore = timestamp
         ignore = bz_gid
         ignore = worktime
diff --git a/tests/rw_functional.py b/tests/rw_functional.py
@@ -241,13 +241,28 @@ def test5ModifyStatus(self):
         self.assertEquals(len(bug.longdescs), desclen + 1)
         self.assertTrue("marked as a duplicate" in bug.longdescs[-1]["text"])
 
-        # Add lone comment
+        # bz.setstatus test
         comment = ("adding lone comment at %s" % datetime.datetime.today())
-        tests.clicomm(cmd + "--comment \"%s\" --private" % comment, bz)
-
+        bug.setstatus("POST", comment=comment, private=True)
         bug.refresh()
         self.assertEquals(bug.longdescs[-1]["is_private"], 1)
         self.assertEquals(bug.longdescs[-1]["text"], comment)
+        self.assertEquals(bug.status, "POST")
+
+        # bz.close test
+        fixed_in = str(datetime.datetime.today())
+        bug.close("ERRATA", fixedin=fixed_in)
+        bug.refresh()
+        self.assertEquals(bug.status, "CLOSED")
+        self.assertEquals(bug.resolution, "ERRATA")
+        self.assertEquals(bug.fixed_in, fixed_in)
+
+        # bz.addcomment test
+        comment = ("yet another test comment %s" % datetime.datetime.today())
+        bug.addcomment(comment, private=False)
+        bug.refresh()
+        self.assertEquals(bug.longdescs[-1]["text"], comment)
+        self.assertEquals(bug.longdescs[-1]["is_private"], 0)
 
         # Reset state
         tests.clicomm(cmd + "--status %s" % origstatus, bz)
