diff --git a/repoapi/models/wni.py b/repoapi/models/wni.py index ba63781..c665a5c 100644 --- a/repoapi/models/wni.py +++ b/repoapi/models/wni.py @@ -76,7 +76,7 @@ def workfront_note_add(instance, message): gerrit_change=change, eventtype=eventtype) if created: - if not utils.workfront_note_send(wid, "%s %s" % (message, url)): + if not utils.workfront_note_send(wid, "%s %s " % (message, url)): logger.error("remove releated WorkfrontNoteInfo") note.delete() diff --git a/repoapi/test/test_workfrontnote.py b/repoapi/test/test_workfrontnote.py index 9c5385d..e304a7d 100644 --- a/repoapi/test/test_workfrontnote.py +++ b/repoapi/test/test_workfrontnote.py @@ -83,7 +83,7 @@ class WorkfrontNoteTestCase(BaseTest): workfront_id="0001", gerrit_change="2054") self.assertEquals(gri.count(), 1) - msg = "[%s] review created %s" % ( + msg = "[%s] review created %s " % ( param['param_branch'], settings.GERRIT_URL.format("2054")) utils.assert_called_once_with("0001", msg) @@ -111,7 +111,7 @@ class WorkfrontNoteTestCase(BaseTest): workfront_id="0001", gerrit_change="7fg4567") self.assertEquals(gri.count(), 0) - msg = "[%s] review created %s" % ( + msg = "[%s] review created %s " % ( param['param_branch'], settings.GERRIT_URL.format("2054")) utils.assert_called_once_with("0001", msg) @@ -130,7 +130,7 @@ class WorkfrontNoteTestCase(BaseTest): workfront_id="0001", gerrit_change="2054") self.assertEquals(gri.count(), 2) - msg = "[%s] review merged %s" % ( + msg = "[%s] review merged %s " % ( param['param_branch'], settings.GERRIT_URL.format("2054")) utils.assert_called_with("0001", msg) @@ -154,7 +154,7 @@ class WorkfrontNoteTestCase(BaseTest): workfront_id="0001", gerrit_change="7fg4567") self.assertEquals(gri.count(), 1) - msg = "[%s] commit created %s" % ( + msg = "[%s] commit created %s " % ( param['param_branch'], settings.GITWEB_URL.format("kamailio", "7fg4567")) utils.assert_called_once_with("0001", msg)