From d82c52da7a0e0b5504203775ccdf30998777715f Mon Sep 17 00:00:00 2001 From: Frederik Gladhorn Date: Wed, 26 Jun 2019 10:45:55 +0200 Subject: Simplify jira bot: we now always get an event instance Change-Id: Ifbbad62d0068e6955fe547c1bbf8a1e895a0b140 Reviewed-by: Cristian Maureira-Fredes Reviewed-by: Daniel Smith Reviewed-by: Frederik Gladhorn --- scripts/jira/jira-bug-closer/bot/bot.py | 5 ++--- scripts/jira/jira-bug-closer/gerrit/streamparser.py | 3 +-- 2 files changed, 3 insertions(+), 5 deletions(-) (limited to 'scripts') diff --git a/scripts/jira/jira-bug-closer/bot/bot.py b/scripts/jira/jira-bug-closer/bot/bot.py index e22c4045..2da80d66 100644 --- a/scripts/jira/jira-bug-closer/bot/bot.py +++ b/scripts/jira/jira-bug-closer/bot/bot.py @@ -62,9 +62,8 @@ class Bot: async def event_handler(self, data: str) -> None: event = self.parser.parse(data) - log.debug(event) - log.debug("Raw data: >>>%s<<<", data) - if event and event.is_branch_update(): + log.debug(f"Gerrit Event: {event} - raw data: >>>{data}<<<") + if event.is_branch_update(): log.info(event) await self.update_project(event.project) diff --git a/scripts/jira/jira-bug-closer/gerrit/streamparser.py b/scripts/jira/jira-bug-closer/gerrit/streamparser.py index e9475c3b..c7b071d8 100644 --- a/scripts/jira/jira-bug-closer/gerrit/streamparser.py +++ b/scripts/jira/jira-bug-closer/gerrit/streamparser.py @@ -28,7 +28,6 @@ ############################################################################# import json -from typing import Optional from logger import logger log = logger(name="gerrit_stream_parser") @@ -57,7 +56,7 @@ class GerritEvent: class GerritStreamParser: - def parse(self, data: str) -> Optional[GerritEvent]: + def parse(self, data: str) -> GerritEvent: try: event = json.loads(data) except json.decoder.JSONDecodeError: -- cgit v1.2.3