Check if `installers` argument is provided
`installers` argument is optional. It means that user doesn't need to provide it. However, due to its default "None" value, without giving anything, list of skipped tests is empty. Updated code to allow to see skipped tests without provided `installers`. Change-Id: I88070aafc8378c3c418365840aa6694bdfcf6eac
This commit is contained in:
parent
fcf64e7702
commit
fc959e9f01
|
@ -51,10 +51,11 @@ class ListSkippedYaml(Lister):
|
|||
if [release for release in test.get('releases', [])
|
||||
if release['name'] == parsed_args.release]]
|
||||
|
||||
tests = [test for test in tests
|
||||
if [release for release in test.get('releases', [])
|
||||
if parsed_args.installer in
|
||||
release.get('installers', ['tripleo', 'osp'])]]
|
||||
if parsed_args.installer:
|
||||
tests = [test for test in tests
|
||||
if [release for release in test.get('releases', [])
|
||||
if parsed_args.installer in
|
||||
release.get('installers', ['tripleo', 'osp'])]]
|
||||
|
||||
if parsed_args.deployment:
|
||||
tests = [test for test in tests
|
||||
|
@ -83,8 +84,8 @@ class ListSkippedYaml(Lister):
|
|||
help='List the tests to be skipped in the '
|
||||
'given deployment')
|
||||
parser.add_argument('--installer', dest='installer',
|
||||
default=None, help='Tests to be skipped for a '
|
||||
'particular installer. Use '
|
||||
default='tripleo', help='Tests to be skipped for '
|
||||
'a particular installer. Use '
|
||||
'tripleo for upstream, and osp'
|
||||
' for downstream')
|
||||
return parser
|
||||
|
|
Loading…
Reference in New Issue