Changing require to pig latin breaking tests

[esri]

Changing require to pig latin breaking tests

[skitterm]

That fixes it for me, thanks! Let me know when it's released.

[tomwayson]

@skitterm

As we discussed this is due to the require() statements that ember-cli-coverage injects into tests/index.html.

I think we can solve this by adding 'tests/index.html' to the files array in index.js. I did some quick tests locally using the dummy app, and that seemed to work, but I did not try specifically using ember-cli-coverage. If this is still something you need to solve, would you mind trying that in your app (i.e. npm link to this addon)?

[skitterm]

I can give that a try. Though I didn't see the tests/index.html in that file you linked to. Would it be like the 5th entry below?

files: [
        new RegExp(path.parse(outputPaths.app.js).name + '(.*js)'),
        new RegExp(path.parse(outputPaths.vendor.js).name + '(.*js)'),
        new RegExp(path.parse(outputPaths.tests.js).name + '(.*js)'),
        new RegExp(path.parse(outputPaths.testSupport.js.testSupport).name + '(.*js)'),
        new RegExp('tests/index.html') // added this line
      ],

[tomwayson]

Yeah,

You don't even need to make it a regEx. I tested w/

files: [
        new RegExp(path.parse(outputPaths.app.js).name + '(.*js)'),
        new RegExp(path.parse(outputPaths.vendor.js).name + '(.*js)'),
        new RegExp(path.parse(outputPaths.tests.js).name + '(.*js)'),
        new RegExp(path.parse(outputPaths.testSupport.js.testSupport).name + '(.*js)'),
        'tests/index.html' // this should work too
      ],