-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Failed unit test #9
Comments
Will investigate right now! |
This is due to the fact that the some tests mock the radar data: Lines 97 to 98 in 104dd6d
but not the actual configuration, so production config is used. Since in this fork, the I was not able to just quickly overwrite configuration in the test files, since this config is loaded at a later stage by the Vue components via VueX config store. The proper way to solve this is to mock the config file / config store for the tests, so they are not dependent anymore of the specific install configuration, which should be done in upstream (INBO) CROW. |
Merging the changes from #8 broke one unit test:
Ping @niconoe
The text was updated successfully, but these errors were encountered: