refactor(tracer): remove @Property for Tracer.context_provider (#1371)

That does not seem to be really useful.

Author: jd

ddtrace/tracer.py

@@ -171,12 +171,7 @@ async def web_handler(request):
         This method makes use of a ``ContextProvider`` that is automatically set during the tracer
         initialization, or while using a library instrumentation.
         """
-        return self._context_provider.active(*args, **kwargs)
-
-    @property
-    def context_provider(self):
-        """Returns the current Tracer Context Provider"""
-        return self._context_provider
+        return self.context_provider.active(*args, **kwargs)
 
     # TODO: deprecate this method and make sure users create a new tracer if they need different parameters
     @debtcollector.removals.removed_kwarg("dogstatsd_host", "Use `dogstatsd_url` instead",
@@ -290,7 +285,7 @@ def configure(
         self.writer.dogstatsd = self._dogstatsd_client
 
         if context_provider is not None:
-            self._context_provider = context_provider
+            self.context_provider = context_provider
 
         if wrap_executor is not None:
             self._wrap_executor = wrap_executor

tests/contrib/asyncio/test_tracer.py

@@ -341,7 +341,7 @@ def test_propagation_with_new_context(self):
     def test_event_loop_unpatch(self):
         # ensures that the event loop can be unpatched
         unpatch()
-        assert isinstance(self.tracer._context_provider, DefaultContextProvider)
+        assert isinstance(self.tracer.context_provider, DefaultContextProvider)
         assert asyncio.BaseEventLoop.create_task == _orig_create_task
 
     def test_event_loop_double_patch(self):