Skip to content

Implement code coverage using SimpleCov - #194

Merged
ekohl merged 1 commit into
fog:masterfrom
davispuh:cov
Jul 27, 2026
Merged

Implement code coverage using SimpleCov#194
ekohl merged 1 commit into
fog:masterfrom
davispuh:cov

Conversation

@davispuh

Copy link
Copy Markdown
Contributor

This PR implements code coverage using SimpleCov

Copilot AI review requested due to automatic review settings July 21, 2026 19:25

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR introduces SimpleCov-based code coverage instrumentation for both the Shindo test suite (tests/) and the Minitest suite (minitests/), with branch coverage configuration via a .simplecov profile.

Changes:

  • Start SimpleCov in both tests/helper.rb and minitests/test_helper.rb, gated by ENV['COVERAGE'].
  • Default ENV['COVERAGE'] to "true" in the Rakefile so rake runs collect coverage unless explicitly disabled.
  • Add SimpleCov configuration (.simplecov) and declare SimpleCov as a development dependency in the gemspec.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tests/helper.rb Starts SimpleCov before loading the library in the Shindo test helper.
Rakefile Sets default COVERAGE env var to enable coverage during rake-driven test runs.
minitests/test_helper.rb Starts SimpleCov before loading the library in the Minitest helper.
fog-libvirt.gemspec Adds SimpleCov as a development dependency with a minimum version.
.simplecov Defines coverage configuration (filters + branch coverage settings).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread minitests/test_helper.rb Outdated
Comment thread fog-libvirt.gemspec Outdated

@ekohl ekohl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@ekohl
ekohl merged commit ffe3e3b into fog:master Jul 27, 2026
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants