[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

jupyter_server/prometheus/server.py

@@ -234,7 +234,7 @@ def stop(self) -> None:
             self.http_server.stop()
             self.http_server = None
 
-        if hasattr(self, 'ioloop') and self.ioloop:
+        if hasattr(self, "ioloop") and self.ioloop:
             # Stop the IOLoop
             try:
                 self.ioloop.add_callback(self.ioloop.stop)

tests/test_metrics.py

@@ -17,7 +17,7 @@ def find_available_port(start_port=9090, max_attempts=10):
         port = start_port + i
         try:
             with socket.socket(socket.AF_INET, socket.SOCK_STREAM) as s:
-                s.bind(('localhost', port))
+                s.bind(("localhost", port))
                 return port
         except OSError:
             continue
@@ -74,7 +74,7 @@ def test_metrics_server_startup(standalone_metrics_server):
     """Test that metrics server starts correctly."""
     assert standalone_metrics_server.port is not None
     assert standalone_metrics_server.port > 0
-    
+
     # Test that metrics endpoint is accessible
     response = wait_for_server(f"http://localhost:{standalone_metrics_server.port}/metrics")
     assert response.status_code == 200
@@ -84,70 +84,64 @@ def test_metrics_server_startup(standalone_metrics_server):
 def test_metrics_server_with_authentication():
     """Test metrics server with authentication enabled."""
     port = find_available_port(9092)
-    
+
     # Create a server app with authentication
     with patch.dict("os.environ", {"JUPYTER_SERVER_METRICS_PORT": str(port)}):
         app = ServerApp()
         app.metrics_port = port
         app.authenticate_prometheus = True
         app.initialize([])
-        
+
         # Start the app
         app.start_app()
-        
+
         # Wait for both servers to be ready
         time.sleep(1.0)
-        
+
         try:
             # Get the token
             token = app.identity_provider.token
-            
+
             # Test metrics endpoint with token
-            response = wait_for_server(
-                f"http://localhost:{port}/metrics?token={token}",
-                timeout=5
-            )
+            response = wait_for_server(f"http://localhost:{port}/metrics?token={token}", timeout=5)
             assert response.status_code == 200
             assert "jupyter_server_info" in response.text
-            
+
             # Test without token should fail
             try:
                 response = requests.get(f"http://localhost:{port}/metrics", timeout=2)
                 assert response.status_code == 403
             except requests.exceptions.ConnectionError:
                 # Server might not be ready yet, which is also acceptable
                 pass
-            
+
         finally:
             app.stop()
 
 
 def test_metrics_server_without_authentication():
     """Test metrics server without authentication."""
     port = find_available_port(9093)
-    
+
     # Create a server app without authentication
     with patch.dict("os.environ", {"JUPYTER_SERVER_METRICS_PORT": str(port)}):
         app = ServerApp()
         app.metrics_port = port
         app.authenticate_prometheus = False
         app.initialize([])
-        
+
         # Start the app
         app.start_app()
-        
+
         # Wait for both servers to be ready
         time.sleep(1.0)
-        
+
         try:
             # Test metrics endpoint without token should work
-            response = wait_for_server(
-                f"http://localhost:{port}/metrics",
-                timeout=5
-            )
+            response = wait_for_server(f"http://localhost:{port}/metrics", timeout=5)
             assert response.status_code == 200
             assert "jupyter_server_info" in response.text
-            
+
         finally:
             app.stop()
 
@@ -156,20 +150,20 @@ def test_metrics_server_port_conflict():
     """Test that metrics server handles port conflicts gracefully."""
     # Use a port that's likely to be in use
     port = 8888  # Default Jupyter port
-    
+
     # Create a server app that should fail to start metrics server
     with patch.dict("os.environ", {"JUPYTER_SERVER_METRICS_PORT": str(port)}):
         app = ServerApp()
         app.metrics_port = port
         app.initialize([])
-        
+
         # Start the app - should not crash
         app.start_app()
-        
+
         try:
             # The app should still be running even if metrics server failed
             assert app.http_server is not None
-            
+
         finally:
             app.stop()
 
@@ -180,22 +174,21 @@ def test_metrics_server_disabled():
         app = ServerApp()
         app.metrics_port = 0
         app.initialize([])
-        
+
         # Start the app
         app.start_app()
-        
+
         # Wait for server to be ready
         time.sleep(0.5)
-        
+
         try:
             # Metrics should be available on main server
             token = app.identity_provider.token
             response = wait_for_server(
-                f"http://localhost:{app.port}/metrics?token={token}",
-                timeout=5
+                f"http://localhost:{app.port}/metrics?token={token}", timeout=5
             )
             assert response.status_code == 200
             assert "jupyter_server_info" in response.text
-            
+
         finally:
             app.stop()