diff options
Diffstat (limited to 'tests/test_subcmds.py')
-rw-r--r-- | tests/test_subcmds.py | 43 |
1 files changed, 43 insertions, 0 deletions
diff --git a/tests/test_subcmds.py b/tests/test_subcmds.py new file mode 100644 index 00000000..2234e646 --- /dev/null +++ b/tests/test_subcmds.py | |||
@@ -0,0 +1,43 @@ | |||
1 | # Copyright (C) 2020 The Android Open Source Project | ||
2 | # | ||
3 | # Licensed under the Apache License, Version 2.0 (the "License"); | ||
4 | # you may not use this file except in compliance with the License. | ||
5 | # You may obtain a copy of the License at | ||
6 | # | ||
7 | # http://www.apache.org/licenses/LICENSE-2.0 | ||
8 | # | ||
9 | # Unless required by applicable law or agreed to in writing, software | ||
10 | # distributed under the License is distributed on an "AS IS" BASIS, | ||
11 | # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. | ||
12 | # See the License for the specific language governing permissions and | ||
13 | # limitations under the License. | ||
14 | |||
15 | """Unittests for the subcmds module (mostly __init__.py than subcommands).""" | ||
16 | |||
17 | import unittest | ||
18 | |||
19 | import subcmds | ||
20 | |||
21 | |||
22 | class AllCommands(unittest.TestCase): | ||
23 | """Check registered all_commands.""" | ||
24 | |||
25 | def test_required_basic(self): | ||
26 | """Basic checking of registered commands.""" | ||
27 | # NB: We don't test all subcommands as we want to avoid "change detection" | ||
28 | # tests, so we just look for the most common/important ones here that are | ||
29 | # unlikely to ever change. | ||
30 | for cmd in {'cherry-pick', 'help', 'init', 'start', 'sync', 'upload'}: | ||
31 | self.assertIn(cmd, subcmds.all_commands) | ||
32 | |||
33 | def test_naming(self): | ||
34 | """Verify we don't add things that we shouldn't.""" | ||
35 | for cmd in subcmds.all_commands: | ||
36 | # Reject filename suffixes like "help.py". | ||
37 | self.assertNotIn('.', cmd) | ||
38 | |||
39 | # Make sure all '_' were converted to '-'. | ||
40 | self.assertNotIn('_', cmd) | ||
41 | |||
42 | # Reject internal python paths like "__init__". | ||
43 | self.assertFalse(cmd.startswith('__')) | ||