fix: use exports.default

[aminya]

Based on the new ES modules standard (what new Babel expects), you should not mix module.exports = with module.exports.name = as one of them overwrites the other.

Fixes the issue with the new Babel:
AtomLinter/linter-eslint#1422

[aminya]

I also think this should not be called (?)

Suggested change

	exports.default = createRunner();
	exports.default = createRunner;

[UziTech]

form the docs:

Your test runner module must export a single function, which Atom will call within a new window to run your package's tests.

the default export is so people can use "atomTestRunner": "atom-jasmine3-test-runner" if they want to use the default configuration.

[UziTech]

Fixes the issue with the new Babel

What new babel? is Atom updating the version of babel that it is using in newer versions?

[UziTech]

This makes the default runner fail with testRunner is not a function

[aminya]

I need to test this more. The only way I could export my runner was by module.exports=

[UziTech]

atom/atom#21715 was added in Atom v1.56.0-beta0 so using a .default export won't work until that is stable.

That is why I have this note in the readme for writing tests in typescript.

⚠ NOTE: If you use a custom runner that file (custom-runner.js) must be a .js file.

[aminya]

Ah, I thought the default export fix is already in the stable version. Isn't it?

Atom's versioning is really getting annoying