coffee/googletest
1
0
Fork 0
mirror of https://github.com/google/googletest.git synced 2025-12-21 11:04:54 +08:00
Code Issues Packages Projects Releases Wiki Activity

Applied C++ Core Guidelines Enum.2 Rule. Problem: switch statements should cover all cases. Solution: Added missing enumerations in switch statements and added [[fallthrough]] attribute to denote that the default case is the desired case for these missing enumerations.

Browse Source
This commit is contained in:
Anthony Graca 2021-11-02 11:52:29 -07:00
parent 16f637fbf4
commit 5864e885eb
No known key found for this signature in database
GPG Key ID: 22EF623BEE2D4AB4
3 changed files with 48 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
googlemock/src/gmock-spec-builders.cc
View File

@ -300,6 +300,8 @@ void ReportUninterestingCall(CallReaction reaction, const std::string& msg) {
"knowing-when-to-expect for details.\n",
stack_frames_to_skip);
break;
case kFail:
[[fallthrough]];
default: // FAIL
Expect(false, nullptr, -1, msg);
}

14
googletest/src/gtest.cc
View File

@ -3254,6 +3254,8 @@ static const char* GetAnsiColorCode(GTestColor color) {
return "2";
case GTestColor::kYellow:
return "3";
case GTestColor::kDefault:
[[fallthrough]];
default:
return nullptr;
}
@ -3502,6 +3504,12 @@ void PrettyUnitTestResultPrinter::OnTestPartResult(
// If the test part succeeded, we don't need to do anything.
case TestPartResult::kSuccess:
return;
case TestPartResult::kNonFatalFailure:
[[fallthrough]];
case TestPartResult::kFatalFailure:
[[fallthrough]];
case TestPartResult::kSkip:
[[fallthrough]];
default:
// Print failure message from the assertion
// (e.g. expected this and got that).
@ -3719,6 +3727,12 @@ void BriefUnitTestResultPrinter::OnTestPartResult(
// If the test part succeeded, we don't need to do anything.
case TestPartResult::kSuccess:
return;
case TestPartResult::kNonFatalFailure:
[[fallthrough]];
case TestPartResult::kFatalFailure:
[[fallthrough]];
case TestPartResult::kSkip:
[[fallthrough]];
default:
// Print failure message from the assertion
// (e.g. expected this and got that).

32
googletest/test/gtest_unittest.cc
View File

@ -146,6 +146,38 @@ TEST_F(StreamingListenerTest, OnTestPartResult) {
*output());
}
TEST_F(StreamingListenerTest, OnTestPartResult) {
*output() = "";
streamer_.OnTestPartResult(TestPartResult(
TestPartResult::kNonFatalFailure, "foo.cc", 42, "failed=\n&%"));
// Meta characters in the failure message should be properly escaped.
EXPECT_EQ(
"event=TestPartResult&file=foo.cc&line=42&message=failed%3D%0A%26%25\n",
*output());
}
TEST_F(StreamingListenerTest, OnTestPartResult) {
*output() = "";
streamer_.OnTestPartResult(TestPartResult(
TestPartResult::kFatalFailure, "foo.cc", 42, "failed=\n&%"));
// Meta characters in the failure message should be properly escaped.
EXPECT_EQ(
"event=TestPartResult&file=foo.cc&line=42&message=failed%3D%0A%26%25\n",
*output());
}
TEST_F(StreamingListenerTest, OnTestPartResult) {
*output() = "";
streamer_.OnTestPartResult(TestPartResult(
TestPartResult::kSkip, "foo.cc", 42, "failed=\n&%"));
// Meta characters in the failure message should be properly escaped.
EXPECT_EQ(
"event=TestPartResult&file=foo.cc&line=42&message=failed%3D%0A%26%25\n",
*output());
}
#endif // GTEST_CAN_STREAM_RESULTS_
// Provides access to otherwise private parts of the TestEventListeners class