tools: Fix benchmark tests

The diff format changed.

Change-Id: I13930e0aeb450384d95d71460f9623adb5c1b192
Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/78581
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Kokoro: Ben Clayton <bclayton@chromium.org>
Commit-Queue: Ben Clayton <bclayton@chromium.org>
This commit is contained in:
Ben Clayton 2022-01-28 19:21:20 +00:00 committed by Tint LUCI CQ
parent 3fe1bd3715
commit f34038b1a0
1 changed files with 12 additions and 4 deletions

View File

@ -197,16 +197,20 @@ func TestCompare(t *testing.T) {
expectEqual(t, "bench.Compare().Format", cmp.Format(bench.DiffFormat{}), "") expectEqual(t, "bench.Compare().Format", cmp.Format(bench.DiffFormat{}), "")
expectEqual(t, "bench.Compare().Format", "\n"+cmp.Format(bench.DiffFormat{TimeA: true}), ` expectEqual(t, "bench.Compare().Format", "\n"+cmp.Format(bench.DiffFormat{TimeA: true}), `
+-----------+
| A | | A |
|-----------| +-----------+
| 1.67681ms | | 1.67681ms |
| 53.15µs | | 53.15µs |
+-----------+
`) `)
expectEqual(t, "bench.Compare().Format", "\n"+cmp.Format(bench.DiffFormat{TimeA: true, TimeB: true}), ` expectEqual(t, "bench.Compare().Format", "\n"+cmp.Format(bench.DiffFormat{TimeA: true, TimeB: true}), `
+-----------+-------------+
| A | B | | A | B |
|-----------+-------------| +-----------+-------------+
| 1.67681ms | 87.657868ms | | 1.67681ms | 87.657868ms |
| 53.15µs | 2.352336ms | | 53.15µs | 2.352336ms |
+-----------+-------------+
`) `)
expectEqual(t, "bench.Compare().Format", "\n"+cmp.Format(bench.DiffFormat{ expectEqual(t, "bench.Compare().Format", "\n"+cmp.Format(bench.DiffFormat{
TestName: true, TestName: true,
@ -215,10 +219,12 @@ func TestCompare(t *testing.T) {
TimeA: true, TimeA: true,
TimeB: true, TimeB: true,
}), ` }), `
+--------------+-------------+-----------+-----------+-------------+
| Test name | Δ (A B) | % (A B) | A | B | | Test name | Δ (A B) | % (A B) | A | B |
|--------------+-------------+-----------+-----------+-------------| +--------------+-------------+-----------+-----------+-------------+
| MyBenchmark3 | 85.981058ms | +5127.7% | 1.67681ms | 87.657868ms | | MyBenchmark3 | 85.981058ms | +5127.7% | 1.67681ms | 87.657868ms |
| MyBenchmark4 | 2.299186ms | +4325.8% | 53.15µs | 2.352336ms | | MyBenchmark4 | 2.299186ms | +4325.8% | 53.15µs | 2.352336ms |
+--------------+-------------+-----------+-----------+-------------+
`) `)
expectEqual(t, "bench.Compare().Format", "\n"+cmp.Format(bench.DiffFormat{ expectEqual(t, "bench.Compare().Format", "\n"+cmp.Format(bench.DiffFormat{
TestName: true, TestName: true,
@ -230,10 +236,12 @@ func TestCompare(t *testing.T) {
TimeA: true, TimeA: true,
TimeB: true, TimeB: true,
}), ` }), `
+--------------+-------------+-----------+-----------+-----------+-----------+-----------+-------------+
| Test name | Δ (A B) | % (A B) | % (B A) | × (A B) | × (B A) | A | B | | Test name | Δ (A B) | % (A B) | % (B A) | × (A B) | × (B A) | A | B |
|--------------+-------------+-----------+-----------+-----------+-----------+-----------+-------------| +--------------+-------------+-----------+-----------+-----------+-----------+-----------+-------------+
| MyBenchmark3 | 85.981058ms | +5127.7% | -98.1% | +52.2766 | +0.0191 | 1.67681ms | 87.657868ms | | MyBenchmark3 | 85.981058ms | +5127.7% | -98.1% | +52.2766 | +0.0191 | 1.67681ms | 87.657868ms |
| MyBenchmark4 | 2.299186ms | +4325.8% | -97.7% | +44.2584 | +0.0226 | 53.15µs | 2.352336ms | | MyBenchmark4 | 2.299186ms | +4325.8% | -97.7% | +44.2584 | +0.0226 | 53.15µs | 2.352336ms |
+--------------+-------------+-----------+-----------+-----------+-----------+-----------+-------------+
`) `)
} }