azdev linter: Fix list index out of range in missing_command_test_coverage regex (#523)

* fix

* Update __init__.py

* Create pull_request_template.md

* Update regex.py

Author: wangzelin007

.github/pull_request_template.md

@@ -0,0 +1,14 @@
+**Related command**
+<!--- Please provide the related command with azdev {command} if you can. --->
+
+**Description**<!--Mandatory-->
+<!--Why this PR? What is changed? What is the effect? etc. A high-quality description can accelerate the review process.-->
+
+
+---
+
+This checklist is used to make sure that common guidelines for a pull request are followed.
+
+- [ ] `pylint azdev --rcfile=.pylintrc -r n`
+
+- [ ] `flake8 --statistics --append-config=.flake8 azdev`

HISTORY.rst

@@ -2,6 +2,10 @@
 
 Release History
 ===============
+0.2.3
+++++++
+* `azdev linter`: Fix `list index out of range` in missing_command_test_coverage regex
+
 0.2.2
 ++++++
 * Update dependency `azure-cli-diff-tool` to `0.1.0`.

azdev/__init__.py

@@ -4,4 +4,4 @@
 # license information.
 # -----------------------------------------------------------------------------
 
-__VERSION__ = '0.2.2'
+__VERSION__ = '0.2.3'

azdev/operations/regex.py

@@ -54,7 +54,8 @@ def get_all_tested_commands_from_regex(lines):
         if re_idx is None and re.findall(CMD_PATTERN[3], lines[row_num]):
             re_idx = 3
         if re_idx is not None:
-            command = re.findall(CMD_PATTERN[re_idx], lines[row_num])[0]
+            matches = re.findall(CMD_PATTERN[re_idx], lines[row_num])
+            command = matches[0] if matches else ''
             while row_num < total_lines:
                 if (re_idx in [0, 1] and not re.findall(END_PATTERN, lines[row_num])) or \
                         (re_idx == 2 and (row_num + 1) < total_lines and
@@ -105,14 +106,17 @@ def search_argument_context(row_num, lines):
         # Match `with self.argument_context(scope) as c:`
         if ref1:
             sub_pattern = r'for scope in (.*):'
-            cmds = json.loads(
-                re.findall(sub_pattern, lines[row_num - 1])[0].replace('\'', '"'))
+            matches = re.findall(sub_pattern, lines[row_num - 1])
+            if matches:
+                cmds = json.loads(matches[0].replace('\'', '"'))
             break
         # Match `with self.argument_context(['"]{} stop['"].format(scope)) as c:',
         if ref2:
             sub_pattern = r'for scope in (.*):'
-            format_strings = json.loads(
-                re.findall(sub_pattern, lines[row_num - 1])[0].replace('\'', '"'))
+            format_strings = ''
+            matches = re.findall(sub_pattern, lines[row_num - 1])
+            if matches:
+                format_strings = json.loads(matches[0].replace('\'', '"'))
             for c in ref2:
                 for f in format_strings:
                     cmds.append(c.replace('{}', f))