Skip to content

Commit b410308

Browse files
authored
chore: remove SimpleCov code-coverage tooling (#118)
The coverage report was never a gate: `SimpleCov.start` deliberately set no `minimum_coverage`, nothing in CI read `coverage/`, and no build could fail because a percentage moved. It cost a development dependency, a require that had to run before any library code loaded, and a `COVERAGE=false` escape hatch to work around that ordering constraint. Drop the gem, the spec_helper block, the gitignored artifact path and the CONTRIBUTING notes that described it. YARD documentation coverage (`rake yard:stats`) is unaffected and stays. Signed-off-by: Tim Smith <tim@mondoo.com>
1 parent 070120b commit b410308

4 files changed

Lines changed: 1 addition & 23 deletions

File tree

.gitignore

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ Gemfile.lock
99
*.gem
1010
.bundle
1111
.config
12-
coverage
1312
InstalledFiles
1413
lib/bundler/man
1514
pkg

CONTRIBUTING.md

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -61,13 +61,8 @@ Useful environment variables:
6161
| `SEED=12345` | reproduce a specific random ordering |
6262
| `VERBOSE=1` | print full backtraces on failure |
6363
| `ONLY_FAILURES=1` | rerun only what failed last time |
64-
| `COVERAGE=false` | skip SimpleCov |
6564
| `RSPEC_WARNINGS=true` | enable Ruby warnings |
6665

67-
Coverage is reported to `coverage/` after each run as a diagnostic. **It is not
68-
a gate** — no build fails because a percentage moved, and neither the coverage
69-
report nor the documentation build runs in CI.
70-
7166
## Documentation
7267

7368
The public API is documented with [YARD](https://yardoc.org/):

Gemfile

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,7 @@ end
99

1010
group :test do
1111
gem "rake"
12-
gem "rspec", "~> 3.13"
13-
gem "simplecov", "~> 0.22"
12+
gem "rspec", "~> 3.13"
1413
end
1514

1615
group :cookstyle do

spec/spec_helper.rb

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -6,21 +6,6 @@
66
# Licensed under the Apache 2 License.
77
# See LICENSE for more details
88

9-
# Coverage must be started before any library code is loaded, otherwise methods
10-
# defined at require-time are recorded as uncovered. Set COVERAGE=false to skip
11-
# it (useful when bisecting or profiling a single spec file).
12-
unless ENV["COVERAGE"] == "false"
13-
require "simplecov"
14-
15-
SimpleCov.start do
16-
enable_coverage :branch
17-
add_filter "/spec/"
18-
track_files "lib/**/*.rb"
19-
# Deliberately no `minimum_coverage`: coverage is a diagnostic here, not a
20-
# gate. CI must never fail because a percentage moved.
21-
end
22-
end
23-
249
require "kitchen"
2510
require "kitchen/provisioner/dsc"
2611
require "kitchen-dsc/version"

0 commit comments

Comments
 (0)