Test tokio with other libraries

[jettify]

• test tokio aio-libs/aiorwlock#35 passes
• Aadd tokio tests aio-libs/aiomonitor#41 fails

[fafhrd91]

Awesome! Thanks! Will check aiomonitor.

…

On May 10, 2017, at 1:29 PM, Nikolay Novik ***@***.***> wrote: aio-libs/aiorwlock#35 <aio-libs/aiorwlock#35> passes aio-libs/aiomonitor#41 <aio-libs/aiomonitor#41> fails — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#34>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAkjzmV0pbWDiXZ2SoXrmNlalxlZJv2jks5r4h5CgaJpZM4NXLr->.

[jettify]

With tokio loop I am getting bunch of errors RuntimeError: Timeout context manager should be used inside a task when using loop.run_until_complete(coro) as result tests does not have a chance to start since such idiom ofter used in test fixtures. (asyncio and uvloop works as expected)

    @pytest.fixture
    def bucket_name(region, create_bucket, s3_client, loop):
>       name = loop.run_until_complete(create_bucket(region))
E       RuntimeError: Timeout context manager should be used inside a task

[fafhrd91]

This is because Task.current_task() does not work with tokio loop. Yury Selivanov will will info loop.current_task() to asyncio pep. I added “loop.current_task()” to latest async_timeout and to aiohttp master.

…

On May 10, 2017, at 1:56 PM, Nikolay Novik ***@***.***> wrote: With tokio loop I am getting bunch of errors RuntimeError: Timeout context manager should be used inside a task when using loop.run_until_complete(coro) as result tests does not have a chance to start since such idiom ofter used in test fixtures. (asyncio and uvloop works as expected) @pytest.fixture def bucket_name(region, create_bucket, s3_client, loop): > name = loop.run_until_complete(create_bucket(region)) E RuntimeError: Timeout context manager should be used inside a task — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#34 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAkjzlQGfoNZ_WGTl_3U7Nt1FiB0kblaks5r4iSNgaJpZM4NXLr->.

[1st1]

Does tokio create its own rust Tasks? Why not Futures instead?

[fafhrd91]

@jettify aiomonitor passes tests with aiohttp master

[fafhrd91]

• https://github.com/aio-libs/aioredis - tests pass. some xfail tests pass as well, which is not good

[jettify]

I will merge those PRs soon, so tokio/uvloop can be tested on each PR continuously.

[fafhrd91]

@jettify I think we need separate package for various asyncio related fixtures.
every library that test different event loops implements loop fixture

[jettify]

@fafhrd91 agreed there are few libraries, but not sure I like them (https://pypi.python.org/pypi/pytest-asyncio)

[fafhrd91]

But we have enough fixters in aiohttp and other aio libs, we just need to extract those to separate package