roaring icon indicating copy to clipboard operation
roaring copied to clipboard

Our coverage is below 90%

Open lemire opened this issue 9 years ago • 12 comments

More testing is needed:

https://gocover.io/github.com/RoaringBitmap/roaring

lemire avatar Apr 08 '16 15:04 lemire

The URL gives an 404 error.

cornelk avatar Jul 25 '16 10:07 cornelk

Looks like gocover.io went under?

lemire avatar Jul 25 '16 11:07 lemire

https://coveralls.io/ looks like a good alternative, free for open source projects.

cornelk avatar Jul 25 '16 21:07 cornelk

I use coveralls.io in other non-Go projects, but I do not know how to use it with Go. Do you?

lemire avatar Jul 25 '16 21:07 lemire

No, I never used it. I saw another Go project using it: https://coveralls.io/github/go-gorp/gorp

cornelk avatar Jul 25 '16 22:07 cornelk

I know that some projects use it, but I tried to follow the fuzzy instructions and it doesn't work. There are some mysterious steps involved. Looks like magic.

lemire avatar Jul 25 '16 23:07 lemire

https://coveralls.io/github/RoaringBitmap/roaring

lemire avatar Jul 25 '16 23:07 lemire

I added the coverage badge to the README, the website still does not show content for this project: "There have been no builds for this repo."

cornelk avatar Jul 26 '16 10:07 cornelk

@cornelk

I am aware that it does not yet work. I tried something else this morning. Let us see.

lemire avatar Jul 26 '16 14:07 lemire

Coverage badge should be ok.

lemire avatar Jul 26 '16 16:07 lemire

Right. So the badge is ok.

lemire avatar Jul 26 '16 18:07 lemire

Great, I started working on some tests to test the panic edge cases now.

cornelk avatar Jul 26 '16 19:07 cornelk