Garden-o-matic: Display bugs from the commit BUG=... line in commit summary

Tools/GardeningServer/scripts/model_unittests.js

 /*
  * Copyright (C) 2011 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 123 matching lines @@
             $.each(recentCommits, function(index, commitData) {
                 delete commitData.message;
             });
             deepEqual(recentCommits, [{
                 "revision": 3,
                 "title": "Throw SecurityError when setting 'Replaceable' properties cross-origin.",
                 "time": "2013-09-30T20:22:01Z",
                 "summary": "This matches Gecko's behavior for these types of properties.",
                 "author": "mkwst@chromium.org",
                 "reviewer": "jochen@chromium.org",
-                "bugID": 13,
+                "bugID": [13],
                 "revertedRevision": undefined,
               },
               {
                 "revision": 2,
                 "title": "Fix one more layering violation caught by check-blink-deps",
                 "time": "2013-09-30T19:36:21Z",
                 "summary": "core/platform may not depend on core/ even for testing.",
                 "author": "eseidel@chromium.org",
                 "reviewer": "abarth@chromium.org, abarth",
-                "bugID": 12,
+                "bugID": [12],
                 "revertedRevision": undefined
               },
               {
                 "revision": 1,
                 "title": "Update DEPS include_rules after addition of root-level platform directory",
                 "time": "2013-09-30T19:28:49Z",
                 "summary": "These were all failures noticed when running check-blink-deps",
                 "author": "eseidel@chromium.org",
                 "reviewer": "abarth@chromium.org, abarth",
-                "bugID": 11,
+                "bugID": [11],
                 "revertedRevision": undefined
               }
             ]);
         });
     }).then(start);
 });
 
 asyncTest("commitDataListForRevisionRange", 6, function() {
     var simulator = new NetworkSimulator();
 
     simulator.xml = function(url)
     {
         var parser = new DOMParser();
         var responseDOM = parser.parseFromString(kExampleCommitDataXML, "application/xml");
         return Promise.resolve(responseDOM);
     };
 
     simulator.runTest(function() {
         model.updateRecentCommits().then(function() {
             function extractBugIDs(commitData)
             {
                 return commitData.bugID;
             }
 
-            deepEqual(model.commitDataListForRevisionRange(3, 3).map(extractBugIDs), [13]);
-            deepEqual(model.commitDataListForRevisionRange(1, 3).map(extractBugIDs), [11, 12, 13]);
-            deepEqual(model.commitDataListForRevisionRange(0, 1).map(extractBugIDs), [11]);
-            deepEqual(model.commitDataListForRevisionRange(0, 4).map(extractBugIDs), [11, 12, 13]);
+            deepEqual(model.commitDataListForRevisionRange(3, 3).map(extractBugIDs), [[13]]);
+            deepEqual(model.commitDataListForRevisionRange(1, 3).map(extractBugIDs), [[11], [12], [13]]);
+            deepEqual(model.commitDataListForRevisionRange(0, 1).map(extractBugIDs), [[11]]);
+            deepEqual(model.commitDataListForRevisionRange(0, 4).map(extractBugIDs), [[11], [12], [13]]);
             deepEqual(model.commitDataListForRevisionRange(4, 0).map(extractBugIDs), []);
             delete model.state.recentCommits;
         });
     }).then(start);
 });
 
 test("buildersInFlightForRevision", 3, function() {
     var unmock = model.state.resultsByBuilder;
     model.state.resultsByBuilder = {
         'Mr. Beasley': {blink_revision: '5'},
@@ skipping 30 matching lines @@
         'Mr. Beasley': { },
         'Mr Dixon': {blink_revision: '2'},
         'Mr. Sabatini': {blink_revision: '4'},
         'Bob': {blink_revision: '6'}
     };
     equals(model.latestRevisionWithNoBuildersInFlight(), 2);
     model.state.resultsByBuilder = unmock;
 });
 
 })();