{CI} Fix cmdcov issue: Any name of a parameter has been tested, the p…

…arameter should be considered as passing the parameter test coverage. (#417)

* Update linter.py

* update

* Update HISTORY.rst

HISTORY.rst

@@ -2,6 +2,10 @@
 
 Release History
 ===============
+0.1.58
+++++++
+* Fix cmdcov issue(#417): Any name of a parameter has been tested, the parameter should be considered as passing the parameter test coverage.
+
 0.1.57
 ++++++
 * `azdev command-change meta-diff`: Remove duplicated meta-diff code and use azure-cli-diff-tool

azdev/__init__.py

@@ -4,4 +4,4 @@
 # license information.
 # -----------------------------------------------------------------------------
 
-__VERSION__ = '0.1.57'
+__VERSION__ = '0.1.58'

azdev/operations/linter/linter.py

@@ -224,7 +224,7 @@ def _detect_new_command(self, diff_index):
                 for p, r in v['parameters'].items():
                     if 'missing_parameter_test_coverage' in r['rule_exclusions']:
                         exclude_parameters.append((c, p))
-            elif 'rule_exclusions' in v:
+            if 'rule_exclusions' in v:
                 if 'missing_command_test_coverage' in v['rule_exclusions']:
                     exclude_comands.append(c)
         _logger.debug('exclude_parameters: %s', exclude_parameters)
@@ -340,11 +340,11 @@ def _run_parameter_test_coverage(parameters, all_tested_command):
                     if command in code and opt in code:
                         flag = True
                         break
-                else:
-                    violations.append(f'Missing parameter test coverage: `{command} {opt}`')
-                    exec_state = False
                 if flag:
                     break
+            else:
+                violations.append(f'Missing parameter test coverage: `{command} {opt}`')
+                exec_state = False
         if violations:
             violations.insert(0, 'Failed.')
             violations.extend([