Fix confirm() util true state
The confirm() util incorrectly returns true only when the answer is false. This patch reverses the logic so that it properly returns true when the user inputs a true statement. Include test case for delete-all command usage to accept or abort. Mocking the input builtin ensures that the behaviour resulting from the user inputting 'y' or 'n' will remain tested for delete-all. Change-Id: I742804a8efe7da8ca6aa70f20059d02758910deb Co-Authored-By: Darragh Bailey <dbailey@hpe.com> Signed-off-by: Thanh Ha <thanh.ha@linuxfoundation.org>
This commit is contained in:
parent
0ece25ced4
commit
fd01e9e916
@ -66,4 +66,4 @@ def recurse_path(root, excludes=None):
|
||||
|
||||
def confirm(question):
|
||||
answer = input('%s (Y/N): ' % question).upper().strip()
|
||||
return not answer == 'Y'
|
||||
return answer == 'Y'
|
||||
|
53
tests/cmd/subcommands/test_delete_all.py
Normal file
53
tests/cmd/subcommands/test_delete_all.py
Normal file
@ -0,0 +1,53 @@
|
||||
# - Copyright 2016 Hewlett-Packard Development Company, L.P.
|
||||
#
|
||||
# Licensed under the Apache License, Version 2.0 (the "License");
|
||||
# you may not use this file except in compliance with the License.
|
||||
# You may obtain a copy of the License at
|
||||
#
|
||||
# http://www.apache.org/licenses/LICENSE-2.0
|
||||
#
|
||||
# Unless required by applicable law or agreed to in writing, software
|
||||
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
|
||||
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
|
||||
# License for the specific language governing permissions and limitations
|
||||
# under the License.
|
||||
|
||||
# The goal of these tests is to check that given a particular set of flags to
|
||||
# Jenkins Job Builder's command line tools it will result in a particular set
|
||||
# of actions by the JJB library, usually through interaction with the
|
||||
# python-jenkins library.
|
||||
|
||||
import six
|
||||
|
||||
from tests.base import mock
|
||||
from tests.cmd.test_cmd import CmdTestsBase
|
||||
|
||||
|
||||
@mock.patch('jenkins_jobs.builder.JenkinsManager.get_plugins_info',
|
||||
mock.MagicMock)
|
||||
class DeleteAllTests(CmdTestsBase):
|
||||
|
||||
input_module = "jenkins_jobs.utils" if six.PY2 else "builtins"
|
||||
|
||||
@mock.patch('jenkins_jobs.cli.subcommand.update.'
|
||||
'JenkinsManager.delete_all_jobs')
|
||||
def test_delete_all_accept(self, delete_job_mock):
|
||||
"""
|
||||
Test handling the deletion of a single Jenkins job.
|
||||
"""
|
||||
|
||||
args = ['--conf', self.default_config_file, 'delete-all']
|
||||
with mock.patch('%s.input' % self.input_module, return_value="y"):
|
||||
self.execute_jenkins_jobs_with_args(args)
|
||||
|
||||
@mock.patch('jenkins_jobs.cli.subcommand.update.'
|
||||
'JenkinsManager.delete_all_jobs')
|
||||
def test_delete_all_abort(self, delete_job_mock):
|
||||
"""
|
||||
Test handling the deletion of a single Jenkins job.
|
||||
"""
|
||||
|
||||
args = ['--conf', self.default_config_file, 'delete-all']
|
||||
with mock.patch('%s.input' % self.input_module, return_value="n"):
|
||||
self.assertRaises(SystemExit,
|
||||
self.execute_jenkins_jobs_with_args, args)
|
Loading…
Reference in New Issue
Block a user