Project

General

Profile

Refactor #14191

Remove test coverage

Added by Tomer Brisker about 6 years ago. Updated almost 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Tests
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

Each test run uses the simplecov gem to generate coverage reports.
This slows down tests and is not used AFAICT by anyone.
This is especially significant for CI as the test suite is executed multiple times on slow workers.
Remove the gem and coverage reporting.


Related issues

Related to Foreman - Refactor #11208: Simplecov should be opt-inRejected2015-07-24

Associated revisions

Revision 1680e944 (diff)
Added by Tomer Brisker about 6 years ago

Fixes #14191 - remove simplecov test coverage reporting

History

#1 Updated by Dominic Cleal about 6 years ago

#2 Updated by Dominic Cleal about 6 years ago

Why remove it entirely? If removed then it's hard to enable if you do want to know about coverage.

Consider disabling it by default with an opt-in like #11208 describes, or disable it only during certain types of tests.

#3 Updated by The Foreman Bot about 6 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/3322 added

#4 Updated by Anonymous about 6 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#5 Updated by Dominic Cleal about 6 years ago

  • Legacy Backlogs Release (now unused) set to 136

Also available in: Atom PDF