diff options
author | Ewout van Bekkum <ewout@google.com> | 2022-05-15 11:03:30 -0700 |
---|---|---|
committer | CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-05-17 17:16:22 +0000 |
commit | c835e4c9a90d7c95a0050d3cdab80fffc824f040 (patch) | |
tree | 19da0b00e61b6119ec7464f57ac6fad6461c8836 /pw_tokenizer/base64_test.cc | |
parent | a7ac227c9d16c7d24bf15e974191174dd93ef1c7 (diff) | |
download | pigweed-c835e4c9a90d7c95a0050d3cdab80fffc824f040.tar.gz |
gtest: Fix up various tests to pass upstream googletest
Updates and fixes unit tests to pass upstream googletest due to:
1) ASSERT_EQ being used in constructors where return is not a
valid action.
2) TEST and TEST_F using the same group/fixture name which is
forbidden.
Also cleaned up pw_kvs to fix log levels for CMake and to not
use printf in tests.
Change-Id: Iee2a07189bc7d380368b648af561a784c0835cac
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/94720
Pigweed-Auto-Submit: Ewout van Bekkum <ewout@google.com>
Reviewed-by: Wyatt Hepler <hepler@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed.google.com.iam.gserviceaccount.com>
Diffstat (limited to 'pw_tokenizer/base64_test.cc')
-rw-r--r-- | pw_tokenizer/base64_test.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/pw_tokenizer/base64_test.cc b/pw_tokenizer/base64_test.cc index d751b8ee1..bdb1faee1 100644 --- a/pw_tokenizer/base64_test.cc +++ b/pw_tokenizer/base64_test.cc @@ -143,7 +143,7 @@ TEST_F(PrefixedBase64, Decode_OutputTooSmall_WritesNothing) { EXPECT_EQ(byte{kUnset}, binary_[0]); } -TEST(PrefixedBase64, DecodeInPlace) { +TEST(PrefixedBase64DecodeInPlace, DecodeInPlace) { byte buffer[32]; for (auto& [binary, base64] : kTestData) { |