{CI} Fix Incorrect Detection of Code Changes as New Commands (#499)

Author: wangzelin007

HISTORY.rst

@@ -2,6 +2,10 @@
 
 Release History
 ===============
+0.1.90
+++++++
+* `azdev cmdcov`: Fix incorrect detection of code changes as new commands
+
 0.1.89
 ++++++
 * `azdev scan/mask`: Add `--continue-on-failure` support

azdev/__init__.py

@@ -4,4 +4,4 @@
 # license information.
 # -----------------------------------------------------------------------------
 
-__VERSION__ = '0.1.89'
+__VERSION__ = '0.1.90'

azdev/operations/linter/rules/command_coverage_rules.py

@@ -8,7 +8,7 @@
 from ..linter import RuleError, LinterSeverity
 
 
-@CommandCoverageRule(LinterSeverity.HIGH)
+@CommandCoverageRule(LinterSeverity.MEDIUM)
 def missing_command_test_coverage(linter):
     exec_state, violations = linter.get_command_test_coverage()
     if not exec_state:

azdev/operations/regex.py

@@ -144,6 +144,7 @@ def search_argument(line):
 def search_command_group(row_num, lines, command):
     cmd = ''
     while row_num > 0:
+        row_num = len(lines) - 1 if row_num >= len(lines) else row_num
         row_num -= 1
         # Match `with self.command_group('local-context',` and `with self.command_group('xxx')`
         sub_pattern = r'with self.command_group\(\'(.*?)\',?'
@@ -166,8 +167,8 @@ def search_command(line):
 
 def search_deleted_command(line):
     command = ''
-    # Match `- g.*command(xxx)`
-    pattern = r'\-\s+g.(?:\w+)?command\((.*)\)'
+    # Match `[-!] g.*command(xxx)`
+    pattern = r'[-!]\s+g.(?:\w+)?command\((.*)\)'
     ref = re.findall(pattern, line)
     if ref:
         command = ref[0].split(',')[0].strip("'")