fix(asyncio): avoid overwriting coroutine names (#12011)

Resolves: #10975

## Checklist
- [x] PR author has checked that all the criteria below are met
- The PR description includes an overview of the change
- The PR description articulates the motivation for the change
- The change includes tests OR the PR description describes a testing
strategy
- The PR description notes risks associated with the change, if any
- Newly-added code is easy to change
- The change follows the [library release note
guidelines](https://ddtrace.readthedocs.io/en/stable/releasenotes.html)
- The change includes or references documentation updates if necessary
- Backport labels are set (if
[applicable](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting))

## Reviewer Checklist
- [ ] Reviewer has checked that all the criteria below are met 
- Title is accurate
- All changes are related to the pull request's stated goal
- Avoids breaking
[API](https://ddtrace.readthedocs.io/en/stable/versioning.html#interfaces)
changes
- Testing strategy adequately addresses listed risks
- Newly-added code is easy to change
- Release note makes sense to a user of the library
- If necessary, author has acknowledged and discussed the performance
implications of this PR as reported in the benchmarks PR comment
- Backport labels are set in a manner that is consistent with the
[release branch maintenance
policy](https://ddtrace.readthedocs.io/en/latest/contributing.html#backporting)

Author: mabdinur

ddtrace/contrib/internal/asyncio/patch.py

@@ -49,5 +49,12 @@ async def traced_coro(*args_c, **kwargs_c):
             pin.tracer.context_provider.activate(dd_active)
         return await coro
 
-    args, kwargs = set_argument_value(args, kwargs, 1, "coro", traced_coro())
+    # DEV: try to persist the original function name (useful for debugging)
+    tc = traced_coro()
+    if hasattr(coro, "__name__"):
+        tc.__name__ = coro.__name__
+    if hasattr(coro, "__qualname__"):
+        tc.__qualname__ = coro.__qualname__
+    args, kwargs = set_argument_value(args, kwargs, 1, "coro", tc)
+
     return wrapped(*args, **kwargs)

releasenotes/notes/avoid-overriding-couroutine-names-0852ea19200feb74.yaml

@@ -0,0 +1,4 @@
+---
+fixes:
+  - |
+    asyncio: Fixes an issue where the name of a coroutine was being overridden by a ddtrace function.

tests/contrib/asyncio/test_tracer.py

@@ -1,5 +1,7 @@
 """Ensure that the tracer works with asynchronous executions within the same ``IOLoop``."""
 import asyncio
+import os
+import re
 
 import pytest
 
@@ -193,3 +195,31 @@ async def test():
     spans = tracer.get_spans()
     assert len(spans) == 3
     assert spans[0].trace_id == spans[1].trace_id == spans[2].trace_id
+
+
+def test_asyncio_scheduled_tasks_debug_logs(run_python_code_in_subprocess):
+    code = """
+import ddtrace
+
+ddtrace.patch(asyncio=True)
+import asyncio
+import time
+import pytest
+
+
+async def my_function():
+    time.sleep(2)
+
+if __name__ == "__main__":
+    asyncio.run(my_function())
+"""
+    env = os.environ.copy()
+    env["PYTHONASYNCIODEBUG"] = "1"
+    out, err, status, _ = run_python_code_in_subprocess(code, env=env)
+    assert status == 0, err + out
+
+    pattern = rb"Executing <Task finished name=\'Task-1\' coro=<my_function\(\) done, "
+    rb"defined at .*/dd-trace-py/ddtrace/contrib/internal/asyncio/patch.py:.* result=None "
+    rb"created at .*/dd-trace-py/ddtrace/contrib/internal/asyncio/patch.py:.* took .* seconds"
+    match = re.match(pattern, err)
+    assert match, err