Convert note-grain-on tests to testharness

third_party/WebKit/LayoutTests/webaudio/resources/note-grain-on-testing.js

 var sampleRate = 44100.0;

[hongchan, 2017/01/20 17:46:10]

var => let (all instances in this file)

[Raymond Toy, 2017/01/20 19:37:10]

I was avoiding that to make the diff smaller, but ok.

 
 // HRTF extra frames.  This is a magic constant currently in
 // AudioBufferSourceNode::process that always extends the
 // duration by this number of samples.  See bug 77224
 // (https://bugs.webkit.org/show_bug.cgi?id=77224).
 var extraFramesHRTF = 512;
 
 // How many grains to play.
 var numberOfTests = 100;
 
@@ skipping 87 matching lines @@
         playGrain(context, source, timeOffset, grainOffset, duration);
         startTimes[k] = timeOffset;
         offsets[k] = grainOffset;
     }
 
     return { startTimes : startTimes, grainOffsetTimes : offsets };
 }
 
 // Verify that the start and end frames for each grain match our
 // expected start and end frames.
-function verifyStartAndEndFrames(startEndFrames) {
+function verifyStartAndEndFrames(startEndFrames, should) {
     var startFrames = startEndFrames.start;
     var endFrames = startEndFrames.end;
       
-    var success = true;
-
     // Count of how many grains started at the incorrect time.
     var errorCountStart = 0;
 
     // Count of how many grains ended at the incorrect time.
     var errorCountEnd = 0;
 
-    if (startFrames.length != endFrames.length) {
-        testFailed("Could not find the beginning or end of a grain.");
-        success = false;
-    }
+    should(startFrames.length == endFrames.length,
+           "Found all grain starts and ends").beTrue();
 
-    if (startFrames.length == numberOfTests && endFrames.length == numberOfTests) {
-        testPassed("Found all " + numberOfTests + " grains.");
-    } else {
-        testFailed("Did not find all " + numberOfTests + " grains.");
-    }
+    should(startFrames.length, "Number of start frames").beEqualTo(numberOfTests);

[hongchan, 2017/01/20 17:46:10]

Wrap at 80.

[Raymond Toy, 2017/01/20 19:37:10]

Done.

+    should(endFrames.length, "Number of end frames").beEqualTo(numberOfTests);
 
     // Examine the start and stop times to see if they match our
     // expectations.
     for (var k = 0; k < startFrames.length; ++k) {
         var expectedStart = timeToSampleFrame(k * timeStep, sampleRate);
         // The end point is the duration, plus the extra frames
         // for HRTF.
         var expectedEnd = extraFramesHRTF + expectedStart + grainLengthInSampleFrames(k * grainOffsetStep, duration, sampleRate);

[hongchan, 2017/01/20 17:46:09]

Let's wrap at 80.

[Raymond Toy, 2017/01/20 19:37:10]

Done.

 
-        if (startFrames[k] != expectedStart) {
-            testFailed("Pulse " + k + " started at " + startFrames[k] + " but expected at " + expectedStart);
-            ++errorCountStart;
-            success = false;
-        }
+        if (startFrames[k] != expectedStart)
+          ++errorCountStart;
+        if (endFrames[k] != expectedEnd)
+          ++errorCountEnd;
 
-        if (endFrames[k] != expectedEnd) {
-            testFailed("Pulse " + k + " ended at " + endFrames[k] + " but expected at " + expectedEnd);
-            ++errorCountEnd;
-            success = false;
-        }
+        should([startFrames[k], endFrames[k]], "Pulse " + k + " boundary")
+            .beEqualToArray([expectedStart, expectedEnd]);
     }
 
     // Check that all the grains started or ended at the correct time.
     if (!errorCountStart) {
-        if (startFrames.length == numberOfTests) {
-            testPassed("All " + numberOfTests + " grains started at the correct time.");
-        } else {
-            testFailed("All grains started at the correct time, but only " + startFrames.length + " grains found.");
-            success = false;
-        }
+      should(startFrames.length,
+             "Number of grains that started at the correct time")
+          .beEqualTo(numberOfTests);
     } else {
-        testFailed(errorCountStart + " out of " + numberOfTests + " grains started at the wrong time.");
-        success = false;
+      should(errorCountStart,
+             "Number of grains out of " + numberOfTests + "that started at the wrong time")

[hongchan, 2017/01/20 17:46:10]

This line needs to be wrapped.

[Raymond Toy, 2017/01/20 19:37:10]

Done.

+          .beEqualTo(0);
     }
 
     if (!errorCountEnd) {
-        if (endFrames.length == numberOfTests) {
-            testPassed("All " + numberOfTests + " grains ended at the correct time.");
-        } else {
-            testFailed("All grains ended at the correct time, but only " + endFrames.length + " grains found.");
-            success = false;
-        }
+      should(endFrames.length,
+             "Number of grains that ended at the correct time")
+          .beEqualTo(numberOfTests);
     } else {
-        testFailed(errorCountEnd + " out of " + numberOfTests + " grains ended at the wrong time.");
-        success = false;
+      should(errorCountEnd,
+             "Number of grains out of " + numberOfTests + " that ended at the wrong time")

[hongchan, 2017/01/20 17:46:10]

Ditto.

[Raymond Toy, 2017/01/20 19:37:10]

Done.

+          .beEqualTo(0);
     }
-
-    return success;
 }