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

Don't call `snap list` on the machine running unit tests #323

Merged

mateoflorido merged 1 commit into main from akd/mock-snap-list-calls-to-unittest-machine

Dec 14, 2023

Member

addyess commented Dec 14, 2023 •

edited

Loading

Found that unit tests depended on the result of a live test from the host machine's snap list <app>.

Update the call to is_snap_installed so the mock returns False


          Don't call snap list on the machine running unit tests

898aa75

mateoflorido approved these changes

View reviewed changes

mateoflorido merged commit e06fb83 into main

7 checks passed

mateoflorido deleted the akd/mock-snap-list-calls-to-unittest-machine branch

December 14, 2023 20:03

kwmonroe pushed a commit that referenced this pull request


          Don't call snap list on the machine running unit tests (#323)

66ca873

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

Labels

None yet