Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Test suite use the real configuration #192

Open
niconoe opened this issue Apr 6, 2023 · 0 comments · Fixed by aloftdata/crow#11
Open

Test suite use the real configuration #192

niconoe opened this issue Apr 6, 2023 · 0 comments · Fixed by aloftdata/crow#11
Labels
bug Something isn't working 🛀 cleanup Code cleanup

Comments

@niconoe
Copy link
Collaborator

niconoe commented Apr 6, 2023

This was discovered here: aloftdata#9.

The test suite should fully mock the configuration (config file? Vue store?) so it doesn't depend of the state of config.ts, which is installation-specific.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working 🛀 cleanup Code cleanup
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Commented broken tests niconoe/crow
2 participants
@niconoe @peterdesmet