diff options
author | Denis 'GNUtoo' Carikli <GNUtoo@cyberdimension.org> | 2022-01-24 22:51:01 +0100 |
---|---|---|
committer | Denis 'GNUtoo' Carikli <GNUtoo@cyberdimension.org> | 2022-01-24 23:09:14 +0100 |
commit | 34a3e1fd501f20c55d41ec9aaa982d79a499bf41 (patch) | |
tree | b6e4e829877d45f31f488731ac38b89ce0df4f89 /tools | |
parent | 9e172e31991ee4aead7e891c111e9734667f29de (diff) | |
download | hardware_replicant_libsamsung-ipc-34a3e1fd501f20c55d41ec9aaa982d79a499bf41.tar.gz hardware_replicant_libsamsung-ipc-34a3e1fd501f20c55d41ec9aaa982d79a499bf41.tar.bz2 hardware_replicant_libsamsung-ipc-34a3e1fd501f20c55d41ec9aaa982d79a499bf41.zip |
tests: python: unify string quotes as per PEP 8
Having a code style makes the code easier to read.
As for which code style to use, the code style defined by the
PEP 8 [1] is used in the python standard library and in the main
Python distribution.
Its official nature (it's standardized by python) probably makes it
the most well known and used code style adopted by python programmers.
In it we have a section about string quotes:
String Quotes
-------------
In Python, single-quoted strings and double-quoted strings are the
same. This PEP does not make a recommendation for this. Pick a rule
and stick to it. When a string contains single or double quote
characters, however, use the other one to avoid backslashes in the
string. It improves readability.
For triple-quoted strings, always use double quote characters to be
consistent with the docstring convention in PEP 257.
Since "if __name__ == '__main__':" is widely used, we choose to use the
single quotes.
[1]https://www.python.org/dev/peps/pep-0008/
Signed-off-by: Denis 'GNUtoo' Carikli <GNUtoo@cyberdimension.org>
Diffstat (limited to 'tools')
-rwxr-xr-x | tools/ipc-modem.py | 14 | ||||
-rwxr-xr-x | tools/nv_data-imei.py | 58 | ||||
-rwxr-xr-x | tools/nv_data-md5.py | 10 |
3 files changed, 41 insertions, 41 deletions
diff --git a/tools/ipc-modem.py b/tools/ipc-modem.py index b1d28b4..0fcd1fc 100755 --- a/tools/ipc-modem.py +++ b/tools/ipc-modem.py @@ -23,11 +23,11 @@ import sys import sh def usage(progname): - print("{} [test]".format(progname)) + print('{} [test]'.format(progname)) sys.exit(1) def get_output(data): - return str(data).replace(os.linesep, "") + return str(data).replace(os.linesep, '') class IpcModem(object): def __init__(self): @@ -36,7 +36,7 @@ class IpcModem(object): if not srcdir: srcdir = os.path.dirname(sys.argv[0]) - ipc_modem = sh.Command(srcdir + os.sep + "ipc-modem") + ipc_modem = sh.Command(srcdir + os.sep + 'ipc-modem') self.ipc_modem = ipc_modem.bake('--dry-run') def test_help(self): @@ -48,17 +48,17 @@ class IpcModem(object): raise Exception() def test_boot(self, timeout=2): - self.ipc_modem("boot", _timeout=timeout) + self.ipc_modem('boot', _timeout=timeout) def test_power_on(self, timeout=2): - self.ipc_modem("power-on", _timeout=timeout) + self.ipc_modem('power-on', _timeout=timeout) def test_power_off(self, timeout=2): - self.ipc_modem("power-off", _timeout=timeout) + self.ipc_modem('power-off', _timeout=timeout) def test_start(self, timeout=3): try: - self.ipc_modem("start", _timeout=timeout) + self.ipc_modem('start', _timeout=timeout) except sh.TimeoutException: pass else: diff --git a/tools/nv_data-imei.py b/tools/nv_data-imei.py index da58a50..dbe9cfb 100755 --- a/tools/nv_data-imei.py +++ b/tools/nv_data-imei.py @@ -30,18 +30,18 @@ class SysExit(object): EX_NOINPUT = sh.ErrorReturnCode_66 def usage(progname): - print("{} [test]".format(progname)) + print('{} [test]'.format(progname)) sys.exit(1) commands = [ - "list-supported", - "read-imei", - "write-imei", - "bruteforce-imei", + 'list-supported', + 'read-imei', + 'write-imei', + 'bruteforce-imei', ] def get_output(data): - return str(data).replace(os.linesep, "") + return str(data).replace(os.linesep, '') class NvDataImei(object): def __init__(self): @@ -50,64 +50,64 @@ class NvDataImei(object): if not srcdir: srcdir = os.path.dirname(sys.argv[0]) - self.nv_data_imei = sh.Command(srcdir + os.sep + "nv_data-imei") + self.nv_data_imei = sh.Command(srcdir + os.sep + 'nv_data-imei') def test_help(self): try: - self.nv_data_imei("") + self.nv_data_imei('') except SysExit.EX_USAGE: pass else: raise Exception() - for help_arg in ["-h", "--help"]: + for help_arg in ['-h', '--help']: self.nv_data_imei(help_arg) for command in commands: self.nv_data_imei(command, help_arg) try: - self.nv_data_imei("file", command, help_arg) + self.nv_data_imei('file', command, help_arg) except SysExit.EX_USAGE: pass else: raise Exception() - self.nv_data_imei("list-supported") + self.nv_data_imei('list-supported') def test_commands(self): # Create nv_data.bin - valid_imei = "123456789012345" + valid_imei = '123456789012345' offset = 0x100 XMM616_NV_DATA_SIZE = 0x200000 nv_data_bin = get_output(sh.mktemp()) - sh.ddrescue("/dev/zero", nv_data_bin, "-s", str(XMM616_NV_DATA_SIZE)) + sh.ddrescue('/dev/zero', nv_data_bin, '-s', str(XMM616_NV_DATA_SIZE)) - self.nv_data_imei(nv_data_bin, "write-imei", "-o", str(hex(offset)), - "-i", valid_imei) - output = get_output(self.nv_data_imei(nv_data_bin, "read-imei", "-o", + self.nv_data_imei(nv_data_bin, 'write-imei', '-o', str(hex(offset)), + '-i', valid_imei) + output = get_output(self.nv_data_imei(nv_data_bin, 'read-imei', '-o', str(hex(offset)))) print(output) - expect = "IMEI: " + valid_imei + expect = 'IMEI: ' + valid_imei if output != expect: raise Exception() - output = get_output(self.nv_data_imei(nv_data_bin, "bruteforce-imei", - "-i", valid_imei)) + output = get_output(self.nv_data_imei(nv_data_bin, 'bruteforce-imei', + '-i', valid_imei)) print(output) - expect = re.escape("Found IMEI at {} ({})".format(str(hex(offset)), + expect = re.escape('Found IMEI at {} ({})'.format(str(hex(offset)), offset)) if not re.search(expect, output): raise Exception() - inaccessible_nv_data_bin = str(sh.mktemp("-u")).replace(os.linesep,"") - sh.ddrescue("/dev/zero", inaccessible_nv_data_bin, "-s", + inaccessible_nv_data_bin = str(sh.mktemp('-u')).replace(os.linesep,'') + sh.ddrescue('/dev/zero', inaccessible_nv_data_bin, '-s', str(XMM616_NV_DATA_SIZE)) - sh.chmod("000", inaccessible_nv_data_bin); + sh.chmod('000', inaccessible_nv_data_bin); try: - self.nv_data_imei(inaccessible_nv_data_bin, "write-imei", - "-o", "0x0", "-i", valid_imei) - self.nv_data_imei(inaccessible_nv_data_bin, "read-imei", - "-o", "0x0") - self.nv_data_imei(inaccessible_nv_data_bin, "bruteforce-imei", - "-i", valid_imei) + self.nv_data_imei(inaccessible_nv_data_bin, 'write-imei', + '-o', '0x0', '-i', valid_imei) + self.nv_data_imei(inaccessible_nv_data_bin, 'read-imei', + '-o', '0x0') + self.nv_data_imei(inaccessible_nv_data_bin, 'bruteforce-imei', + '-i', valid_imei) except SysExit.EX_NOINPUT: pass else: diff --git a/tools/nv_data-md5.py b/tools/nv_data-md5.py index 2897f18..2dc98ab 100755 --- a/tools/nv_data-md5.py +++ b/tools/nv_data-md5.py @@ -23,11 +23,11 @@ import sys import sh def usage(progname): - print("{} [test]".format(progname)) + print('{} [test]'.format(progname)) sys.exit(1) def get_output(data): - return str(data).replace(os.linesep, "") + return str(data).replace(os.linesep, '') class NvDataMD5(object): def __init__(self): @@ -36,7 +36,7 @@ class NvDataMD5(object): if not srcdir: srcdir = os.path.dirname(sys.argv[0]) - self.nv_data_md5 = sh.Command(srcdir + os.sep + "nv_data-md5") + self.nv_data_md5 = sh.Command(srcdir + os.sep + 'nv_data-md5') def test_help(self): try: self.nv_data_md5() @@ -46,12 +46,12 @@ class NvDataMD5(object): raise Exception() def test_commands(self): - expected_md5 = "5293814414abb3831e3fc1a1b35e69bc" + expected_md5 = '5293814414abb3831e3fc1a1b35e69bc' NV_DATA_SIZE = 0x200000 nv_data_bin = get_output(sh.mktemp()) # Create nv_data.bin - sh.ddrescue("/dev/zero", nv_data_bin, "-s", str(NV_DATA_SIZE)) + sh.ddrescue('/dev/zero', nv_data_bin, '-s', str(NV_DATA_SIZE)) output = get_output(self.nv_data_md5(nv_data_bin)) |