Fix extract_from_see_also() with None values (#223)

Fix condition in extract_from_see_also()

Author: leplatrem

jbi/models.py

@@ -277,7 +277,7 @@ def issue_type(self) -> str:
 
     def extract_from_see_also(self):
         """Extract Jira Issue Key from see_also if jira url present"""
-        if not self.see_also and len(self.see_also) > 0:
+        if not self.see_also or len(self.see_also) == 0:
             return None
 
         for url in self.see_also:  # pylint: disable=not-an-iterable

jbi/runner.py

@@ -27,8 +27,8 @@ def execute_action(
     The value returned by the action call is returned.
     """
     log_context = RunnerLogContext(
-        bug=request.bug.dict(),
-        request=request.dict(),
+        bug=request.bug,
+        request=request,
         operation=Operation.HANDLE,
     )
     try:

tests/unit/test_bugzilla.py

@@ -32,6 +32,7 @@ def test_jira_labels(whiteboard, expected):
 @pytest.mark.parametrize(
     "see_also,expected",
     [
+        (None, None),
         ([], None),
         (["foo"], None),
         (["fail:/format"], None),