1
0
Fork 0

Small CLI cleanups

* yapf changes
* Fix the cformat test
* Make the normpath test work when run from /
* `qmk config`: Mark `--read-only` as arg_only
This commit is contained in:
skullY 2019-11-12 17:27:08 -08:00 committed by skullydazed
parent d1b6c11b7f
commit 79edb7c594
6 changed files with 4 additions and 5 deletions

View file

@ -12,7 +12,7 @@ def print_config(section, key):
cli.echo('%s.%s{fg_cyan}={fg_reset}%s', section, key, cli.config[section][key]) cli.echo('%s.%s{fg_cyan}={fg_reset}%s', section, key, cli.config[section][key])
@cli.argument('-ro', '--read-only', action='store_true', help='Operate in read-only mode.') @cli.argument('-ro', '--read-only', arg_only=True, action='store_true', help='Operate in read-only mode.')
@cli.argument('configs', nargs='*', arg_only=True, help='Configuration options to read or write.') @cli.argument('configs', nargs='*', arg_only=True, help='Configuration options to read or write.')
@cli.subcommand("Read and write configuration settings.") @cli.subcommand("Read and write configuration settings.")
def config(cli): def config(cli):

View file

@ -6,6 +6,7 @@ import glob
from milc import cli from milc import cli
@cli.subcommand("List the keyboards currently defined within QMK") @cli.subcommand("List the keyboards currently defined within QMK")
def list_keyboards(cli): def list_keyboards(cli):
"""List the keyboards currently defined within QMK """List the keyboards currently defined within QMK

View file

@ -1,6 +1,5 @@
class NoSuchKeyboardError(Exception): class NoSuchKeyboardError(Exception):
"""Raised when we can't find a keyboard/keymap directory. """Raised when we can't find a keyboard/keymap directory.
""" """
def __init__(self, message): def __init__(self, message):
self.message = message self.message = message

View file

@ -3,7 +3,6 @@ class AttrDict(dict):
This should only be used to mock objects for unit testing. Please do not use this outside of qmk.tests. This should only be used to mock objects for unit testing. Please do not use this outside of qmk.tests.
""" """
def __init__(self, *args, **kwargs): def __init__(self, *args, **kwargs):
super(AttrDict, self).__init__(*args, **kwargs) super(AttrDict, self).__init__(*args, **kwargs)
self.__dict__ = self self.__dict__ = self

View file

@ -7,7 +7,7 @@ def check_subcommand(command, *args):
def test_cformat(): def test_cformat():
assert check_subcommand('cformat', 'tmk_core/common/backlight.c').returncode == 0 assert check_subcommand('cformat', 'tmk_core/common/keyboard.c').returncode == 0
def test_compile(): def test_compile():

View file

@ -10,4 +10,4 @@ def test_keymap_onekey_pytest():
def test_normpath(): def test_normpath():
path = qmk.path.normpath('lib/python') path = qmk.path.normpath('lib/python')
assert path == os.environ['ORIG_CWD'] + '/lib/python' assert path == os.path.join(os.environ['ORIG_CWD'], 'lib/python')