openstack/election
Code Issues Proposed changes

Merge "Don't validate .placeholder"

Browse Source
This commit is contained in:
Zuul 2019-01-14 14:07:56 +00:00 committed by Gerrit Code Review
commit 5ab6e158c1
2 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
openstack_election/cmds/ci_check_all_candidate_files.py
View File

@ -84,7 +84,8 @@ def find_modified_candidate_files():
filenames = [
l.strip()
for l in results.splitlines()
if l.startswith(utils.CANDIDATE_PATH + '/')
if (l.startswith(utils.CANDIDATE_PATH + '/') and
not l.endswith('.placeholder'))
]
return filenames

6
openstack_election/tests/cmds/test_ci_checks.py
View File

@ -35,7 +35,7 @@ class TestFindModifiedCandidateFiles(base.ElectionTestCase):
check_output = (('configuration.yaml\n'
'%(path)s/.placeholder\n'
) % ({'path': utils.CANDIDATE_PATH})).encode('utf-8')
expected_filenames = ['%s/.placeholder' % utils.CANDIDATE_PATH]
expected_filenames = []
mock_check_output.return_value = check_output
filenames = \
ci_check_all_candidate_files.find_modified_candidate_files()
@ -48,9 +48,7 @@ class TestFindModifiedCandidateFiles(base.ElectionTestCase):
'%(path)s/cadidate\n'
'%(path)s/.placeholder\n'
) % ({'path': utils.CANDIDATE_PATH})).encode('utf-8')
expected_filenames = ['%s/cadidate' % utils.CANDIDATE_PATH,
'%s/.placeholder' % utils.CANDIDATE_PATH
]
expected_filenames = ['%s/cadidate' % utils.CANDIDATE_PATH]
mock_check_output.return_value = check_output
filenames = \
ci_check_all_candidate_files.find_modified_candidate_files()