allow to redefine root directory, merge suites and report to external systems #13
Conversation
albfan
force-pushed
the
master
branch
9 times, most recently
from
3953521 to
f54aa09
Compare
Using test suites based on sandbox, allow to define command name to avoid clobber results. For detailed report, allow to rename coverage dir.
Mimic merge feature from simplecov
albfan
changed the title
|
All this stuff is working https://coveralls.io/github/albfan/git-ignore Let's cover new features to pass bashcov filters! |
albfan
force-pushed
the
master
branch
11 times, most recently
from
d63ba92 to
8832b69
Compare
albfan
force-pushed
the
master
branch
4 times, most recently
from
7958fa7 to
bb546f9
Compare
construct like
cat <<EOF
a large amount of text
with complex indentation
and variable resolution $HOME
EOF
are ignored by xtrace output. Detect those lines
|
Code coverage fails to show 100% with I suppose is something related with a timeout or the like |
| formatters.push(r[:formatter].split("::").inject(Object) { |a, e| a.const_get e }) | ||
| end | ||
| SimpleCov.formatters = formatters | ||
| end |
There was a problem hiding this comment.
I don't think we need to reimplement that in Bashcov. I just tried this and it builds fine on Travis. However it doesn't seem to push coverage results even with your original code. So I must have misconfigured Coveralls somehow. Could you try something along the lines of this in your own repo and see if that works?
There was a problem hiding this comment.
I think #13 (diff) redefine all formatters. That's why for example doesn't work. If it's possible I would like to have both approach. git-ignore needs .simplecov because of use of test suite sharness, but in general people should use bashcov as another bash script. Anyway, I can keep this on my own fork till it's mature. I understand your are reluctant to add this
|
Cool. I have basic skills on ruby, but will apply all your comments. See the plugin style report is working and I took the license to suggest bashcov to be the tool to create reports on the cloud to codecov, coveralls and codeclimate This is a repo codecov creates to show people how to support bash coverage (they insist to use python codecov, let's see if we can change his mind) https://github.com/codecov/example-bash/issues/1 This is a repo pushing coverage reports to all of those systems codeclimate is not working yet. |
|
Sounds great @albfan - thanks for your efforts. Please let me know if you need help. I'll take another look into this too when I get a moment. |
|
Looks like everything is available to do this now:
Please reopen if not. |
Mimic behaviour of simplecov to allow bashcov to produce output on a directory different from current dir.
See it working here.
https://github.com/albfan/git-ignore/blob/master/.travis.yml
git-ignore is test with sharness, which creates a sandbox for every test group.
A way to configure bashcov is wanted too. See the hack on every test to read .simplecov
https://github.com/albfan/git-ignore/blob/master/t/t0001-exclude.sh#L11