@@ -49,8 +49,8 @@ def test_util_alias_import_warns() -> None:
4949assert "should not be relied on" in message
5050
5151
52- def test_private_module_aliases ()-> None :
53- """These existdynamically (for now) but mypy treats them as absent (intentionally).
52+ def test_private_module_aliases_exist_dynamically ()-> None :
53+ """These existat runtime (for now) but mypy treats them as absent (intentionally).
5454
5555 This code verifies the effect of static type checking when analyzed by mypy, if mypy
5656 is configured with ``warn_unused_ignores = true``.
@@ -67,7 +67,7 @@ def test_private_module_aliases() -> None:
6767git .typ # type: ignore[attr-defined]
6868
6969
70- _parametrize_by_private_alias = pytest .mark .parametrize (
70+ @ pytest .mark .parametrize (
7171"name, fullname" ,
7272 [
7373 ("head" ,"git.refs.head" ),
@@ -80,32 +80,26 @@ def test_private_module_aliases() -> None:
8080 ("typ" ,"git.index.typ" ),
8181 ],
8282)
83+ class TestPrivateModuleAliases :
84+ """Tests of the private module aliases' shared specific runtime behaviors."""
8385
86+ def test_private_module_alias_access_resolves (self ,name :str ,fullname :str )-> None :
87+ """These resolve for now, though they're private we do not guarantee this."""
88+ assert getattr (git ,name )is importlib .import_module (fullname )
8489
85- @_parametrize_by_private_alias
86- def test_private_module_alias_access_resolves (name :str ,fullname :str )-> None :
87- """These resolve for now, though they're private we do not guarantee this."""
88- assert getattr (git ,name )is importlib .import_module (fullname )
89-
90-
91- @_parametrize_by_private_alias
92- def test_private_module_alias_import_resolves (name :str ,fullname :str )-> None :
93- exec (f"from git import{ name } )
94- assert locals ()[name ]is importlib .import_module (fullname )
95-
96-
97- @_parametrize_by_private_alias
98- def test_private_module_alias_access_warns (name :str ,fullname :str )-> None :
99- with pytest .deprecated_call ()as ctx :
100- getattr (git ,name )
90+ def test_private_module_alias_import_resolves (self ,name :str ,fullname :str )-> None :
91+ exec (f"from git import{ name } )
92+ assert locals ()[name ]is importlib .import_module (fullname )
10193
102- assert len (ctx )== 1
103- assert ctx [0 ].message .args [0 ].endswith (f"Use{ fullname } )
94+ def test_private_module_alias_access_warns (self ,name :str ,fullname :str )-> None :
95+ with pytest .deprecated_call ()as ctx :
96+ getattr (git ,name )
10497
98+ assert len (ctx )== 1
99+ assert ctx [0 ].message .args [0 ].endswith (f"Use{ fullname } )
105100
106- @_parametrize_by_private_alias
107- def test_private_module_alias_import_warns (name :str ,fullname :str )-> None :
108- with pytest .deprecated_call ()as ctx :
109- exec (f"from git import{ name } )
101+ def test_private_module_alias_import_warns (self ,name :str ,fullname :str )-> None :
102+ with pytest .deprecated_call ()as ctx :
103+ exec (f"from git import{ name } )
110104
111- assert ctx [0 ].message .args [0 ].endswith (f"Use{ fullname } )
105+ assert ctx [0 ].message .args [0 ].endswith (f"Use{ fullname } )