Skip to content

Conversation

ricab
Copy link
Collaborator

@ricab ricab commented Mar 16, 2020

Fixes #1429.


Manual test examples (beyond unit tests):

  • Daemon :
$ ./bin/multipassd --logger=stderr -V debug  # non-admin user, no sudo
  • CLI client:
$ chmod -r ~/.local/share/multipass/client-certificate/multipass_cert.pem  # don't forget to revert
$ multipass
  • GUI client:
$ echo asdf > ~/.config/multipass/multipass.conf  # don't forget to revert
$ multipass.gui

@multipass-ci-bot

This comment has been minimized.

@codecov
Copy link

codecov bot commented Mar 16, 2020

Codecov Report

Merging #1430 into master will increase coverage by 0.02%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1430      +/-   ##
==========================================
+ Coverage   73.21%   73.24%   +0.02%     
==========================================
  Files         208      209       +1     
  Lines        7549     7557       +8     
==========================================
+ Hits         5527     5535       +8     
  Misses       2022     2022
Impacted Files Coverage Δ
include/multipass/top_catch_all.h 100% <100%> (ø)
include/multipass/logging/cstring.h 100% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f13243a...abd64e8. Read the comment docs.

@Saviq Saviq requested a review from luis4a0 March 17, 2020 13:08
Copy link
Contributor

@luis4a0 luis4a0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code is clear and seems to be in the correct place (in a new file). I tested it works well. The tests cover much cases, including the std::runtime_error, which is the most commonly raised exception.

I'll approve the PR, leaving the comments in the code as suggestions.

@luis4a0
Copy link
Contributor

luis4a0 commented Mar 17, 2020

bors r+

@multipass-ci-bot
Copy link
Collaborator

multipass-ci-bot commented Mar 17, 2020

macOS build available: multipass-1.2.0-dev.95.pr1430+g59799500.mac-Darwin.pkg
Snap build available: snap refresh multipass --channel edge/pr1430

@bors
Copy link
Contributor

bors bot commented Mar 17, 2020

Build succeeded

@bors bors bot merged commit f93497b into master Mar 17, 2020
@bors bors bot deleted the top-level-catch-all branch March 17, 2020 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Multipass needs a top-level catch-all
3 participants