diff --git a/src/dvsim/testplan.py b/src/dvsim/testplan.py
index dee1a95a..47b11263 100644
--- a/src/dvsim/testplan.py
+++ b/src/dvsim/testplan.py
@@ -83,18 +83,19 @@ def _validate(self) -> None:
             msg = f"'tags' key in {self} is not a list."
             raise ValueError(msg)
 
-    def has_tags(self, tags: set) -> bool:
-        """Checks if the provided tags match the tags originally set.
+    def has_tags(self, tags: set[str]) -> bool:
+        """Return true if the element matches the provided tags.
 
-        tags is a list of tags that are we are filtering this testpoints with.
-        Tags may be preceded with `-` to exclude the testpoints that contain
-        that tag.
+        The element should match every item in the set of tags. If one of these
+        elements is preceded with "-", the meaning is negated (so the element
+        should not match the tag name).
 
-        Vacuously returns true if tags is an empty list.
-        """
-        if not tags:
-            return True
+        If tags is empty, this will vacuously return true.
+
+        Args:
+          tags: The set of named tags against which to match.
 
+        """
         for tag in tags:
             if tag.startswith("-"):
                 if tag[1:] in self.tags: