summaryrefslogtreecommitdiff
path: root/virtinst
diff options
context:
space:
mode:
authorCole Robinson <crobinso@redhat.com>2022-08-20 09:42:47 -0400
committerCole Robinson <crobinso@redhat.com>2022-08-20 09:47:49 -0400
commit1cb0be4002445e5755ead2423b5a4e9e06f0a3cb (patch)
treeb972e9694b230bc72f86670468aadef4d004bc10 /virtinst
parent1b87e3e54c782da39cf30b100a22f70c37bfcddd (diff)
downloadvirt-manager-1cb0be4002445e5755ead2423b5a4e9e06f0a3cb.tar.gz
virtinstall: split no_install conditional apart to track code coverage
Each bit here is part of the CLI API, we need to be sure we are covering each one. Extend the test suite to hit one case we are missing Signed-off-by: Cole Robinson <crobinso@redhat.com>
Diffstat (limited to 'virtinst')
-rw-r--r--virtinst/virtinstall.py14
1 files changed, 9 insertions, 5 deletions
diff --git a/virtinst/virtinstall.py b/virtinst/virtinstall.py
index 6d7f56b8..32434119 100644
--- a/virtinst/virtinstall.py
+++ b/virtinst/virtinstall.py
@@ -429,11 +429,15 @@ def build_installer(options, guest, installdata):
install_bootdev = "network"
elif installdata.is_set:
pass
- elif (options.import_install or
- options.xmlonly or
- options.boot or
- options.cloud_init or
- options.unattended):
+ elif options.xmlonly:
+ no_install = True
+ elif options.import_install:
+ no_install = True
+ elif options.boot:
+ no_install = True
+ elif options.cloud_init:
+ no_install = True
+ elif options.unattended:
no_install = True
installer = virtinst.Installer(guest.conn,