New issue
Advanced search Search tips

Issue 763957 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 1
Type: Bug

Blocking:
issue 757515



Sign in to add a comment

Exclude test files from code coverage report

Project Member Reported by liaoyuke@chromium.org, Sep 11 2017

Issue description

Right now, the generated code coverage report includes unit test files, but it shouldn't.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Sep 15 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4292b2b9c7ce439bc34450f50ee0dd20f4a98433

commit 4292b2b9c7ce439bc34450f50ee0dd20f4a98433
Author: Yuke Liao <liaoyuke@chromium.org>
Date: Fri Sep 15 23:52:02 2017

[Coverage] Exclude test files from code coveage report.

Previously, the generated code covearge report includes test files
(unit test and egtest), but it shouldn't.

So this CL excludes test files from code coverage files, and the test
files are identified by "unittest.cc", "unittest.mm" and "egtest.mm".

Bug:  763957 
Change-Id: I6a0eb938432135736ba8f15ced0452cee67ee4fd
Reviewed-on: https://chromium-review.googlesource.com/664241
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Reviewed-by: Mike Baxley <baxley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#502446}
[modify] https://crrev.com/4292b2b9c7ce439bc34450f50ee0dd20f4a98433/ios/tools/coverage/coverage.py

Status: Fixed (was: Started)

Sign in to add a comment