Tags Filtering Fix

Filtering now properly handles both verbose and non-verbose output modes
while checking for tag matches.
without changing the existing behavior that users expect when using
'avocado list -t <tag>'

Reference:      #6066
Signed-off-by: Harvey Lynden <hlynden@redhat.com>

avocado/plugins/list.py

@@ -22,6 +22,7 @@
 from avocado.core.resolver import ReferenceResolutionResult
 from avocado.core.settings import settings
 from avocado.core.suite import TestSuite
+from avocado.core.tags import filter_tags_on_runnables
 from avocado.utils.astring import iter_tabular_output
 
 
@@ -76,11 +77,7 @@ def _display(self, suite, matrix):
                 TERM_SUPPORT.header_str("Tag(s)"),
             )
 
-        # Any kind of color, string format and term specific should be applied
-        # only during output/display phase. So this seems to be a better place
-        # for this:
         matrix = self._prepare_matrix_for_display(matrix, verbose)
-
         for line in iter_tabular_output(matrix, header=header, strip=True):
             LOG_UI.debug(line)
 
@@ -142,21 +139,35 @@ def _get_resolution_matrix(suite):
         """
         test_matrix = []
         verbose = suite.config.get("core.verbose")
+        tags = suite.config.get("filter.by_tags.tags")
+        include_empty = suite.config.get("filter.by_tags.include_empty")
+        include_empty_key = suite.config.get("filter.by_tags.include_empty_key")
+
         for resolution in suite.resolutions:
-            for runnable in resolution.resolutions:
+            if resolution.result != ReferenceResolutionResult.SUCCESS:
+                continue
+
+            if tags:
+                runnables = filter_tags_on_runnables(
+                    [resolution], tags, include_empty, include_empty_key
+                )
+            else:
+                runnables = resolution.resolutions
+
+            for runnable in runnables:
                 if verbose:
-                    tags = runnable.tags or {}
                     test_matrix.append(
                         (
                             runnable.kind,
                             runnable.identifier,
                             runnable.uri,
                             resolution.origin,
-                            tags,
+                            runnable.tags or {},
                         )
                     )
                 else:
                     test_matrix.append((runnable.kind, runnable.identifier))
+
         return test_matrix
 
     @staticmethod

selftests/functional/list.py

@@ -153,6 +153,23 @@ def test_list_filter_by_tags(self):
             (f"Avocado did not return rc " f"{exit_codes.AVOCADO_ALL_OK}:\n{result}"),
         )
         stdout_lines = result.stdout_text.splitlines()
+        self.assertRegex(
+            stdout_lines[1],
+            r"avocado-instrumented\s+.*examples/tests/cit_parameters.py:CitParameters.test\s+.*examples/tests/cit_parameters.py:CitParameters.test\s+avocado-instrumented\s+fast",
+        )
+        self.assertRegex(
+            stdout_lines[2],
+            r"avocado-instrumented\s+.*examples/tests/passtest.py:PassTest.test\s+.*examples/tests/passtest.py:PassTest.test\s+avocado-instrumented\s+fast",
+        )
+        self.assertRegex(
+            stdout_lines[3],
+            r"avocado-instrumented\s+.*examples/tests/tags.py:FastTest.test_fast\s+.*examples/tests/tags.py:FastTest.test_fast\s+avocado-instrumented\s+net,fast",
+        )
+        self.assertRegex(
+            stdout_lines[4],
+            r"avocado-instrumented\s+.*examples/tests/tags.py:FastTest.test_fast_other\s+.*examples/tests/tags.py:FastTest.test_fast_other\s+avocado-instrumented\s+net,fast",
+        )
+        self.assertEqual("", stdout_lines[5])
         self.assertIn("TEST TYPES SUMMARY", stdout_lines)
         self.assertIn("avocado-instrumented: 4", stdout_lines)
         self.assertIn("TEST TAGS SUMMARY", stdout_lines)