You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Dec 28, 2023. It is now read-only.
If unit tests are loaded from modules (which is supported via the files configuration option) then __karma__.start() should really be delayed until after all the modules have been loaded, to give unit tests the ability to load before the start event.
I'm not sure about all test frameworks, but at least for jasmine it is necessary, as you cannot add tests to it, after it has started executing all it's configured tests.
<scripttype="module">window.__karma__.loaded();</script><scriptnomoduletype="text/javascript">alert("your browser doesn't support modules, use a different one")</script>
Not quite sure what the best way to patch this in would be, but it seems to work great for me in local tests.
So: are there any downsides to this approach? How would you be willing to take this as a pull request?
The text was updated successfully, but these errors were encountered:
dwt
changed the title
Delay __karma__.start() until test modules have been loaded
Delay __karma__.start() until es6 test modules have been loaded
Sep 27, 2018
If unit tests are loaded from modules (which is supported via the
files
configuration option) then__karma__.start()
should really be delayed until after all the modules have been loaded, to give unit tests the ability to load before the start event.I'm not sure about all test frameworks, but at least for jasmine it is necessary, as you cannot add tests to it, after it has started executing all it's configured tests.
So with a configuration like this:
I would really like the context.html to read
instead of
which it is currently executing.
Not quite sure what the best way to patch this in would be, but it seems to work great for me in local tests.
So: are there any downsides to this approach? How would you be willing to take this as a pull request?
The text was updated successfully, but these errors were encountered: