Skip to content

Commit 8c90e94

Browse files
committed
fix: rename --quite CLI flag to --quiet and remove dead exit() call
1 parent 148bd85 commit 8c90e94

2 files changed

Lines changed: 9 additions & 4 deletions

File tree

compliance_tool/aas_compliance_tool/cli.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ def parse_cli_arguments() -> argparse.ArgumentParser:
7676
parser.add_argument('-v', '--verbose', help="Print detailed information for each check. Multiple -v options "
7777
"increase the verbosity. 1: Detailed error information, 2: Additional "
7878
"detailed success information", action='count', default=0)
79-
parser.add_argument('-q', '--quite', help="no information output if successful", action='store_true')
79+
parser.add_argument('-q', '--quiet', help="no information output if successful", action='store_true')
8080
group = parser.add_mutually_exclusive_group(required=True)
8181
group.add_argument('--json', help="Use AAS json format when checking or creating files", action='store_true')
8282
group.add_argument('--xml', help="Use AAS xml format when checking or creating files", action='store_true')
@@ -177,9 +177,8 @@ def main():
177177
**data_checker_kwargs)
178178
else:
179179
parser.error("f or files requires two file path.")
180-
exit()
181180

182-
if manager.status is Status.SUCCESS and args.quite:
181+
if manager.status is Status.SUCCESS and args.quiet:
183182
exit()
184183

185184
print(manager.format_state_manager(args.verbose))

compliance_tool/test/test_aas_compliance_tool.py

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -115,11 +115,17 @@ def test_parse_args(self) -> None:
115115
self.assertNotIn('ERROR', str(output.stderr))
116116
self.assertIn('INFO', str(output.stdout))
117117

118-
# test quite
118+
# test quiet (short form)
119119
output = _run_compliance_tool("e", os.path.join(test_file_path, "test_demo_full_example.json"), "--json", "-q")
120120
self.assertEqual(0, output.returncode)
121121
self.assertEqual("b''", str(output.stdout))
122122

123+
# test quiet (long form -- previously misspelled as --quite)
124+
output = _run_compliance_tool("e", os.path.join(test_file_path, "test_demo_full_example.json"), "--json",
125+
"--quiet")
126+
self.assertEqual(0, output.returncode)
127+
self.assertEqual("b''", str(output.stdout))
128+
123129
# test logfile
124130
output = _run_compliance_tool("e", os.path.join(test_file_path, "test_demo_full_example.json"), "--json", "-l")
125131
self.assertNotEqual(0, output.returncode)

0 commit comments

Comments
 (0)