Cleanup op and description mapping

[sl0thentr0py]

The main change is that op is now optional and name and description are basically interchangeable and map to each other as best as possible.

Preference is given to explicitly set values (SentrySpanAttribute) and if not (in the case of pure otel spans), we try to derive them. The entire flow is much cleaner now instead of repeated calls to the same attributes.

(For the main mapping changes in opentelemetry/test_utils I'd recommend reading the new code instead of the diff.)

[codecov]

❌ 33 Tests Failed:

Tests completed	Failed	Passed	Skipped
21176	33	21143	4366

View the top 3 failed test(s) by shortest run time

tests.integrations.redis.test_redis::test_data_truncation_custom

Stack Traces | 0.079s run time

.../integrations/redis/test_redis.py:209: in test_data_truncation_custom
    assert (
E   assert '- op=null: description=null\n  - op="db.redis": description="SET \'somekey1\' \'aaaaaaaaaaa..."\n  - op="db.redis": description="SET \'somekey2\' \'bbbbbbbbbb\'"' == '- op="<unlabeled span>": description=null\n  - op="db.redis": description="SET \'somekey1\' \'aaaaaaaaaaa..."\n  - op="db.redis": description="SET \'somekey2\' \'bbbbbbbbbb\'"'
E     - - op="<unlabeled span>": description=null
E     ?      ^^ - --- ---------
E     + - op=null: description=null
E     ?      ^
E         - op="db.redis": description="SET 'somekey1' 'aaaaaaaaaaa..."
E         - op="db.redis": description="SET 'somekey2' 'bbbbbbbbbb'"

tests.integrations.redis.test_redis::test_sensitive_data

Stack Traces | 0.079s run time

.../integrations/redis/test_redis.py:103: in test_sensitive_data
    assert (
E   assert '- op=null: description=null\n  - op="db.redis": description="GET [Filtered]"' == '- op="<unlabeled span>": description=null\n  - op="db.redis": description="GET [Filtered]"'
E     - - op="<unlabeled span>": description=null
E     ?      ^^ - --- ---------
E     + - op=null: description=null
E     ?      ^
E         - op="db.redis": description="GET [Filtered]"

tests.integrations.redis.test_redis_cache_module_async::test_cache_basic

Stack Traces | 0.079s run time

.../integrations/redis/test_redis_cache_module_async.py:64: in test_cache_basic
    assert (
E   assert '- op=null: description=null\n  - op="cache.get": description="myasynccachekey"\n    - op="db.redis": description="GET \'myasynccachekey\'"' == '- op="<unlabeled span>": description=null\n  - op="cache.get": description="myasynccachekey"\n    - op="db.redis": description="GET \'myasynccachekey\'"'
E     - - op="<unlabeled span>": description=null
E     ?      ^^ - --- ---------
E     + - op=null: description=null
E     ?      ^
E         - op="cache.get": description="myasynccachekey"
E           - op="db.redis": description="GET 'myasynccachekey'"

To view more test analytics, go to the Test Analytics Dashboard
_{📋 Got 3 mins? Take this short survey to help us improve Test Analytics.}

[sl0thentr0py]

these were all None before anyway

[sl0thentr0py]

here and in all subsequent tests, I made it clear that the transaction name is correct and that the op of the root span is not the transaction name anymore