Skip to content

Commit fb6a94e

Browse files
committed
Use a non-_ name for arguments
The new version of `pydoclint` doesn't allow to document arguments named `_` (it complains with "Docstring contains more arguments than in function signature."). Instead of removing the documentation, we can just rename the arguments to have a more sensible name, which should make the test more readable anyway. Signed-off-by: Leandro Lucarella <[email protected]>
1 parent 92f686a commit fb6a94e

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

tests/microgrid/test_microgrid_api.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -102,15 +102,15 @@ def metadata(self) -> meta.Metadata:
102102
@mock.patch("grpc.aio.insecure_channel")
103103
async def test_connection_manager(
104104
self,
105-
_: MagicMock,
105+
_insecure_channel_mock: MagicMock,
106106
components: list[list[Component]],
107107
connections: list[list[Connection]],
108108
metadata: meta.Metadata,
109109
) -> None:
110110
"""Test microgrid api.
111111
112112
Args:
113-
_: insecure channel mock from `mock.patch`
113+
_insecure_channel_mock: insecure channel mock from `mock.patch`
114114
components: components
115115
connections: connections
116116
metadata: the metadata of the microgrid
@@ -179,15 +179,15 @@ async def test_connection_manager(
179179
@mock.patch("grpc.aio.insecure_channel")
180180
async def test_connection_manager_another_method(
181181
self,
182-
_: MagicMock,
182+
_insecure_channel_mock: MagicMock,
183183
components: list[list[Component]],
184184
connections: list[list[Connection]],
185185
metadata: meta.Metadata,
186186
) -> None:
187187
"""Test if the api was not deallocated.
188188
189189
Args:
190-
_: insecure channel mock
190+
_insecure_channel_mock: insecure channel mock
191191
components: components
192192
connections: connections
193193
metadata: the metadata of the microgrid

0 commit comments

Comments
 (0)