From 8310436be083f8268bf8325bb2ffb65e9e178f6f Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Tue, 25 Mar 2025 12:50:36 -0400 Subject: run_tests: move test filtering to pytest markers Move the test disable logic even closer to the exact test that's disabled. This way people updating tests have a better chance of seeing they'll get reduced coverage in the CQ. Change-Id: I57c1a073a844019798b27e14d742fd32925d9ae8 Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/462882 Tested-by: Mike Frysinger Reviewed-by: Gavin Mak Commit-Queue: Mike Frysinger --- tests/test_git_superproject.py | 3 +++ 1 file changed, 3 insertions(+) (limited to 'tests/test_git_superproject.py') diff --git a/tests/test_git_superproject.py b/tests/test_git_superproject.py index 4e66521b..3ceb9320 100644 --- a/tests/test_git_superproject.py +++ b/tests/test_git_superproject.py @@ -21,6 +21,7 @@ import tempfile import unittest from unittest import mock +import pytest from test_manifest_xml import sort_attributes import git_superproject @@ -145,6 +146,7 @@ class SuperprojectTestCase(unittest.TestCase): ) self.assertIsNone(manifest.superproject) + @pytest.mark.skip_cq("TODO(b/266734831): Find out why this takes 8m+ in CQ") def test_superproject_get_superproject_invalid_url(self): """Test with an invalid url.""" manifest = self.getXmlManifest( @@ -168,6 +170,7 @@ class SuperprojectTestCase(unittest.TestCase): self.assertFalse(sync_result.success) self.assertTrue(sync_result.fatal) + @pytest.mark.skip_cq("TODO(b/266734831): Find out why this takes 8m+ in CQ") def test_superproject_get_superproject_invalid_branch(self): """Test with an invalid branch.""" manifest = self.getXmlManifest( -- cgit v1.2.3-54-g00ecf