[wpt-importer] Include Pri and Type when filing bugs

These two fields have long been required and are now enforced in
Chromium. Note that in the API these are represented as labels with
specific formats.

Also add "Test-WebTest" label by the way.

Bug: 1132456
Change-Id: If8c226472f62a563a6f25f393e9dda563970de84
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2432055
Commit-Queue: Stephen McGruer <smcgruer@chromium.org>
Reviewed-by: Stephen McGruer <smcgruer@chromium.org>
Auto-Submit: Robert Ma <robertma@chromium.org>
Cr-Commit-Position: refs/heads/master@{#810859}

third_party/blink/tools/blinkpy/w3c/import_notifier.py

@@ -223,8 +223,11 @@ class ImportNotifier(object):
 
             description = prologue + failure_list + epilogue + commit_list
 
-            bug = MonorailIssue.new_chromium_issue(summary, description, cc,
-                                                   components)
+            bug = MonorailIssue.new_chromium_issue(summary,
+                                                   description,
+                                                   cc,
+                                                   components,
+                                                   labels=['Test-WebTest'])
             _log.info(unicode(bug))
 
             if is_wpt_notify_enabled:

third_party/blink/tools/blinkpy/w3c/monorail.py

@@ -66,22 +66,35 @@ class MonorailIssue(object):
         return self._body
 
     @staticmethod
-    def new_chromium_issue(summary, description='', cc=None, components=None):
+    def new_chromium_issue(summary,
+                           description='',
+                           cc=None,
+                           components=None,
+                           priority='3',
+                           type='Bug',
+                           labels=None):
         """Creates a minimal new Chromium issue.
 
+        Chromium requires at least summary, priority and type: you must provide
+        the summary, whereas priority defaults to 3 and type defaults to Bug.
+
         Args:
             summary: The summary line.
             description: The issue description.
             cc: A list of email addresses to CC.
             components: A list of components.
+            priority: A string, defaults to '3'.
+            type: A string, defaults to 'Bug'.
+            labels: A list of labels (strings).
         """
-        return MonorailIssue(
-            'chromium',
-            summary=summary,
-            description=description,
-            cc=cc or [],
-            components=components or [],
-            status='Untriaged')
+        return MonorailIssue('chromium',
+                             summary=summary,
+                             description=description,
+                             cc=cc or [],
+                             components=components or [],
+                             status='Untriaged',
+                             labels=['Pri-' + priority, 'Type-' + type] +
+                             (labels or []))
 
     @staticmethod
     def crbug_link(issue_id):

third_party/blink/tools/blinkpy/w3c/monorail_unittest.py

@@ -80,16 +80,18 @@ class MonorailIssueTest(unittest.TestCase):
                 'chromium', summary='test', status='Untriaged', labels='Flaky')
 
     def test_new_chromium_issue(self):
-        issue = MonorailIssue.new_chromium_issue(
-            'test',
-            description='body',
-            cc=['foo@chromium.org'],
-            components=['Infra'])
+        issue = MonorailIssue.new_chromium_issue('test',
+                                                 description='body',
+                                                 cc=['foo@chromium.org'],
+                                                 components=['Infra'],
+                                                 labels=['Test-WebTest'])
         self.assertEqual(issue.project_id, 'chromium')
         self.assertEqual(issue.body['summary'], 'test')
         self.assertEqual(issue.body['description'], 'body')
         self.assertEqual(issue.body['cc'], ['foo@chromium.org'])
         self.assertEqual(issue.body['components'], ['Infra'])
+        self.assertEqual(issue.body['labels'],
+                         ['Pri-3', 'Type-Bug', 'Test-WebTest'])
 
     def test_crbug_link(self):
         self.assertEqual(