improve script

This commit is contained in:
Benoit Marty 2020-02-10 22:19:43 +01:00
parent a7274b9df0
commit 35ed22ab2b
1 changed files with 3 additions and 4 deletions

View File

@ -44,7 +44,6 @@ parser.add_argument('-b',
parser.add_argument('-e', parser.add_argument('-e',
'--expecting', '--expecting',
type=int, type=int,
default=-1,
help='the expected number of artifacts. If omitted, no check will be done.') help='the expected number of artifacts. If omitted, no check will be done.')
parser.add_argument('-d', parser.add_argument('-d',
'--directory', '--directory',
@ -93,8 +92,8 @@ print(" git commit message : \"%s\"" % data0.get('message'))
print(" build state : %s" % data0.get('state')) print(" build state : %s" % data0.get('state'))
if data0.get('state') != 'passed': if data0.get('state') != 'passed':
print("❌ Error, the build failed (state: %s)" % data0.get('state')) print("❌ Error, the build is in state '%s', and not 'passed'" % data0.get('state'))
exit(0) exit(1)
### Fetch artifacts list ### Fetch artifacts list
@ -110,7 +109,7 @@ data = json.loads(r.content.decode())
print(" %d artifact(s) found." % len(data)) print(" %d artifact(s) found." % len(data))
if args.expecting != -1 and args.expecting != len(data): if args.expecting is not None and args.expecting != len(data):
print("Error, expecting %d artifacts and found %d." % (args.expecting, len(data))) print("Error, expecting %d artifacts and found %d." % (args.expecting, len(data)))
exit(1) exit(1)