639
62%

Ran 15 Apr 2016 03:17AM UTC

Jobs 1

Files 450

Run time 14s

Badge

Embed ▾

pending completion

Build # 639

Build Type

push

travis-ci

Committed by Commit bot

Commit Message

tsmon: Stop linking "goconvey" and "testing" into real production executables.

Attempting to use tsmon library in an executable that uses "flag.Parse()"
reveals that "goconvey" and "testing" register all their flags there, suggesting
that some testing init() code executes somewhere and register all these flags:

$ ./luci_machine_tokend
-pkey-pem is required
Usage of ./luci_machine_tokend:
  ...
  -convey-json
      When true, emits results in JSON blocks. Default: 'false'
  -convey-silent
      When true, all output from GoConvey is suppressed.
  ....
  -test.bench string
      regular expression to select benchmarks to run
  -test.benchmem
      print memory allocations for benchmarks
  ...

Move store/testing.go into its own package, and do a bit of duck typing to avoid
module import cycle (storetest package tests interface that quack similarly
enough to store.Store).

Remove ModifyMulti since it wasn't used and it references real Modification
struct that's non-importable from storetest. If it is ever needed, Modification
should be moved to its own small package, importable from both tsmon/store and
tsmon/store/storetest.

R=dsansome@chromium.org, nodir@chromium.org
BUG=

Review URL: https://codereview.chromium.org/1889693004

Coverage Stats

24910 of 36083 relevant lines covered (69.04%)

2123.0 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	639.1	15 Apr 2016 03:17AM UTC	0	69.04	Travis Job 639.1

Source Files on build 639

Detailed source file information is not available for this build.

luci / luci-go / 639
62%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 639

luci / luci-go / 639 62%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 639

luci / luci-go / 639
62%

README BADGES
x