Fix unittest

web-platform-tests · Jul 31, 2023 · 490feeb · 490feeb
1 parent b7cf1dc
commit 490feeb
Showing 1 changed file with 6 additions and 8 deletions.
diff --git a/tools/wptrunner/wptrunner/tests/test_testloader.py b/tools/wptrunner/wptrunner/tests/test_testloader.py
@@ -248,14 +248,14 @@ def test_loader_filter_tags():
         loader = TestLoader({manifest: {"metadata_path": metadata_path}}, ["testharness"], None)
         assert len(loader.tests["testharness"]) == 4
 
-        # Check: specifying a single `test-include` inclusion yields `/a/bar`
+        # Check: specifying a single `test-include` inclusion yields `/a/bar` and `/b/baz`
         loader = TestLoader({manifest: {"metadata_path": metadata_path}}, ["testharness"], None,
-                            test_filters=[TagFilter({"test-include"})])
-        assert len(loader.tests["testharness"]) == 1
+                            test_filters=[TagFilter({"test-include"}, {})])
+        assert len(loader.tests["testharness"]) == 2
         assert loader.tests["testharness"][0].id == "/a/bar.html"
         assert loader.tests["testharness"][0].tags == {"dir:a", "test-include"}
         assert loader.tests["testharness"][1].id == "/b/baz.html"
-        assert loader.tests["testharness"][1].tags == {"dir:b", "test-include"}
+        assert loader.tests["testharness"][1].tags == {"dir:b", "test-include", "test-exclude"}
 
         # Check: specifying a single `test-exclude` exclusion rejects only `/b/baz`
         loader = TestLoader({manifest: {"metadata_path": metadata_path}}, ["testharness"], None,
@@ -274,13 +274,11 @@ def test_loader_filter_tags():
 
         loader = TestLoader({manifest: {"metadata_path": metadata_path}}, ["testharness"], None,
                             test_filters=[TagFilter({"test-include"}, {"test-include"})])
-        assert (len(loader.tests["testharness"] == 0))
+        assert len(loader.tests["testharness"]) == 0
 
         loader = TestLoader({manifest: {"metadata_path": metadata_path}}, ["testharness"], None,
                             test_filters=[TagFilter({"test-include", "test-exclude"}, {"test-include"})])
-        assert len(loader.tests["testharness"]) == 1
-        assert loader.tests["testharness"][0].id == "/b/baz.html"
-        assert loader.tests["testharness"][0].tags == {"dir:b", "test-include", "test-exclude"}
+        assert len(loader.tests["testharness"]) == 0
 
 
 def test_chunk_hash(manifest):