bazelbuild/bazel

Default value for `instrumentation_filter` will change when using on instrumentation test targets

Open

#14,652 建立於 2022年1月27日

在 GitHub 查看
 (2 留言) (1 反應) (0 負責人)Java (4,465 fork)batch import
P3coveragehelp wantedteam-Documentationteam-Rules-Servertype: documentation (cleanup)

倉庫指標

Star
 (25,384 star)
PR 合併指標
 (平均合併 22天 20小時) (30 天內合併 77 個 PR)

描述

Description of the problem / feature request:

The behavior of instrumentation_filter doesn't match the documentation's description: default: "-/javatests[/:],-/test/java[/:]"

For example, here is the result of offline instrumentation test result of different commands with options:

bazel coverage app → offline instrumentation works
bazel coverage app //unit-test-target → only unit test lib got offline instrumentation
bazel coverage app-instrumentation-test → offline instrumentation doesn't work
bazel build app-instrumentation-test --collect_code_coverage  → offline instrumentation works
bazel coverage app-instrumentation-test --collect_code_coverage  → offline instrumentation doesn't work

As you can see, the behavior is very inconsistent and the root cause is that the default value of instrumentation_filter will be changed to match the target itself when we run test command on an instrumentation test target, meaning none of the dependency will got instrumented.

Example, say //app depends on //battery, when we run bazel coverage app-instrumentation-test, the default instrumentation_filter for //battery will be "-/battery[/:]" and the same applies to all other libraries

Expected behavior

Either update the documentation, or keep default value of instrumentation_filter unchanged everywhere.

貢獻者指南