chore: linter fixes

Author: jsalaber

devcycle_python_sdk/models/config_metadata.py

@@ -22,4 +22,4 @@ def from_json(json_str: str) -> "ConfigMetadata":
         return ConfigMetadata(
             project=ProjectMetadata.from_json(json_str["project"]),
             environment=EnvironmentMetadata.from_json(json_str["environment"]),
-        )
+        )

devcycle_python_sdk/models/environment_metadata.py

@@ -14,4 +14,4 @@ def from_json(json_str: str) -> "EnvironmentMetadata":
         return EnvironmentMetadata(
             id=json_str["id"],
             key=json_str["key"],
-        )
+        )

devcycle_python_sdk/models/eval_hook_context.py

@@ -3,8 +3,15 @@
 from devcycle_python_sdk.models.user import DevCycleUser
 from devcycle_python_sdk.models.config_metadata import ConfigMetadata
 
+
 class HookContext:
-    def __init__(self, key: str, user: DevCycleUser, default_value: Any, config_metadata: ConfigMetadata = None):
+    def __init__(
+        self,
+        key: str,
+        user: DevCycleUser,
+        default_value: Any,
+        config_metadata: ConfigMetadata = None,
+    ):
         self.key = key
         self.default_value = default_value
         self.user = user

devcycle_python_sdk/models/project_metadata.py

@@ -14,4 +14,4 @@ def from_json(json_str: str) -> "ProjectMetadata":
         return ProjectMetadata(
             id=json_str["id"],
             key=json_str["key"],
-        )
+        )

devcycle_python_sdk/options.py

@@ -90,7 +90,7 @@ def __init__(
 
         if self.event_request_chunk_size > self.max_event_queue_size:
             logger.warning(
-                f"DevCycle: event_request_chunk_size: {self.event_request_chunk_size} must be smaller than max_event_queue_size: { self.max_event_queue_size}"
+                f"DevCycle: event_request_chunk_size: {self.event_request_chunk_size} must be smaller than max_event_queue_size: {self.max_event_queue_size}"
             )
             self.event_request_chunk_size = 100
 

devcycle_python_sdk/protobuf/utils.py

@@ -34,11 +34,17 @@ def create_nullable_custom_data(val: Optional[dict]) -> pb2.NullableCustomData:
             if value is None:
                 values[key] = pb2.CustomDataValue(type=pb2.CustomDataType.Null)  # type: ignore
             elif isinstance(value, bool):
-                values[key] = pb2.CustomDataValue(type=pb2.CustomDataType.Bool, boolValue=value)  # type: ignore
+                values[key] = pb2.CustomDataValue(
+                    type=pb2.CustomDataType.Bool, boolValue=value
+                )  # type: ignore
             elif isinstance(value, str):
-                values[key] = pb2.CustomDataValue(type=pb2.CustomDataType.Str, stringValue=value)  # type: ignore
+                values[key] = pb2.CustomDataValue(
+                    type=pb2.CustomDataType.Str, stringValue=value
+                )  # type: ignore
             elif isinstance(value, (int, float)):
-                values[key] = pb2.CustomDataValue(type=pb2.CustomDataType.Num, doubleValue=value)  # type: ignore
+                values[key] = pb2.CustomDataValue(
+                    type=pb2.CustomDataType.Num, doubleValue=value
+                )  # type: ignore
             else:
                 logger.warning(
                     f"Custom Data contains data type that can't be written, will be ignored. Key: {key}, Type: {str(type(value))}"

devcycle_python_sdk/protobuf/variableForUserParams_pb2.py

@@ -1,5 +1,6 @@
 #!/usr/bin/env python
 """Django's command-line utility for administrative tasks."""
+
 import os
 import sys
 

example/django-app/manage.py

@@ -375,9 +375,9 @@ def test_context_has_null_config_metadata(self, mock_variable_call):
         mock_variable_call.return_value = Variable(
             _id="123", key="strKey", value=999, type=TypeEnum.NUMBER
         )
-        
+
         context_received = None
-        
+
         def before_hook(context):
             nonlocal context_received
             context_received = context
@@ -398,9 +398,11 @@ def error_hook(context, error):
 
         # Test that context has config_metadata field but it's null for cloud client
         variable = self.test_client.variable(self.test_user, "strKey", 42)
-        
+
+        # Verify the variable evaluation worked
+        self.assertIsNotNone(variable)
         self.assertIsNotNone(context_received)
-        self.assertTrue(hasattr(context_received, 'config_metadata'))
+        self.assertTrue(hasattr(context_received, "config_metadata"))
         # Cloud client should have null config_metadata since it's not implemented
         self.assertIsNone(context_received.config_metadata)
 

test/test_cloud_client.py

@@ -460,9 +460,9 @@ def error_hook(context, error):
     @responses.activate
     def test_context_has_config_metadata(self):
         self.setup_client()
-        
+
         context_received = None
-        
+
         def before_hook(context):
             nonlocal context_received
             context_received = context
@@ -485,18 +485,26 @@ def error_hook(context, error):
 
         # Test that context has config_metadata field
         variable = self.client.variable(user, "num-var", 42)
-        
+
+        # Verify the variable evaluation worked
+        self.assertIsNotNone(variable)
         self.assertIsNotNone(context_received)
-        self.assertTrue(hasattr(context_received, 'config_metadata'))
+        self.assertTrue(hasattr(context_received, "config_metadata"))
         # For local client, config_metadata should be populated
         self.assertIsNotNone(context_received.config_metadata)
-        self.assertTrue(hasattr(context_received.config_metadata, 'project'))
-        self.assertTrue(hasattr(context_received.config_metadata, 'environment'))
+        self.assertTrue(hasattr(context_received.config_metadata, "project"))
+        self.assertTrue(hasattr(context_received.config_metadata, "environment"))
         # Verify the project and environment data
-        self.assertEqual(context_received.config_metadata.project.id, "61f97628ff4afcb6d057dbf0")
+        self.assertEqual(
+            context_received.config_metadata.project.id, "61f97628ff4afcb6d057dbf0"
+        )
         self.assertEqual(context_received.config_metadata.project.key, "emma-project")
-        self.assertEqual(context_received.config_metadata.environment.id, "61f97628ff4afcb6d057dbf2")
-        self.assertEqual(context_received.config_metadata.environment.key, "development")
+        self.assertEqual(
+            context_received.config_metadata.environment.id, "61f97628ff4afcb6d057dbf2"
+        )
+        self.assertEqual(
+            context_received.config_metadata.environment.key, "development"
+        )
 
 
 def _benchmark_variable_call(client: DevCycleLocalClient, user: DevCycleUser, key: str):