Bug 1394957 - Fix logging of number of tests when running chunked reftests; r=ahal
authorGeoff Brown <gbrown@mozilla.com>
Wed, 30 Aug 2017 10:11:28 -0600
changeset 377819 01b5e496f4234fe73a62284eccae861772bd303d
parent 377818 5f6ae84f80cb56c071d9a9e5d8e07621ac1d8fe1
child 377820 1271f11e57e7715d892f98e2cadfc1e9c5f93106
push id50043
push userkwierso@gmail.com
push dateThu, 31 Aug 2017 03:03:23 +0000
treeherderautoland@04b6be50a252 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersahal
bugs1394957
milestone57.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1394957 - Fix logging of number of tests when running chunked reftests; r=ahal
layout/tools/reftest/reftest.jsm
--- a/layout/tools/reftest/reftest.jsm
+++ b/layout/tools/reftest/reftest.jsm
@@ -547,20 +547,21 @@ function StartTests()
             var testsPerChunk = tURLs.length / gTotalChunks;
             var start = Math.round((gThisChunk-1) * testsPerChunk);
             var end = Math.round(gThisChunk * testsPerChunk);
 
             // Map these indices onto the gURLs array. This avoids modifying the
             // gURLs array which prevents skipped tests from showing up in the log
             start = gThisChunk == 1 ? 0 : gURLs.indexOf(tURLs[start]);
             end = gThisChunk == gTotalChunks ? gURLs.length : gURLs.indexOf(tURLs[end + 1]) - 1;
-            gURLs = gURLs.slice(start, end);
 
             logger.info("Running chunk " + gThisChunk + " out of " + gTotalChunks + " chunks.  " +
                 "tests " + (start+1) + "-" + end + "/" + gURLs.length);
+
+            gURLs = gURLs.slice(start, end);
         }
 
         if (gShuffle) {
             if (gStartAfter !== undefined) {
                 logger.error("Can't resume from a crashed test when " +
                              "--shuffle is enabled, continue by shuffling " +
                              "all the tests");
                 DoneTests();