[SVLS-5262] remove unnecessary comment and add an exception for futur…

…e safety

ddtrace/_trace/utils_botocore/span_pointers.py

@@ -89,6 +89,11 @@ def _aws_s3_object_span_pointer_description(
 
 
 def _aws_s3_object_span_pointer_hash(bucket: str, key: str, etag: str) -> str:
+    if '"' in etag:
+        # Some AWS API endpoints put the ETag in double quotes. We expect the
+        # calling code to have correctly fixed this already.
+        raise ValueError(f"ETag should not have double quotes: {etag}")
+
     return _standard_hashing_function(
         bucket.encode("ascii"),
         key.encode("utf-8"),

tests/tracer/utils_botocore/test_span_pointers.py

@@ -24,10 +24,6 @@ class HashingCase(NamedTuple):
     @pytest.mark.parametrize(
         "hashing_case",
         [
-            # These test cases are copied from the dd-span-pointer-rules
-            # pointer_testset.json file. We will add a system test to ensure
-            # that these hashing functions are consistent now and in the future
-            # once this code is merged.
             HashingCase(
                 name="a basic S3 object",
                 bucket="some-bucket",