fix tc tests with the web worker
Current testing utilities don't like webworker scripts, even though my tests are running in real browsers.
The issue might be mocha or enzyme or karma
- Show closed items
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Jaden DIEFENBAUGH added medium label
added medium label
- Jaden DIEFENBAUGH added hard and removed medium labels
- Jaden DIEFENBAUGH added blocked label
added blocked label
- Author Contributor
Maybe a way to do this is to conditionally export stuff based on whether or not karma is running.
Basically, detect if
window.karma
orwindow.__karma__
(not sure which one works now) exists. If it does exist, export some single-threaded stuff. If it doesn't, export the multi-threaded stuff. Could even export a flag so the code uses it a certain way.Hopefully there's a better solution when I have more time
- Jaden DIEFENBAUGH added ~1720 and removed blocked labels
added ~1720 and removed blocked labels
- Jaden DIEFENBAUGH changed milestone to %JS & Python Testing
changed milestone to %JS & Python Testing
- Jaden DIEFENBAUGH added in-progress label
added in-progress label
- Jaden DIEFENBAUGH mentioned in commit 6340fcfd
mentioned in commit 6340fcfd
- Jaden DIEFENBAUGH added blocked and removed in-progress labels
added blocked and removed in-progress labels
- Jaden DIEFENBAUGH mentioned in commit 05b00c73
mentioned in commit 05b00c73
- Jaden DIEFENBAUGH mentioned in commit 62a487b4
mentioned in commit 62a487b4
- Author Contributor
This has not just been fixed, I've also added tests that account for state updates and actually using the component that uses the worker stuff!
- Jaden DIEFENBAUGH mentioned in commit 3bd52765
mentioned in commit 3bd52765
- Flavio TARSETTI mentioned in commit 57b8dd0d
mentioned in commit 57b8dd0d
- Flavio TARSETTI closed via merge request !10 (merged)
closed via merge request !10 (merged)
- Flavio TARSETTI mentioned in commit 4664e022
mentioned in commit 4664e022