99fd084099
Also, add copyright notices to subcommand files and fix mock import paths in tests. Change-Id: Ia8a96d5d5c460387968437d86cf7144ef382b20c
153 lines
6.4 KiB
Python
153 lines
6.4 KiB
Python
# Joint copyright:
|
|
# - Copyright 2015 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 os
|
|
import six
|
|
|
|
from tests.base import mock
|
|
from tests.cmd.test_cmd import CmdTestsBase
|
|
|
|
|
|
@mock.patch('jenkins_jobs.builder.Jenkins.get_plugins_info', mock.MagicMock)
|
|
class UpdateTests(CmdTestsBase):
|
|
|
|
@mock.patch('jenkins_jobs.builder.jenkins.Jenkins.job_exists')
|
|
@mock.patch('jenkins_jobs.builder.jenkins.Jenkins.get_jobs')
|
|
@mock.patch('jenkins_jobs.builder.jenkins.Jenkins.reconfig_job')
|
|
def test_update_jobs(self,
|
|
jenkins_reconfig_job,
|
|
jenkins_get_jobs,
|
|
jenkins_job_exists, ):
|
|
"""
|
|
Test update_job is called
|
|
"""
|
|
path = os.path.join(self.fixtures_path, 'cmd-002.yaml')
|
|
args = ['--conf', self.default_config_file, 'update', path]
|
|
|
|
self.execute_jenkins_jobs_with_args(args)
|
|
|
|
jenkins_reconfig_job.assert_has_calls(
|
|
[mock.call(job_name, mock.ANY)
|
|
for job_name in ['bar001', 'bar002', 'baz001', 'bam001']],
|
|
any_order=True
|
|
)
|
|
|
|
@mock.patch('jenkins_jobs.builder.Jenkins.is_job', return_value=True)
|
|
@mock.patch('jenkins_jobs.builder.Jenkins.get_jobs')
|
|
@mock.patch('jenkins_jobs.builder.Jenkins.get_job_md5')
|
|
@mock.patch('jenkins_jobs.builder.Jenkins.update_job')
|
|
def test_update_jobs_decode_job_output(self, update_job_mock,
|
|
get_job_md5_mock, get_jobs_mock,
|
|
is_job_mock):
|
|
"""
|
|
Test that job xml output has been decoded before attempting to update
|
|
"""
|
|
# don't care about the value returned here
|
|
update_job_mock.return_value = ([], 0)
|
|
|
|
path = os.path.join(self.fixtures_path, 'cmd-002.yaml')
|
|
args = ['--conf', self.default_config_file, 'update', path]
|
|
|
|
self.execute_jenkins_jobs_with_args(args)
|
|
self.assertTrue(isinstance(update_job_mock.call_args[0][1],
|
|
six.text_type))
|
|
|
|
@mock.patch('jenkins_jobs.builder.jenkins.Jenkins.job_exists')
|
|
@mock.patch('jenkins_jobs.builder.jenkins.Jenkins.get_jobs')
|
|
@mock.patch('jenkins_jobs.builder.jenkins.Jenkins.reconfig_job')
|
|
@mock.patch('jenkins_jobs.builder.jenkins.Jenkins.delete_job')
|
|
def test_update_jobs_and_delete_old(self,
|
|
jenkins_delete_job,
|
|
jenkins_reconfig_job,
|
|
jenkins_get_jobs,
|
|
jenkins_job_exists, ):
|
|
"""
|
|
Test update behaviour with --delete-old option
|
|
|
|
* mock out a call to jenkins.Jenkins.get_jobs() to return a known list
|
|
of job names.
|
|
* mock out a call to jenkins.Jenkins.reconfig_job() and
|
|
jenkins.Jenkins.delete_job() to detect calls being made to determine
|
|
that JJB does correctly delete the jobs it should delete when passed
|
|
a specific set of inputs.
|
|
* mock out a call to jenkins.Jenkins.job_exists() to always return
|
|
True.
|
|
"""
|
|
jobs = ['old_job001', 'old_job002', 'unmanaged']
|
|
extra_jobs = [{'name': name} for name in jobs]
|
|
|
|
path = os.path.join(self.fixtures_path, 'cmd-002.yaml')
|
|
args = ['--conf', self.default_config_file, 'update', '--delete-old',
|
|
path]
|
|
|
|
jenkins_get_jobs.return_value = extra_jobs
|
|
|
|
with mock.patch('jenkins_jobs.builder.Jenkins.is_managed',
|
|
side_effect=(lambda name: name != 'unmanaged')):
|
|
self.execute_jenkins_jobs_with_args(args)
|
|
|
|
jenkins_reconfig_job.assert_has_calls(
|
|
[mock.call(job_name, mock.ANY)
|
|
for job_name in ['bar001', 'bar002', 'baz001', 'bam001']],
|
|
any_order=True
|
|
)
|
|
jenkins_delete_job.assert_has_calls(
|
|
[mock.call(name) for name in jobs if name != 'unmanaged'])
|
|
|
|
@mock.patch('jenkins_jobs.builder.jenkins.Jenkins')
|
|
def test_update_timeout_not_set(self, jenkins_mock):
|
|
"""Check that timeout is left unset
|
|
|
|
Test that the Jenkins object has the timeout set on it only when
|
|
provided via the config option.
|
|
"""
|
|
|
|
path = os.path.join(self.fixtures_path, 'cmd-002.yaml')
|
|
args = ['--conf', self.default_config_file, 'update', path]
|
|
|
|
import_path = 'jenkins_jobs.cli.subcommand.update.Builder.update_job'
|
|
with mock.patch(import_path) as update_mock:
|
|
update_mock.return_value = ([], 0)
|
|
self.execute_jenkins_jobs_with_args(args)
|
|
# unless the timeout is set, should only call with 3 arguments
|
|
# (url, user, password)
|
|
self.assertEqual(len(jenkins_mock.call_args[0]), 3)
|
|
|
|
@mock.patch('jenkins_jobs.builder.jenkins.Jenkins')
|
|
def test_update_timeout_set(self, jenkins_mock):
|
|
"""Check that timeout is set correctly
|
|
|
|
Test that the Jenkins object has the timeout set on it only when
|
|
provided via the config option.
|
|
"""
|
|
|
|
path = os.path.join(self.fixtures_path, 'cmd-002.yaml')
|
|
config_file = os.path.join(self.fixtures_path,
|
|
'non-default-timeout.ini')
|
|
args = ['--conf', config_file, 'update', path]
|
|
|
|
import_path = 'jenkins_jobs.cli.subcommand.update.Builder.update_job'
|
|
with mock.patch(import_path) as update_mock:
|
|
update_mock.return_value = ([], 0)
|
|
self.execute_jenkins_jobs_with_args(args)
|
|
# when timeout is set, the fourth argument to the Jenkins api init
|
|
# should be the value specified from the config
|
|
self.assertEqual(jenkins_mock.call_args[0][3], 0.2)
|