Merge ~pieq/bugit/+git/qabro:fix-1806832-bug-info-per-series into bugit:master

[Pierre Equoy]

See commit message for more information.

To test, you can create a snap, then run:

APPORT_LAUNCHPAD_INSTANCE=staging qabro --fwts -p lisbon -t toyota

This will create a bug report with an additional series "hwe-lisbon", assigned to Alex Hung. Both status and importance should be set in the "hwe-lisbon" series only.

You can also try the classic

APPORT_LAUNCHPAD_INSTANCE=staging qabro

and make sure things work as expected.

[Sylvain Pineau (sylvain-pineau)]

Just a minor pythonic advice, otherwise +1

[Pierre Equoy (pieq)]

Thanks Sylvain!

bug_tasks cannot use negative indexes unfortunately (I added a comment inline below). I'll mark this MR approved then.

diff --git a/qabro/bug_assistant.py b/qabro/bug_assistant.py
index 824dc5f..dd5fb87 100644
--- a/qabro/bug_assistant.py
+++ b/qabro/bug_assistant.py
@@ -143,21 +143,24 @@ class BugAssistant:
             print('Updating project...')
             task.target = project
             task.lp_save()
+
+        if series:
+            print('Setting series...')
+            nomination = self.bug.addNomination(target=series)
+            nomination.approve()
+
+        # We update bug info only for the latest created series
+        task = self.bug.bug_tasks[len(self.bug.bug_tasks)-1]
+        if assignee:
+            print('Setting assignee for series {}...').format(task.bug_target_name)
+            task.assignee = assignee
         print('Setting status...')
         task.status = self.lp_status
         print('Setting importance...')
         task.importance = self.lp_importance
-        if assignee:
-            print('Setting assignee...')
-            task.assignee = assignee
 
         task.lp_save()
 
-        if series:
-            print('Setting series...')
-            nomination = self.bug.addNomination(target=series)
-            nomination.approve()
-
         service_root = os.environ.get('APPORT_LAUNCHPAD_INSTANCE', 'production')
         if service_root == 'qastaging':
             bug_url = QASTAGING_WEB_ROOT+'bugs/{}'.format(self.bug.id)