clusterfuzz
clusterfuzz copied to clipboard
Update jira to only add watchers that are missing
There are two issues that this PR fixes:
- When self.client.add_watcher(issue.jira_issue, watcher) fails previously, it'd throw an exception resulting in the entire .save or .create call to fail, which would retry continuously hammering the jira instance.
- The function would always try to add watchers to issues regardless if the watchers were already added previously.
@jonathanmetzman would you be able to take a look whenever you have the time?
Is there any unit tests that can be added ?
@javanlacerda added some unit tests here
@javanlacerda could you take another look whenever you have the chance?