tree bf0d10de5214503fbe465353fa3e724815a476a2
parent f9e8da1d0645dbfd376e37ea964cf551c2b74ad4
author Ben Clayton <bclayton@google.com> 1607976989 +0000
committer Commit Bot service account <commit-bot@chromium.org> 1607976989 +0000

Improve error messages raised by SpvParserTest

    EXPECT_THAT(xxx, Eq(yyy)) << ToString(xxx);

is a less-idiomatic way of writing:

    EXPECT_EQ(xxx, yyy);

The latter also provides a diff when two strings do not match.

Refactor these to:

    auto got = xxx;
    auto expect = yyy;
    EXPECT_EQ(got, expect);

So that the error message clearly shows which one is the generated, and which one is the reference.

Change-Id: I781437ee63abdff3a67798b09e958be603c21313
Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/35507
Commit-Queue: Ben Clayton <bclayton@google.com>
Reviewed-by: dan sinclair <dsinclair@chromium.org>