devpi-admin 1.0.0__tar.gz → 1.0.1__tar.gz

{devpi_admin-1.0.0 → devpi_admin-1.0.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devpi-admin
-Version: 1.0.0
+Version: 1.0.1
 Summary: Modern web UI plugin for devpi-server — drop-in replacement for devpi-web
 Author-email: Pavel Revak <pavelrevak@gmail.com>
 License: MIT
@@ -145,6 +145,15 @@ The static files live at `src/devpi_admin/static/` and can be edited in place
 show up on the next browser reload, no restart of devpi-server required (static views
 read from disk on each request).
 
+Run the unit tests:
+
+```bash
+PYTHONWARNINGS="ignore::UserWarning" python -m unittest discover -v tests/
+```
+
+(The `PYTHONWARNINGS` shim hides an unrelated deprecation warning emitted by Pyramid 2.1
+when it imports `pkg_resources`.)
+
 ## Author
 
 Pavel Revak <pavelrevak@gmail.com>

{devpi_admin-1.0.0 → devpi_admin-1.0.1}/README.md

@@ -124,6 +124,15 @@ The static files live at `src/devpi_admin/static/` and can be edited in place
 show up on the next browser reload, no restart of devpi-server required (static views
 read from disk on each request).
 
+Run the unit tests:
+
+```bash
+PYTHONWARNINGS="ignore::UserWarning" python -m unittest discover -v tests/
+```
+
+(The `PYTHONWARNINGS` shim hides an unrelated deprecation warning emitted by Pyramid 2.1
+when it imports `pkg_resources`.)
+
 ## Author
 
 Pavel Revak <pavelrevak@gmail.com>

{devpi_admin-1.0.0 → devpi_admin-1.0.1}/src/devpi_admin/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '1.0.0'
-__version_tuple__ = version_tuple = (1, 0, 0)
+__version__ = version = '1.0.1'
+__version_tuple__ = version_tuple = (1, 0, 1)
 
 __commit_id__ = commit_id = None

{devpi_admin-1.0.0 → devpi_admin-1.0.1}/src/devpi_admin/main.py

@@ -7,6 +7,7 @@ the standard devpi JSON API (``/+login``, ``/<user>/<index>``, ...).
 """
 from pathlib import Path
 
+from devpi_server.config import hookimpl
 from pyramid.httpexceptions import HTTPFound
 from pyramid.response import FileResponse
 
@@ -14,11 +15,12 @@ from pyramid.response import FileResponse
 STATIC_DIR = Path(__file__).parent / "static"
 
 
+@hookimpl
 def devpiserver_get_features():
-    # Advertise the plugin to devpi-server
     return {"devpi-admin"}
 
 
+@hookimpl
 def devpiserver_pyramid_configure(config, pyramid_config):
     # Serve bundled static assets (index.html, css/, js/) under /+admin/.
     pyramid_config.add_static_view(

devpi_admin-1.0.1/tests/test_hooks.py

@@ -0,0 +1,55 @@
+"""Tests that plugin hooks are properly registered with pluggy."""
+import unittest
+
+from pluggy import PluginManager
+
+from devpi_server import hookspecs
+import devpi_admin.main as plugin
+
+
+class HookRegistrationTests(unittest.TestCase):
+
+    def setUp(self):
+        self.pm = PluginManager("devpiserver")
+        self.pm.add_hookspecs(hookspecs)
+        self.pm.register(plugin)
+
+    def test_plugin_registered(self):
+        self.assertTrue(
+            self.pm.is_registered(plugin),
+            "plugin module not registered with pluggy")
+
+    def test_get_features_recognized(self):
+        results = self.pm.hook.devpiserver_get_features()
+        found = set()
+        for s in results:
+            found.update(s)
+        self.assertIn("devpi-admin", found)
+
+    def test_pyramid_configure_recognized(self):
+        callers = self.pm.hook.devpiserver_pyramid_configure.\
+            get_hookimpls()
+        modules = [impl.plugin for impl in callers]
+        self.assertIn(plugin, modules,
+                      "devpiserver_pyramid_configure not found in hook impls")
+
+    def test_no_unknown_hooks(self):
+        # All public devpiserver_* functions must be valid hookspecs.
+        # If a function has @hookimpl but a typo in the name, pluggy
+        # would reject it at register() — this test catches that.
+        hook_names = [
+            name for name in dir(plugin)
+            if name.startswith("devpiserver_") and callable(getattr(plugin, name))
+        ]
+        spec_names = [
+            name for name in dir(hookspecs)
+            if name.startswith("devpiserver_") and callable(getattr(hookspecs, name))
+        ]
+        for name in hook_names:
+            self.assertIn(
+                name, spec_names,
+                "{} is not a valid devpi-server hookspec".format(name))
+
+
+if __name__ == "__main__":
+    unittest.main()

devpi_admin-1.0.0/tests/__init__.py

@@ -1,8 +0,0 @@
-import warnings
-
-# Pyramid 2.1 imports pkg_resources at module load time and triggers a
-# setuptools deprecation warning. Silence it so test output stays clean.
-warnings.filterwarnings(
-    "ignore", category=UserWarning, module="pyramid.path")
-warnings.filterwarnings(
-    "ignore", category=DeprecationWarning, module="pkg_resources")