Skip to content

Commit 48bb0f6

Browse files
committed
Update CONTRIBUTING.md to include Simplecov
1 parent 837c7d7 commit 48bb0f6

File tree

1 file changed

+27
-0
lines changed

1 file changed

+27
-0
lines changed

.github/CONTRIBUTING.md

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,33 @@ We are running [mdformat](https://github.com/executablebooks/mdformat) for forma
5151
$ mdformat . --number
5252
```
5353

54+
### Test Coverage - Line and Branch
55+
56+
We are using [Simplecov](https://github.com/colszowka/simplecov) to track test coverage.
57+
58+
It is included and reported when you run `bundle exec rake` or `bundle exec rspec`.
59+
60+
To view the coverage report, open the `coverage/index.html` file in your browser.
61+
62+
E.g. on macOS:
63+
```console
64+
$ open coverage/index.html
65+
```
66+
If you have unreachable lines, you can add `# :nocov` around those lines. The code itself or a comment should explain why the line is unreachable.
67+
68+
Example:
69+
```ruby
70+
# :nocov:
71+
raise ArgumentError("Unsupported style :#{style}")
72+
# :nocov:
73+
```
74+
75+
This can happen for a few reasons, including:
76+
1. When you handle config with a case statement and there is no else block
77+
2. When matching with a node pattern even when you handle all cases: all other node types will be excluded before reaching your handler, because the node pattern will not match them.
78+
79+
You will need full line and branch coverage to merge. This helps detect edge cases and prevent errors.
80+
5481
## Creating new cops
5582

5683
- Document examples of good and bad code in your cop.

0 commit comments

Comments
 (0)