Cats-effect: Improve code coverage for cats.effect.concurrent

Created on 28 Aug 2018  路  5Comments  路  Source: typelevel/cats-effect

We've got some function implementations not being tested in Ref and Semaphore and we have to cover them:

chore good first issue help wanted
Source
picture alexandru

Most helpful comment

WDYT about a good first issue label for things like this?

picture SystemFw on 28 Aug 2018
👍4

All 5 comments

WDYT about a good first issue label for things like this?

SystemFw picture SystemFw on 28 Aug 2018
👍4

I will add tests for Semaphore :)

kun-song picture kun-song on 29 Sep 2018

@satansk are you still working on it? If not I will be super happy to take it over form you, if you are still working on it please let me know - i'll find other issue to solve :)

CucumisSativus picture CucumisSativus on 15 Oct 2018

@CucumisSativus it's yours :)

kun-song picture kun-song on 16 Oct 2018

Looks like MVar is missing quite a lot of tests, I'll take this possibility and write tem

CucumisSativus picture CucumisSativus on 21 Jan 2019
❤1
Was this page helpful?
0 / 5 - 0 ratings

Related issues

ronanM picture ronanM  路  5Comments
hilios picture hilios  路  5Comments
durban picture durban  路  4Comments
Odomontois picture Odomontois  路  8Comments
guizmaii picture guizmaii  路  3Comments