Add test to ensure varnames are calculated correctly. Ref #596.

jaraco · Sep 17, 2022 · 845e739 · 845e739
1 parent 6466e1e
commit 845e739
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/conftest.py b/conftest.py
@@ -19,5 +19,3 @@ def macos_api_ignore():
 
 
 collect_ignore.extend(['keyring/backends/macOS/api.py'] * macos_api_ignore())
-
-collect_ignore.append('keyring/devpi_client.py')
diff --git a/keyring/devpi_client.py b/keyring/devpi_client.py
@@ -1,12 +1,12 @@
 import contextlib
 
-from pluggy import HookimplMarker
+import pluggy
 
 import keyring
 from keyring.errors import KeyringError
 
 
-hookimpl = HookimplMarker("devpiclient")
+hookimpl = pluggy.HookimplMarker("devpiclient")
 
 
 # https://github.com/jaraco/jaraco.context/blob/c3a9b739/jaraco/context.py#L205
@@ -16,4 +16,8 @@
 @hookimpl()
 @suppress(KeyringError)
 def devpiclient_get_password(url, username):
+    """
+    >>> pluggy._hooks.varnames(devpiclient_get_password)
+    (('url', 'username'), ())
+    """
     return keyring.get_password(url, username)