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

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

Author: pre-commit-ci[bot]

docs/source/users/kernel-envs.md

@@ -80,7 +80,7 @@ There are several supported `KERNEL_` variables that the Enterprise Gateway serv
     (e.g "{{ kernel_prefix }}-{{ kernel_id | replace('-', '') }}")
     which will be processed for safe substitution against existing list
     of environment variables. In case of invalid template (e.g. missing variables)
-    it will fall back to original way to calculate the pod name using 
+    it will fall back to original way to calculate the pod name using
     KERNEL_USERNAME - KERNEL_ID.
 
   KERNEL_REMOTE_HOST=<remote host name>

enterprise_gateway/services/processproxies/k8s.py

@@ -240,7 +240,9 @@ def replace_var(match):
         # Check if there are any remaining {{ }} patterns that didn't match our simple pattern
         # This catches malicious templates like {{ foo.__class__ }} or {{ 1+1 }}
         if '{{' in result and '}}' in result:
-            self.log.warning(f"Invalid template syntax detected in KERNEL_POD_NAME: contains unsupported expressions")
+            self.log.warning(
+                "Invalid template syntax detected in KERNEL_POD_NAME: contains unsupported expressions"
+            )
             return None
 
         # Log missing variables and return None if any are missing

enterprise_gateway/tests/test_process_proxy.py

@@ -6,8 +6,7 @@
 from unittest.mock import Mock, patch
 
 # Mock Kubernetes configuration before importing the module
-with patch('kubernetes.config.load_incluster_config'), \
-     patch('kubernetes.config.load_kube_config'):
+with patch('kubernetes.config.load_incluster_config'), patch('kubernetes.config.load_kube_config'):
     from enterprise_gateway.services.processproxies.k8s import KubernetesProcessProxy
 
 
@@ -19,15 +18,14 @@ def setUp(self):
         self.mock_kernel_manager = Mock()
         self.mock_kernel_manager.get_kernel_username.return_value = "testuser"
         self.mock_kernel_manager.port_range = "0..0"  # Mock port range
-        
+
         # Mock proxy config
-        self.proxy_config = {
-            "kernel_id": "test-kernel-id",
-            "kernel_name": "python3"
-        }
+        self.proxy_config = {"kernel_id": "test-kernel-id", "kernel_name": "python3"}
 
         # Mock KernelSessionManager methods
-        with patch('enterprise_gateway.services.processproxies.k8s.KernelSessionManager') as mock_session_manager:
+        with patch(
+            'enterprise_gateway.services.processproxies.k8s.KernelSessionManager'
+        ) as mock_session_manager:
             mock_session_manager.get_kernel_username.return_value = "testuser"
             self.proxy = KubernetesProcessProxy(self.mock_kernel_manager, self.proxy_config)
             self.proxy.kernel_id = "test-kernel-id"
@@ -54,7 +52,7 @@ def test_valid_template_substitution(self):
                 result = self.proxy._safe_template_substitute(template, variables)
                 self.assertEqual(result, expected)
 
-    def test_missing_variables_fallback(self):        
+    def test_missing_variables_fallback(self):
         # Test the full pod name determination process
         kwargs = {
             "env": {
@@ -63,8 +61,9 @@ def test_missing_variables_fallback(self):
             }
         }
 
-        with patch.object(self.proxy, 'log'), \
-             patch('enterprise_gateway.services.processproxies.k8s.KernelSessionManager') as mock_session_manager:
+        with patch.object(self.proxy, 'log'), patch(
+            'enterprise_gateway.services.processproxies.k8s.KernelSessionManager'
+        ) as mock_session_manager:
             mock_session_manager.get_kernel_username.return_value = "testuser"
             result = self.proxy._determine_kernel_pod_name(**kwargs)
             # Should fall back to default naming: kernel_username + "-" + kernel_id
@@ -132,8 +131,9 @@ def test_pod_name_determination_with_malicious_template(self):
             }
         }
 
-        with patch.object(self.proxy, 'log'), \
-             patch('enterprise_gateway.services.processproxies.k8s.KernelSessionManager') as mock_session_manager:
+        with patch.object(self.proxy, 'log'), patch(
+            'enterprise_gateway.services.processproxies.k8s.KernelSessionManager'
+        ) as mock_session_manager:
             mock_session_manager.get_kernel_username.return_value = "testuser"
             result = self.proxy._determine_kernel_pod_name(**kwargs)
             # Should fall back to default naming
@@ -148,8 +148,9 @@ def test_pod_name_determination_with_missing_variables(self):
             }
         }
 
-        with patch.object(self.proxy, 'log'), \
-             patch('enterprise_gateway.services.processproxies.k8s.KernelSessionManager') as mock_session_manager:
+        with patch.object(self.proxy, 'log'), patch(
+            'enterprise_gateway.services.processproxies.k8s.KernelSessionManager'
+        ) as mock_session_manager:
             mock_session_manager.get_kernel_username.return_value = "testuser"
             result = self.proxy._determine_kernel_pod_name(**kwargs)
             # Should fall back to default naming