Publish ambient types

[gitKrystan]

Needed for emberjs/ember-qunit#994
See emberjs/ember.js#20162

[gitKrystan]

Based on the docs for this library, I'm not sure how much of this should actually be public. All I need for emberjs/ember-qunit#994 is loadModules, moduleLoadFailure, addModuleExcludeMatcher and addModuleIncludeMatcher.

[chancancode]

Should we comment out the rest and see if anyone asks for them?

[chancancode]

I guess at the end of the day this whole package is probably pretty "private" anyway 🤷🏼‍♂️

[chancancode]

Ok so looking at the implementation, I would guess that shouldLoadModule, listModules, listTestModules, require, unsee and moduleLoadFailure are probably meant to be private helpers.

[chancancode]

In that case, it may make more sense to declare the default export as an object with a load method on it? I don't think TestLoader is really meant to be instantiated or subclassed by anyone outside of here. The fact that it happens to be a class appears to be an implementation detail.

[chancancode]

OTOH, the readme says....

// optionally override TestLoader.prototype.shouldLoadModule

🤷🏼 🤷🏼 🤷🏼

I guess you can take all of that information and decide what to do with them as you see fit, this is just an "internal" support package and I don't think these things were thought through very rigorously at the time.

[chancancode]

ember-qunit does new TestLoader().loadModules()

It probably should have just called load() which does exactly that, but oh well

[gitKrystan]

lol true

[gitKrystan]

README says "optionally override TestLoader.prototype.shouldLoadModule" so I am going to make that public also

[gitKrystan]

I am working on ember-qunit anyway, so I just made that call the static load method and made loadModules private and the constructor protected.