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")