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.

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

Add tests for launcher/UpdaterApp.py #649

Closed
emkll opened this issue Nov 20, 2020 · 1 comment
Closed

Add tests for launcher/UpdaterApp.py #649

emkll opened this issue Nov 20, 2020 · 1 comment
Assignees

Comments

@emkll
Copy link
Contributor

emkll commented Nov 20, 2020

Related to #646

We currently, by policy, targeting 100% test coverage for Updater.py, which contains most of updater logic.

While UpdaterApp.py primarily handles the UI, we should still consider adding test coverage there to prevent regressions, given that we are introducing new functionality and further complexity (e.g. handling migrations)

@zenmonkeykstop
Copy link
Contributor

Test coverage has been increased from 2022 on. Closing, but test coverage for new launcher should also be at acceptable levels.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants