diff options
author | Roman Lebedev <lebedev.ri@gmail.com> | 2018-09-13 15:08:15 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-09-13 15:08:15 +0300 |
commit | 58588476ce52c0af4cb54f9c596c4579f78bc953 (patch) | |
tree | bd31b23a9d809961302a02e9871857f96ec3d19b /test/skip_with_error_test.cc | |
parent | c614dfc0d4eadcd19b188ff9c7e226c138f894a1 (diff) | |
download | google-benchmark-58588476ce52c0af4cb54f9c596c4579f78bc953.tar.gz |
Track two more details about runs - the aggregate name, and run name. (#675)
This is related to @BaaMeow's work in https://github.com/google/benchmark/pull/616 but is not based on it.
Two new fields are tracked, and dumped into JSON:
* If the run is an aggregate, the aggregate's name is stored.
It can be RMS, BigO, mean, median, stddev, or any custom stat name.
* The aggregate-name-less run name is additionally stored.
I.e. not some name of the benchmark function, but the actual
name, but without the 'aggregate name' suffix.
This way one can group/filter all the runs,
and filter by the particular aggregate type.
I *might* need this for further tooling improvement.
Or maybe not.
But this is certainly worthwhile for custom tooling.
Diffstat (limited to 'test/skip_with_error_test.cc')
-rw-r--r-- | test/skip_with_error_test.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/skip_with_error_test.cc b/test/skip_with_error_test.cc index 39785fb..0657977 100644 --- a/test/skip_with_error_test.cc +++ b/test/skip_with_error_test.cc @@ -33,8 +33,8 @@ struct TestCase { typedef benchmark::BenchmarkReporter::Run Run; void CheckRun(Run const& run) const { - CHECK(name == run.benchmark_name) - << "expected " << name << " got " << run.benchmark_name; + CHECK(name == run.benchmark_name()) + << "expected " << name << " got " << run.benchmark_name(); CHECK(error_occurred == run.error_occurred); CHECK(error_message == run.error_message); if (error_occurred) { |