deepdive icon indicating copy to clipboard operation
deepdive copied to clipboard

Find code coverage/profiling tool (for bash/jq)

Open chrismre opened this issue 9 years ago • 3 comments

Looking at where we're missing on the coveralls, they seem pretty easy... it's a lot of malformed input stuff, etc.) We should make sure we handle failures gracefully... I know everyone is busy, but it highlights for us a bunch of easy to fix tests.

To be concrete, it looks like we can get into the high 80s or low 90s with a code cleanup pass.

@SenWu is probably in charge of some of this? and I would also guess @netj oversees it?

chrismre avatar May 25 '15 23:05 chrismre

Increasing test coverage will happen in the next release.

netj avatar Jun 04 '15 21:06 netj

:(

chrismre avatar Jan 08 '16 07:01 chrismre

Due to the lack of code coverage tools for our primary languages: bash and jq, code coverage and coveralls badge has been dropped. I'd say we should do a more thorough code review of these less bloated code. Nobody would argue that one should write database queries in C just because their coverage can be easily measured while it's not so for SQL. Anyway, having a simple profiling tool that shows which commands are actually unused or used how many times may be useful, though. Let's keep our eyes open looking for such tool and track it in this issue.

netj avatar Feb 19 '16 19:02 netj