Quit with error message when no config key is provided. Fixes #4223

This commit is contained in:
Manu 2019-01-30 11:30:31 +08:00
parent e35091596d
commit 38e6ff0a54
2 changed files with 6 additions and 0 deletions

View File

@ -1807,6 +1807,10 @@ class Archiver:
print('%s = %s' % (key, value)) print('%s = %s' % (key, value))
if not args.list: if not args.list:
if args.name is None:
self.print_error('No config key name was provided.')
return self.exit_code
try: try:
section, name = args.name.split('.') section, name = args.name.split('.')
except ValueError: except ValueError:

View File

@ -2804,6 +2804,8 @@ id: 2 / e29442 3506da 4e1ea7 / 25f62a 5a3d41 - 02
self.cmd('config', '--delete', self.repository_location, cfg_key) self.cmd('config', '--delete', self.repository_location, cfg_key)
self.cmd('config', self.repository_location, cfg_key, exit_code=1) self.cmd('config', self.repository_location, cfg_key, exit_code=1)
self.cmd('config', '--list', '--delete', self.repository_location, exit_code=2) self.cmd('config', '--list', '--delete', self.repository_location, exit_code=2)
self.cmd('config', self.repository_location, exit_code=2)
self.cmd('config', self.repository_location, 'invalid-option', exit_code=1)
requires_gnutar = pytest.mark.skipif(not have_gnutar(), reason='GNU tar must be installed for this test.') requires_gnutar = pytest.mark.skipif(not have_gnutar(), reason='GNU tar must be installed for this test.')
requires_gzip = pytest.mark.skipif(not shutil.which('gzip'), reason='gzip must be installed for this test.') requires_gzip = pytest.mark.skipif(not shutil.which('gzip'), reason='gzip must be installed for this test.')