add get_dist_dependency_conflicts back

Signed-off-by: emdneto <[email protected]>

Author: emdneto

opentelemetry-instrumentation/src/opentelemetry/instrumentation/dependencies.py

@@ -14,12 +14,14 @@
 
 from __future__ import annotations
 
+import warnings
 from logging import getLogger
 from typing import Collection
 
 from packaging.requirements import InvalidRequirement, Requirement
 
 from opentelemetry.util._importlib_metadata import (
+    Distribution,
     PackageNotFoundError,
     version,
 )
@@ -49,6 +51,30 @@ def __str__(self):
         return str(self.conflict)
 
 
+def get_dist_dependency_conflicts(
+    dist: Distribution,
+) -> DependencyConflict | None:
+    warnings.warn(
+        "get_dist_dependency_conflicts is deprecated since 0.53b0 and will be removed in a future release.",
+        DeprecationWarning,
+        stacklevel=2,
+    )
+    instrumentation_deps = []
+    extra = "extra"
+    instruments = "instruments"
+    instruments_marker = {extra: instruments}
+    if dist.requires:
+        for dep in dist.requires:
+            if extra not in dep or instruments not in dep:
+                continue
+
+            req = Requirement(dep)
+            if req.marker.evaluate(instruments_marker):
+                instrumentation_deps.append(req)
+
+    return get_dependency_conflicts(instrumentation_deps)
+
+
 def get_dependency_conflicts(
     deps: Collection[str | Requirement],
 ) -> DependencyConflict | None:

opentelemetry-instrumentation/tests/test_dependencies.py

@@ -20,8 +20,10 @@
 from opentelemetry.instrumentation.dependencies import (
     DependencyConflict,
     get_dependency_conflicts,
+    get_dist_dependency_conflicts,
 )
 from opentelemetry.test.test_base import TestBase
+from opentelemetry.util._importlib_metadata import Distribution
 
 
 class TestDependencyConflicts(TestBase):
@@ -62,3 +64,41 @@ def test_get_dependency_conflicts_mismatched_version(self):
             str(conflict),
             f'DependencyConflict: requested: "pytest == 5000" but found: "pytest {pytest.__version__}"',
         )
+
+    def test_get_dist_dependency_conflicts(self):
+        class MockDistribution(Distribution):
+            def locate_file(self, path):
+                pass
+
+            def read_text(self, filename):
+                pass
+
+            @property
+            def requires(self):
+                return ['test-pkg ~= 1.0; extra == "instruments"']
+
+        dist = MockDistribution()
+
+        conflict = get_dist_dependency_conflicts(dist)
+        self.assertTrue(conflict is not None)
+        self.assertTrue(isinstance(conflict, DependencyConflict))
+        self.assertEqual(
+            str(conflict),
+            'DependencyConflict: requested: "test-pkg~=1.0; extra == "instruments"" but found: "None"',
+        )
+
+    def test_get_dist_dependency_conflicts_requires_none(self):
+        class MockDistribution(Distribution):
+            def locate_file(self, path):
+                pass
+
+            def read_text(self, filename):
+                pass
+
+            @property
+            def requires(self):
+                return None
+
+        dist = MockDistribution()
+        conflict = get_dist_dependency_conflicts(dist)
+        self.assertTrue(conflict is None)