| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
| |
Add --gtest_fail_fast support to googletest.
- Analogous functionality to to golang -test.failfast and python --failfast
- Stops test execution upon first test failure.
- Also add support Bazel equivalent env var (TESTBRIDGE_TEST_RUNNER_FAIL_FAST)
PiperOrigin-RevId: 302488880
|
|
|
|
|
|
| |
Internal change
PiperOrigin-RevId: 302048013
|
|\
| |
| |
| | |
PiperOrigin-RevId: 302045808
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Exposition of the problem:
> cat main.cpp
TEST(errnoTest, errnoTest)
{
ASSERT_EQ(errno, 0);
}
int main(int argc, char** argv)
{
::testing::InitGoogleTest(&argc, argv);
return RUN_ALL_TESTS();
}
Compiled with gcc 10 like this:
> g++ -pthread -o runtest main.cpp -Wl,-Bstatic -lgtest -Wl,-Bdynamic
Before patch:
> ./runtest
[==========] Running 1 test from 1 test suite.
[----------] Global test environment set-up.
[----------] 1 test from errnoTest
[ RUN ] errnoTest.errnoTest
[ OK ] errnoTest.errnoTest (0 ms)
[----------] 1 test from errnoTest (0 ms total)
[----------] Global test environment tear-down
[==========] 1 test from 1 test suite ran. (0 ms total)
[ PASSED ] 1 test.
(output is colored, I run this inside an interactive terminal).
> ./runtest | cat
[==========] Running 1 test from 1 test suite.
[----------] Global test environment set-up.
[----------] 1 test from errnoTest
[ RUN ] errnoTest.errnoTest
main.cpp:5: Failure
Expected equality of these values:
(*__errno_location ())
Which is: 25
0
[ FAILED ] errnoTest.errnoTest (0 ms)
[----------] 1 test from errnoTest (0 ms total)
[----------] Global test environment tear-down
[==========] 1 test from 1 test suite ran. (0 ms total)
[ PASSED ] 0 tests.
[ FAILED ] 1 test, listed below:
[ FAILED ] errnoTest.errnoTest
1 FAILED TEST
(output is not colored, since IsTTY return false, because of the pipe,
however it also clobbered errno for the tests).
After the patch, both cases are working fine:
> ./runtest
[==========] Running 1 test from 1 test suite.
[----------] Global test environment set-up.
[----------] 1 test from errnoTest
[ RUN ] errnoTest.errnoTest
[ OK ] errnoTest.errnoTest (0 ms)
[----------] 1 test from errnoTest (0 ms total)
[----------] Global test environment tear-down
[==========] 1 test from 1 test suite ran. (0 ms total)
[ PASSED ] 1 test.
> ./runtest | cat
[==========] Running 1 test from 1 test suite.
[----------] Global test environment set-up.
[----------] 1 test from errnoTest
[ RUN ] errnoTest.errnoTest
[ OK ] errnoTest.errnoTest (0 ms)
[----------] 1 test from errnoTest (0 ms total)
[----------] Global test environment tear-down
[==========] 1 test from 1 test suite ran. (0 ms total)
[ PASSED ] 1 test.
|
| |
| |
| |
| |
| |
| | |
Move internal function out of header because Hyrum's law always wins.
PiperOrigin-RevId: 301446904
|
| |
| |
| |
| |
| |
| |
| | |
Use a polymorphic matcher instead of the GreaterThan<int> test matcher
to fix the sign-comparison warning on MSVC.
PiperOrigin-RevId: 301163657
|
| |
| |
| |
| |
| |
| |
| |
| | |
Fix Compilation against CMake < 3.7 (Fix #2711)
`VERSION_GREATER_EQUAL` was introduced in CMake 3.7
ref: https://cmake.org/cmake/help/v3.7/release/3.7.html#commands
PiperOrigin-RevId: 300817917
|
| |
| |
| |
| |
| |
| | |
Internal change
PiperOrigin-RevId: 299345492
|
| |
| |
| |
| |
| |
| | |
Simplify the fallback printing logic to have a single sequence of trial printers.
PiperOrigin-RevId: 298621376
|
|\ \
| | |
| | |
| | | |
PiperOrigin-RevId: 298608772
|
| | | |
|
|\ \ \
| | | |
| | | |
| | | | |
PiperOrigin-RevId: 298599235
|
| | | | |
|
| |_|/
|/| |
| | |
| | |
| | |
| | |
| | |
| | | |
Remove public buganizer reference from googletest cookbook.
It also seems that this bug is obsolete.
PiperOrigin-RevId: 298598298
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Rename internal color enumerators to avoid conflicts with curses.h macro definitions.
Fixes #2685
PiperOrigin-RevId: 297639382
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Relax the implementation of MatcherCast to allow conversion of `Matcher<T>` to
`Matcher<const T&>`. They have the same match signature.
PiperOrigin-RevId: 297115843
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Allow construction of an Action from a callable of zero args
Action already allows construction from a callable with the same args as the mocked function, without needing to wrap the callable in Invoke. However, if you don't care about the arguments to the mocked function you need to either accept all of them or wrap your callable in InvokeWithoutArgs. This change makes both of those unnecessary, since it allows you to pass a no-args callable to Action directly.
PiperOrigin-RevId: 296117034
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Remove the CMAKE_CXX_STANDARD from GoogleTest's CMakeLists.txt
This causes ABI issues since it can create a mixed-mode build. The
value should be inherited from the top-level build if it needs to be
set.
PiperOrigin-RevId: 294730724
|
| | |
| | |
| | |
| | |
| | |
| | | |
Add gmock Matcher<std::string_view> specialization.
PiperOrigin-RevId: 294443240
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Fix gmock_gen to use MOCK_METHOD instead of old style macros. Fix several
related bugs in argument parsing and return types.
- handle commas more correctly in return types
- handle commas correctly in arguments
- handle default values more correctly
PiperOrigin-RevId: 294435093
|
| | |
| | |
| | |
| | |
| | |
| | | |
Fix gmock_gen to use MOCK_METHOD instead of old style macros.
PiperOrigin-RevId: 294360947
|
| | |
| | |
| | |
| | |
| | |
| | | |
Fix gmock_gen to use MOCK_METHOD instead of old style macros.
PiperOrigin-RevId: 294332975
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Refactor function GetNextPrime so that the loop precondition is checked before
loop instead of during every loop run. Also by removing the loop condition,
it shows that the only exit from the loop is the return statement.
PiperOrigin-RevId: 293932783
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Get rid of gmock-generated-matchers.h and gmock-generated-matchers.h.pump.
Stop using pump for MATCHER* macroses generation.
PiperOrigin-RevId: 293878808
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Tag the function generated by MATCHER with GTEST_ATTRIBUTE_UNUSED_ to fix CI builds of gmock-matchers_test.cc vs. -Wunused-function.
See https://github.com/google/googletest/pull/2697 for breakage.
PiperOrigin-RevId: 293669752
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Get rid of gmock-generated-function-mockers.h and
gmock-generated-function-mockers.h.pump.
Stop using pump for GMOCK_METHOD* macroses generation.
PiperOrigin-RevId: 293454519
|
| | |
| | |
| | |
| | |
| | |
| | | |
...text exposed to GitHub repo https://www.github.com/google/googletest
PiperOrigin-RevId: 293438092
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Pass method's parameters count to internal GMOCK_METHOD* macro.
This will help removing copypaste in every GMOCK_METHOD* macro in future.
PiperOrigin-RevId: 292932554
|
| | |
| | |
| | |
| | |
| | |
| | | |
Fix std::move to std::forward where appropriate to support reference types.
PiperOrigin-RevId: 292923058
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Disable warning C4800 for Visual Studio 2019.
The compiler warning C4800 is disabled by default in Visual Studio 2019,
but it can be enabled on the command line. The only version of
Visual Studio that does not support warning C4800 is Visual Studio 2017.
PiperOrigin-RevId: 292624510
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Fix use of reserved names.
Minimize code duplication needed for explict-vs-nonexplicit constructor.
PiperOrigin-RevId: 292555014
|
| | |
| | |
| | |
| | |
| | |
| | | |
Add documentation for ASSERT_DEBUG_DEATH/EXPECT_DEBUG_DEATH
PiperOrigin-RevId: 292138974
|
| | |
| | |
| | |
| | |
| | |
| | | |
Add includes for type_traits and utility to gmock-function-mocker.h: macros in the file require these headers.
PiperOrigin-RevId: 291782497
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Create implementation macroses for matchers to move variadic parameters to the
end of parameters list.
To save backward compatibility, old macroses will be still taking `description`
parameter as the last one. But they will use INTERNAL macro that takes
`description` as the second parameter.
PiperOrigin-RevId: 291724469
|
| | |
| | |
| | |
| | |
| | |
| | | |
Adds missing `#define` guard around `TEST_F(...)`
PiperOrigin-RevId: 291703056
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Move part of functionality of Matcher* class to the base one. Reduce copypaste.
Make constructor and conversion operator of Matcher* class independent of pump.
PiperOrigin-RevId: 291405510
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Deleted an orphaned duplicate file and exclude another that shouldn't be part of :gtest_all_test.
This showed up while trying to debug the presubmit failure for: https://github.com/google/googletest/pull/2683
PiperOrigin-RevId: 291398123
|
|/ /
| |
| |
| |
| |
| |
| |
| |
| | |
Add missing explicit keyword for gmock_Impl constructor.
When switching to using GMOCK_PP in ACTION* macroses `explicit` keyword was
missed in gmock_Impl constructor causing ClangTidy warnings in ACTION_P macro.
PiperOrigin-RevId: 291159975
|
| |
| |
| |
| |
| |
| | |
Fix SCOPED_TRACE() message header in example
PiperOrigin-RevId: 290800241
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Allow copying of the string in MatchAndExplain.
Otherwise, conversions from std::string_view to std::string will fail as being
explicit
PiperOrigin-RevId: 290301103
|
|\ \
| | |
| | |
| | |
| | |
| | | |
bysreg:fix_noshortcircuitfailure_detectsflakyshortcircuit_test
PiperOrigin-RevId: 290256504
|
| | |
| | |
| | |
| | | |
gmock-matchers_test.cc
|
| | |
| | |
| | |
| | | |
DetectsFlakyShortCircuit when GTEST_HAS_RTTI is 1
|
|\ \ \
| | | |
| | | |
| | | | |
PiperOrigin-RevId: 290255937
|
| |/ /
| | |
| | | |
Removing exclusion of *-main and*-all targets from the library.json used on platformio.
|
| | |
| | |
| | |
| | |
| | |
| | | |
Add extra filtering so that the reported message differentiates between the case where INSTANTIATE_TEST_SUITE_P is missing vs. the case where TEST_P is missing.
PiperOrigin-RevId: 290114508
|
| | |
| | |
| | |
| | |
| | |
| | | |
Correct the spelling of PARAMETERIZED.
PiperOrigin-RevId: 289897278
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Use GMOCK_PP to create GMOCK_INTERNAL_ACTION macro.
Create GMOCK_INTERNAL_ACTION macro that generates ACTION_P* macroses using
GMOCK_PP.
PiperOrigin-RevId: 289815906
|
| | |
| | |
| | |
| | |
| | |
| | | |
Wire up things to support marking a type paramaterized test as allowed to be un-instantiated.
PiperOrigin-RevId: 289699939
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Use GMOCK_PP to generate args boilerplate.
Move common args describing part to separate macroses that uses GMOCK_PP to
generate sequences.
PiperOrigin-RevId: 289655624
|