Skip to content

Commit

Permalink
add comment to screenshotgather test.
Browse files Browse the repository at this point in the history
  • Loading branch information
paulirish committed Jun 1, 2016
1 parent 8ecaafc commit 95a7581
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions test/src/gatherers/screenshots.js
Expand Up @@ -24,6 +24,9 @@ let screenshotsGather = new ScreenshotsGather();
describe('Screenshot gatherer', () => {
it('returns an artifact for a real trace', () => {
const traceData = require('../audits/performance/progressive-app.json');
// Currently this test must rely on knowing the phase hook for the gatherer.
// A little unfortunate, but we need a "run scheduler with this gatherer, this mocked driver,
// and this trace" test class to do that right
return screenshotsGather.postProfiling(undefined, {traceContents: traceData}).then(_ => {
assert.ok(Array.isArray(screenshotsGather.artifact));
assert.equal(screenshotsGather.artifact.length, 7);
Expand Down

0 comments on commit 95a7581

Please sign in to comment.