From 2aa0ac5462c34a3b2ffb5c51f5d9cd0ff273509f Mon Sep 17 00:00:00 2001 From: Vsevolod Fedorov Date: Thu, 21 Mar 2024 14:07:20 +0300 Subject: [PATCH] Refactor 'update' cli command Change-Id: I4b253c30ed517bba38979b759082b37879af393e --- jenkins_jobs/cli/subcommand/update.py | 17 ++--------------- 1 file changed, 2 insertions(+), 15 deletions(-) diff --git a/jenkins_jobs/cli/subcommand/update.py b/jenkins_jobs/cli/subcommand/update.py index efb36abbf..1215524da 100644 --- a/jenkins_jobs/cli/subcommand/update.py +++ b/jenkins_jobs/cli/subcommand/update.py @@ -129,27 +129,14 @@ class UpdateSubCommand(base.JobsSubCommand): ) xml_jobs = xml_jobs_filtered - if options.update == "jobs": - jobs, num_updated_jobs = builder.update_jobs( - xml_jobs, - n_workers=options.n_workers, - existing_only=options.existing_only, - ) - logger.info("Number of jobs updated: %d", num_updated_jobs) - elif options.update == "views": - views, num_updated_views = builder.update_views( - xml_views, - n_workers=options.n_workers, - existing_only=options.existing_only, - ) - logger.info("Number of views updated: %d", num_updated_views) - else: + if options.update in {"jobs", "all"}: jobs, num_updated_jobs = builder.update_jobs( xml_jobs, n_workers=options.n_workers, existing_only=options.existing_only, ) logger.info("Number of jobs updated: %d", num_updated_jobs) + if options.update in {"views", "all"}: views, num_updated_views = builder.update_views( xml_views, n_workers=options.n_workers,