chore: run pre-commit

Signed-off-by: Danju Visvanathan <danju.visvanathan@gmail.com>

Author: danjuv

README.md

@@ -198,8 +198,8 @@ client = api.get_client()
 
 # trigger tracking event action
 client.track(
-    'visited-promo-page', 
-    evaluation_context=EvaluationContext(), 
+    'visited-promo-page',
+    evaluation_context=EvaluationContext(),
     tracking_event_details=openfeature.TrackingEventDetails(99.77).add("currencyCode", "USD"),
     )
 ```

openfeature/client.py

@@ -955,8 +955,13 @@ def add_handler(self, event: ProviderEvent, handler: EventHandler) -> None:
 
     def remove_handler(self, event: ProviderEvent, handler: EventHandler) -> None:
         _event_support.remove_client_handler(self, event, handler)
-    
-    def track(self, tracking_event_name: str, evaluation_context: EvaluationContext | None = None, tracking_event_details: TrackingEventDetails | None = None) -> None:
+
+    def track(
+        self,
+        tracking_event_name: str,
+        evaluation_context: EvaluationContext | None = None,
+        tracking_event_details: TrackingEventDetails | None = None,
+    ) -> None:
         """
         Tracks the occurrence of a particular action or application state.
 
@@ -967,7 +972,7 @@ def track(self, tracking_event_name: str, evaluation_context: EvaluationContext
 
         if not hasattr(self.provider, "track"):
             return
-        
+
         if evaluation_context is None:
             evaluation_context = EvaluationContext()
 
@@ -977,7 +982,10 @@ def track(self, tracking_event_name: str, evaluation_context: EvaluationContext
             .merge(self.context)
             .merge(evaluation_context)
         )
-        self.provider.track(tracking_event_name, merged_eval_context, tracking_event_details)
+        self.provider.track(
+            tracking_event_name, merged_eval_context, tracking_event_details
+        )
+
 
 def _typecheck_flag_value(
     value: typing.Any, flag_type: FlagType

openfeature/provider/__init__.py

@@ -117,7 +117,12 @@ async def resolve_object_details_async(
         Sequence[FlagValueType] | Mapping[str, FlagValueType]
     ]: ...
 
-    def track(self, tracking_event_name: str, evaluation_context: EvaluationContext | None = None, tracking_event_details: TrackingEventDetails | None = None) -> None: ...
+    def track(
+        self,
+        tracking_event_name: str,
+        evaluation_context: EvaluationContext | None = None,
+        tracking_event_details: TrackingEventDetails | None = None,
+    ) -> None: ...
 
 
 class AbstractProvider(FeatureProvider):
@@ -141,7 +146,12 @@ def initialize(self, evaluation_context: EvaluationContext) -> None:
     def shutdown(self) -> None:
         pass
 
-    def track(self, tracking_event_name: str, evaluation_context: EvaluationContext | None = None, tracking_event_details: TrackingEventDetails | None = None) -> None:
+    def track(
+        self,
+        tracking_event_name: str,
+        evaluation_context: EvaluationContext | None = None,
+        tracking_event_details: TrackingEventDetails | None = None,
+    ) -> None:
         pass
 
     @abstractmethod

openfeature/provider/in_memory_provider.py

@@ -22,12 +22,14 @@
 class InMemoryMetadata(Metadata):
     name: str = "In-Memory Provider"
 
+
 @dataclass
-class InMemoryTrackingEvent():
+class InMemoryTrackingEvent:
     value: float | None = None
     details: dict[str, typing.Any] = field(default_factory=dict)
-    eval_context_attributes: Mapping[str, EvaluationContextAttribute] = field(default_factory=dict)
-
+    eval_context_attributes: Mapping[str, EvaluationContextAttribute] = field(
+        default_factory=dict
+    )
 
 
 T_co = typing.TypeVar("T_co", covariant=True)
@@ -74,8 +76,11 @@ def resolve(
 class InMemoryProvider(AbstractProvider):
     _flags: FlagStorage
     _tracking_events: TrackingStorage
+
     # tracking_events defaults to an empty dict
-    def __init__(self, flags: FlagStorage, tracking_events: TrackingStorage | None = None) -> None:
+    def __init__(
+        self, flags: FlagStorage, tracking_events: TrackingStorage | None = None
+    ) -> None:
         self._flags = flags.copy()
         if tracking_events is not None:
             self._tracking_events = tracking_events.copy()
@@ -192,10 +197,23 @@ async def _resolve_async(
     ) -> FlagResolutionDetails[V]:
         return self._resolve(flag_key, default_value, evaluation_context)
 
-    def track(self, tracking_event_name: str, evaluation_context: EvaluationContext | None = None, tracking_event_details: TrackingEventDetails | None = None) -> None:
-        value = tracking_event_details.value if tracking_event_details is not None else None
-        details = tracking_event_details.attributes if tracking_event_details is not None else {}
-        eval_context_attributes = evaluation_context.attributes if evaluation_context is not None else {}
+    def track(
+        self,
+        tracking_event_name: str,
+        evaluation_context: EvaluationContext | None = None,
+        tracking_event_details: TrackingEventDetails | None = None,
+    ) -> None:
+        value = (
+            tracking_event_details.value if tracking_event_details is not None else None
+        )
+        details = (
+            tracking_event_details.attributes
+            if tracking_event_details is not None
+            else {}
+        )
+        eval_context_attributes = (
+            evaluation_context.attributes if evaluation_context is not None else {}
+        )
 
         self._tracking_events[tracking_event_name] = InMemoryTrackingEvent(
             value=value,

openfeature/provider/no_op_provider.py

@@ -84,5 +84,10 @@ def resolve_object_details(
             variant=PASSED_IN_DEFAULT,
         )
 
-    def track(self, tracking_event_name: str, evaluation_context: EvaluationContext | None = None, tracking_event_details: TrackingEventDetails | None = None) -> None:
+    def track(
+        self,
+        tracking_event_name: str,
+        evaluation_context: EvaluationContext | None = None,
+        tracking_event_details: TrackingEventDetails | None = None,
+    ) -> None:
         pass

openfeature/track/__init__.py

@@ -1,18 +1,25 @@
 from __future__ import annotations
 
-from collections.abc import Mapping, Sequence
 import typing
+from collections.abc import Mapping, Sequence
+
+TrackingValue: typing.TypeAlias = (
+    bool | int | float | str | Sequence["TrackingValue"] | Mapping[str, "TrackingValue"]
+)
 
-TrackingValue: typing.TypeAlias = bool | int | float | str | Sequence["TrackingValue"] | Mapping[str, "TrackingValue"]
 
 class TrackingEventDetails:
     value: float | None
     attributes: dict[str, TrackingValue]
 
-    def __init__(self, value: float | None = None, attributes: dict[str, TrackingValue] | None = None):
+    def __init__(
+        self,
+        value: float | None = None,
+        attributes: dict[str, TrackingValue] | None = None,
+    ):
         self.value = value
         self.attributes = attributes or {}
 
-    def add(self, key: str, value: TrackingValue) -> "TrackingEventDetails":
+    def add(self, key: str, value: TrackingValue) -> TrackingEventDetails:
         self.attributes[key] = value
         return self

tests/provider/test_in_memory_provider.py

@@ -1,13 +1,16 @@
 from numbers import Number
 
-from openfeature.evaluation_context import EvaluationContext
-from openfeature.provider.no_op_provider import NoOpProvider
-from openfeature.track import TrackingEventDetails
 import pytest
 
+from openfeature.evaluation_context import EvaluationContext
 from openfeature.exception import ErrorCode
 from openfeature.flag_evaluation import FlagResolutionDetails, Reason
-from openfeature.provider.in_memory_provider import InMemoryFlag, InMemoryProvider, InMemoryTrackingEvent
+from openfeature.provider.in_memory_provider import (
+    InMemoryFlag,
+    InMemoryProvider,
+    InMemoryTrackingEvent,
+)
+from openfeature.track import TrackingEventDetails
 
 
 def test_should_return_in_memory_provider_metadata():
@@ -204,5 +207,15 @@ async def test_should_track_event():
     provider = InMemoryProvider(
         {"Key": InMemoryFlag("hundred", {"zero": 0, "hundred": 100})}
     )
-    provider.track(tracking_event_name="test", evaluation_context=EvaluationContext(attributes={"key": "value"}), tracking_event_details=TrackingEventDetails(value=1, attributes={"key": "value"}))
-    assert provider._tracking_events == {"test": InMemoryTrackingEvent(value=1, details={"key": "value"}, eval_context_attributes={"key": "value"})}
+    provider.track(
+        tracking_event_name="test",
+        evaluation_context=EvaluationContext(attributes={"key": "value"}),
+        tracking_event_details=TrackingEventDetails(
+            value=1, attributes={"key": "value"}
+        ),
+    )
+    assert provider._tracking_events == {
+        "test": InMemoryTrackingEvent(
+            value=1, details={"key": "value"}, eval_context_attributes={"key": "value"}
+        )
+    }

tests/test_client.py

@@ -5,11 +5,17 @@
 from concurrent.futures import ThreadPoolExecutor
 from unittest.mock import MagicMock
 
-from openfeature.track import TrackingEventDetails
 import pytest
 
 from openfeature import api
-from openfeature.api import add_hooks, clear_hooks, get_client, set_evaluation_context, set_provider, set_transaction_context
+from openfeature.api import (
+    add_hooks,
+    clear_hooks,
+    get_client,
+    set_evaluation_context,
+    set_provider,
+    set_transaction_context,
+)
 from openfeature.client import OpenFeatureClient, _typecheck_flag_value
 from openfeature.evaluation_context import EvaluationContext
 from openfeature.event import EventDetails, ProviderEvent, ProviderEventDetails
@@ -625,21 +631,34 @@ def test_client_should_merge_contexts():
     assert context.attributes["client_attr"] == "client_value"
     assert context.attributes["invocation_attr"] == "invocation_value"
 
+
 def test_client_should_track_event():
     spy_provider = MagicMock(spec=NoOpProvider)
     set_provider(spy_provider)
     client = get_client()
     client.track(tracking_event_name="test")
     spy_provider.track.assert_called_once()
 
+
 def test_tracking_merges_evaluation_contexts():
     spy_provider = MagicMock(spec=NoOpProvider)
     api.set_provider(spy_provider)
     client = get_client()
     set_evaluation_context(EvaluationContext("id", attributes={"key": "eval_value"}))
-    set_transaction_context(EvaluationContext("id", attributes={"transaction_attr": "transaction_value"}))
-    client.track(tracking_event_name="test", evaluation_context=EvaluationContext("id", attributes={"key": "value"}))
-    spy_provider.track.assert_called_once_with("test", EvaluationContext("id", attributes={"transaction_attr": "transaction_value", "key": "value"}), None)
+    set_transaction_context(
+        EvaluationContext("id", attributes={"transaction_attr": "transaction_value"})
+    )
+    client.track(
+        tracking_event_name="test",
+        evaluation_context=EvaluationContext("id", attributes={"key": "value"}),
+    )
+    spy_provider.track.assert_called_once_with(
+        "test",
+        EvaluationContext(
+            "id", attributes={"transaction_attr": "transaction_value", "key": "value"}
+        ),
+        None,
+    )
 
 
 def test_should_noop_if_provider_does_not_support_tracking(monkeypatch):

tests/track/test_tracking.py

@@ -1,29 +1,38 @@
 from openfeature.track import TrackingEventDetails
-import pytest
+
 
 def test_add_attribute_to_tracking_event_details():
     tracking_event_details = TrackingEventDetails()
     tracking_event_details.add("key", "value")
     assert tracking_event_details.attributes == {"key": "value"}
 
+
 def test_add_attribute_to_tracking_event_details_dict():
     tracking_event_details = TrackingEventDetails()
     tracking_event_details.add("key", {"key1": "value1", "key2": "value2"})
-    assert tracking_event_details.attributes == {"key": {"key1": "value1", "key2": "value2"}}
+    assert tracking_event_details.attributes == {
+        "key": {"key1": "value1", "key2": "value2"}
+    }
+
 
 def test_get_value_from_tracking_event_details():
     tracking_event_details = TrackingEventDetails(value=1)
     assert tracking_event_details.value == 1
 
+
 def test_get_attributes_from_tracking_event_details():
-    tracking_event_details = TrackingEventDetails(value=5.0, attributes={"key": "value"})
+    tracking_event_details = TrackingEventDetails(
+        value=5.0, attributes={"key": "value"}
+    )
     assert tracking_event_details.attributes == {"key": "value"}
 
+
 def test_get_attributes_from_tracking_event_details_with_none_value():
     tracking_event_details = TrackingEventDetails(attributes={"key": "value"})
     assert tracking_event_details.attributes == {"key": "value"}
     assert tracking_event_details.value is None
 
+
 def test_get_attributes_from_tracking_event_details_with_none_attributes():
     tracking_event_details = TrackingEventDetails(value=5.0)
     assert tracking_event_details.attributes == {}