From c83578038a6909fd4ca6032b5eb9a992e7c2fcd7 Mon Sep 17 00:00:00 2001 From: dan sinclair Date: Tue, 17 Mar 2020 16:13:31 +0000 Subject: [PATCH] Disable lint warnings in fail stream test. This CL marks the specific `bool` test as NOLINT in fail_stream to stop the linter from emitting errors. Change-Id: Ifa3d9b0a2e0f06bcd0d48cbe79f8dbe91fe7b4ef Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/17061 Reviewed-by: David Neto --- src/reader/spv/fail_stream_test.cc | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/reader/spv/fail_stream_test.cc b/src/reader/spv/fail_stream_test.cc index 394dc2c889..d496a80616 100644 --- a/src/reader/spv/fail_stream_test.cc +++ b/src/reader/spv/fail_stream_test.cc @@ -33,23 +33,23 @@ TEST_F(FailStreamTest, ConversionToBoolIsSameAsStatusMethod) { FailStream fs(&flag, nullptr); EXPECT_TRUE(fs.status()); - EXPECT_TRUE(bool(fs)); + EXPECT_TRUE(bool(fs)); // NOLINT flag = false; EXPECT_FALSE(fs.status()); - EXPECT_FALSE(bool(fs)); + EXPECT_FALSE(bool(fs)); // NOLINT flag = true; EXPECT_TRUE(fs.status()); - EXPECT_TRUE(bool(fs)); + EXPECT_TRUE(bool(fs)); // NOLINT } TEST_F(FailStreamTest, FailMethodChangesStatusToFalse) { bool flag = true; FailStream fs(&flag, nullptr); EXPECT_TRUE(flag); - EXPECT_TRUE(bool(fs)); + EXPECT_TRUE(bool(fs)); // NOLINT fs.Fail(); EXPECT_FALSE(flag); - EXPECT_FALSE(bool(fs)); + EXPECT_FALSE(bool(fs)); // NOLINT } TEST_F(FailStreamTest, FailMethodReturnsSelf) {