remove unnecessary track definitions

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

Author: danjuv

openfeature/client.py

@@ -970,9 +970,6 @@ def track(
         :param tracking_event_details: Optional data relevant to the tracking event
         """
 
-        if not hasattr(self.provider, "track"):
-            return
-
         if evaluation_context is None:
             evaluation_context = EvaluationContext()
 

openfeature/provider/no_op_provider.py

@@ -83,11 +83,3 @@ def resolve_object_details(
             reason=Reason.DEFAULT,
             variant=PASSED_IN_DEFAULT,
         )
-
-    def track(
-        self,
-        tracking_event_name: str,
-        evaluation_context: EvaluationContext | None = None,
-        tracking_event_details: TrackingEventDetails | None = None,
-    ) -> None:
-        pass

tests/test_client.py

@@ -663,7 +663,6 @@ def test_tracking_merges_evaluation_contexts():
 
 def test_should_noop_if_provider_does_not_support_tracking(monkeypatch):
     provider = NoOpProvider()
-    monkeypatch.delattr(NoOpProvider, "track", False)
     set_provider(provider)
     client = get_client()
     client.track(tracking_event_name="test")