Only test the contents of __all__, not the presence

python · Jul 31, 2022 · 02b4f14 · 02b4f14
1 parent 2f06a77
commit 02b4f14
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/mypy/stubtest.py b/mypy/stubtest.py
@@ -316,7 +316,7 @@ def _belongs_to_runtime(r: types.ModuleType, attr: str) -> bool:
         return not isinstance(obj, types.ModuleType)
 
     runtime_public_contents = (
-        runtime_all_as_set | {"__all__"}
+        runtime_all_as_set
         if runtime_all_as_set is not None
         else {
             m

diff --git a/mypy/test/teststubtest.py b/mypy/test/teststubtest.py
@@ -934,7 +934,7 @@ def test_all_at_runtime_not_stub(self) -> Iterator[Case]:
             runtime="""
             __all__ = []
             Z = 5""",
-            error="__all__",
+            error=None,
         )
 
     @collect_cases
@@ -974,7 +974,7 @@ def h(x: str): ...
             runtime="",
             error="h",
         )
-        yield Case(stub="", runtime="__all__ = []", error="__all__")  # dummy case
+        yield Case(stub="", runtime="__all__ = []", error=None)  # dummy case
         yield Case(stub="", runtime="__all__ += ['y']\ny = 5", error="y")
         yield Case(stub="", runtime="__all__ += ['g']\ndef g(): pass", error="g")
         # Here we should only check that runtime has B, since the stub explicitly re-exports it