Commit fb9aeb53 by Frak

Add colors for OK and FAIL cases

parent 108eb737
...@@ -11,6 +11,22 @@ This module is the engine of checks. ...@@ -11,6 +11,22 @@ This module is the engine of checks.
# pylint: disable=missing-class-docstring,missing-function-docstring # pylint: disable=missing-class-docstring,missing-function-docstring
# pylint: disable=line-too-long,invalid-name,too-many-branches # pylint: disable=line-too-long,invalid-name,too-many-branches
GREEN_COLOR = '\x1b[32m'
RED_COLOR = '\x1b[31m'
COLOR_END = '\x1b[0m'
def colorize_result(input):
if input.startswith('OK'):
color = GREEN_COLOR
elif input.startswith('FAIL:'):
color = RED_COLOR
else:
assert(False), f'unexpected result "{input}"'
colored_result = f'{color}{input}{COLOR_END}'
print(f'| {colored_result}', end='')
class OptCheck: class OptCheck:
def __init__(self, reason, decision, name, expected): def __init__(self, reason, decision, name, expected):
...@@ -78,7 +94,7 @@ class OptCheck: ...@@ -78,7 +94,7 @@ class OptCheck:
def table_print(self, _mode, with_results): def table_print(self, _mode, with_results):
print(f'{self.name:<40}|{self.type:^7}|{self.expected:^12}|{self.decision:^10}|{self.reason:^18}', end='') print(f'{self.name:<40}|{self.type:^7}|{self.expected:^12}|{self.decision:^10}|{self.reason:^18}', end='')
if with_results: if with_results:
print(f'| {self.result}', end='') colorize_result(self.result)
def json_dump(self, with_results): def json_dump(self, with_results):
dump = [self.name, self.type, self.expected, self.decision, self.reason] dump = [self.name, self.type, self.expected, self.decision, self.reason]
...@@ -137,7 +153,7 @@ class VersionCheck: ...@@ -137,7 +153,7 @@ class VersionCheck:
ver_req = f'kernel version >= {self.ver_expected[0]}.{self.ver_expected[1]}' ver_req = f'kernel version >= {self.ver_expected[0]}.{self.ver_expected[1]}'
print(f'{ver_req:<91}', end='') print(f'{ver_req:<91}', end='')
if with_results: if with_results:
print(f'| {self.result}', end='') colorize_result(self.result)
class ComplexOptCheck: class ComplexOptCheck:
...@@ -167,7 +183,8 @@ class ComplexOptCheck: ...@@ -167,7 +183,8 @@ class ComplexOptCheck:
if mode == 'verbose': if mode == 'verbose':
print(f' {"<<< " + self.__class__.__name__ + " >>>":87}', end='') print(f' {"<<< " + self.__class__.__name__ + " >>>":87}', end='')
if with_results: if with_results:
print(f'| {self.result}', end='') colorize_result(self.result)
for o in self.opts: for o in self.opts:
print() print()
o.table_print(mode, with_results) o.table_print(mode, with_results)
...@@ -175,7 +192,10 @@ class ComplexOptCheck: ...@@ -175,7 +192,10 @@ class ComplexOptCheck:
o = self.opts[0] o = self.opts[0]
o.table_print(mode, False) o.table_print(mode, False)
if with_results: if with_results:
print(f'| {self.result}', end='') colorize_result(self.result)
def json_dump(self, with_results): def json_dump(self, with_results):
dump = self.opts[0].json_dump(False) dump = self.opts[0].json_dump(False)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment