skip flaky test in aioredis (#1143)

The reason for the flakines is as of yet unknown. It looks like the `execute_pubsub` method sometimes isn't instrumented, which might point towards an import-time instrumentation-race. A new version of aioredis will be released soon with completely rewritten internals. The hope is that this issue will go away at that point. Using `pytest.mark.flaky` isn't an option, as `flaky` doesn't support async tests.

Author: beniwohli

tests/instrumentation/asyncio_tests/aioredis_tests.py

@@ -125,14 +125,15 @@ async def test_redis_client(instrument, elasticapm_client, redis_conn):
  assert len(spans) == 3
 
 
+@pytest.mark.skip(reason="Test is flaky for some reason, possibly related to import-time instrumentation")
 @pytest.mark.integrationtest
-async def test_publish_subscribe(instrument, elasticapm_client, redis_conn):
+async def test_publish_subscribe_async(instrument, elasticapm_client, redis_conn):
  elasticapm_client.begin_transaction("transaction.test")
  with capture_span("test_publish_subscribe", "test"):
  # publish
  await redis_conn.publish("mykey", "a")
 
- #subscribe
+ # subscribe
  await redis_conn.subscribe("mykey")
 
  elasticapm_client.end_transaction("MyView")