Automate building “wrapped” Benchmark.js

[mathiasbynens]

On jsPerf we have to use a special custom build of Benchmark.js to avoid creating more global variables than necessary (since it may interfere with people’s tests otherwise).

Historically we’ve used a very old/hacky/ugly build script for that: https://github.com/mathiasbynens/jsperf.com/blob/master/_tmp/build.php

The plan is to create a new standalone script:

• in its own repository: https://github.com/jsperf/build-benchmark.js-wrapper (this logic doesn’t belong in the main repository with jsperf.com server code)
• with proper tests to avoid issues like IE 9 not able to run test - Script error in benchmark-47.js #44, Receiving error on simple script #45, Uncaught TypeError: Cannot read property 'name' of undefined #53

cc @jdalton

[jdalton]

👏

[syedirfan7]

@mathiasbynens
i am new to this ...
is there a live way where i can avoid this error ?

Uncaught TypeError: Cannot read property 'name' of undefined.

message: Uncaught TypeError: Cannot read property 'name' of undefined
fileName: http://jsperf.com/benchmark-47.js
lineNumber: 21

[mathiasbynens]

Okay, the build is ready. What’s left now is to fetch https://jsperf.github.io/build-benchmark.js-wrapper/all.js and save it as public/_js/benchmark.js, preferably automatically on each build (since it’s really just a dependency with its own repo). I’d prefer not to have this file as part of the repository. @maxbeatty @asilluron what would be the best way to do this?

[maxbeatty]

You could publish to npm as a scoped package like @jsperf/build-benchmark.js-wrapper, add as dependency in package.json, and then create a special route like robots.txt

  server.route({
    method: 'GET',
    path: '/public/_js/benchmark.js',
    handler: {
      file: require('build-benchmark.js-wrapper')
    }
  });