Project

General

Profile

Actions

Refactor #14191

closed

Remove test coverage

Added by Tomer Brisker about 8 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Tests
Target version:
Difficulty:
Triaged:
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 1 (0 open1 closed)

Related to Foreman - Refactor #11208: Simplecov should be opt-inRejected07/24/2015Actions
Actions #1

Updated by Dominic Cleal about 8 years ago

Actions #2

Updated by Dominic Cleal about 8 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.

Actions #3

Updated by The Foreman Bot about 8 years ago

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

Updated by Anonymous about 8 years ago

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

Updated by Dominic Cleal about 8 years ago

  • translation missing: en.field_release set to 136
Actions

Also available in: Atom PDF