summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* Merge pull request #3791 from MR-A0101:patch-1Copybara-Service2022-04-181-2/+2
|\ | | | | | | | | PiperOrigin-RevId: 442609985 Change-Id: I4239171ef8a671ef6d548ba94e02104c5f0ecd36
| * Removing commas.refs/pull/3791/headMR-A2022-04-061-2/+2
| | | | | | Readme Update!
* | Document two surprises in `DoAllAction::NonFinalType`.Aaron Jacobs2022-04-132-0/+128
| | | | | | | | | | PiperOrigin-RevId: 441589196 Change-Id: Ic3e483ca70d72261046bad464d817f9dfd4bec65
* | Replace infinite recursion call (intentionally invoking undefined behavior ↵Abseil Team2022-04-131-3/+6
| | | | | | | | | | | | | | to indicate unreachability) with explicit unreachability marker. PiperOrigin-RevId: 441474979 Change-Id: I1fcbb437026631212fec954c663482bb7e1cf819
* | Add support for move-only and &&-qualified actions in WillOnce.Abseil Team2022-04-094-35/+584
|/ | | | | | | | | | | | This provides a type-safe way for an action to express that it wants to be called only once, or to capture move-only objects. It is a generalization of the type system-evading hack in ByMove, with the improvement that it works for _any_ action (including user-defined ones), and correctly expresses that the action can only be used with WillOnce. I'll make existing actions benefit in a future commit. PiperOrigin-RevId: 440496139 Change-Id: I4145d191cca5655995ef41360bb126c123cb41d3
* Clarify what live-at-head actually meansDerek Mauro2022-04-061-1/+2
| | | | | PiperOrigin-RevId: 439835271 Change-Id: I40765e607a96dfc3f0679450938daf25163c48a4
* Trying to make it more clear: this is probably not about mock classes, but ↵Abseil Team2022-04-041-6/+6
| | | | | | | about mocked classes PiperOrigin-RevId: 439427291 Change-Id: I3cac035e732fb3fe4f9c314657932a55269e0416
* gmock: improve SFINAE for actions involving arguments.Abseil Team2022-04-042-2/+4
| | | | | | | | Avoid instantiating functions like std::get<index> for an out of range index when doing SFINAE on the invocability of the action itself. PiperOrigin-RevId: 439415110 Change-Id: Ifc20285a6d526c34830870cd1910c2b2b92e1e81
* Use ResultOf()'s result_description when explaining match result.Abseil Team2022-04-042-1/+16
| | | | | PiperOrigin-RevId: 439389646 Change-Id: Ie34adcdd2b24378e85962efac53b7bb89ed93803
* Use the Abseil flags library when Abseil is presentDerek Mauro2022-04-049-113/+242
| | | | | | | | | | | | | | | | | | | | | | | | | | When built with `--define=absl=1` under Bazel, GoogleTest flags use ABSL_FLAG instead of GoogleTest's own implementation. There are some minor behavior differences in this mode. The most notable difference is that unrecognized flags result in a flag parsing error, and are not returned to the user though a modified argc/argv, unless they appear after the positional argument delimiter ("--"). For example, to pass a non-Abseil flag, you would have to do ./mytest --gtest_color=false -- --myflag=myvalue The documentation at https://abseil.io/docs/cpp/guides/flags may be helpful in understanding the behavior. There are some other minor differences. For example, passing --help results in the program returning 1 instead of 0. https://github.com/google/googletest/issues/3646 PiperOrigin-RevId: 439312700 Change-Id: Id696a25f50f24a5b1785c45ca8fa59794f86fd5c
* Address deprecation warning surfaced by Github presubmit testsAbseil Team2022-03-231-4/+4
| | | | | PiperOrigin-RevId: 436839469 Change-Id: I2bb90dce524c16bcc31569bb98772dcc62d08fbf
* Split gmock-matchers_test into 4 smaller test #3653Mattias Ellert2022-03-239-8645/+8857
| | | | | | | | | The one large test does not link in limited memory environments Closes #3653 PiperOrigin-RevId: 436753193 Change-Id: Idd59b6509994fc642147b88279ee791cd1d7bdd0
* Merge pull request #3779 from messerb5467:promote-inclusive-behaviorCopybara-Service2022-03-224-4/+4
|\ | | | | | | | | PiperOrigin-RevId: 436579789 Change-Id: I19faae341027efba83f1c2b3839636fc95780fc7
| * Merge branch 'main' into promote-inclusive-behaviorrefs/pull/3779/headBrad Messer2022-03-18135-7135/+5943
| |\
| * | Remove sanity as it is offensive to neurodiverse individuals.Brad Messer2022-03-184-4/+4
| | |
* | | Only print disabled test banner if the test matches gtest_filterAbseil Team2022-03-222-1/+32
| | | | | | | | | | | | | | | | | | | | | | | | Currently, the "[ DISABLED ]" banner is printed for every test in a suite. When iterating on a single test gtest_filter this is very noisy. PiperOrigin-RevId: 436489088 Change-Id: If337087a7a0986b073fabf2b0a55d26485eb5c37
* | | Clarify public access on gmock examples.Abseil Team2022-03-211-4/+4
| |/ |/| | | | | | | PiperOrigin-RevId: 436268062 Change-Id: I3161b36121f78acf920d66fe121b2f1d5e5707b9
* | Running clang-format over all of GoogleTestAbseil Team2022-03-15126-7092/+5808
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A few tests are examining code locations and looking af the resulting line numbers to verify that GoogleTest shows those to users correctly. Some of those locations change when clang-format is run. For those locations, I've wrapped portions in: // clang-format off ... // clang-format on There may be other locations that are currently not tickled by running clang-format. PiperOrigin-RevId: 434844712 Change-Id: I3a9f0a6f39eff741c576b6de389bef9b1d11139d
* | Merge pull request #3774 from sobczyk:mainCopybara-Service2022-03-1510-43/+135
|\ \ | |/ |/| | | | | PiperOrigin-RevId: 434738675 Change-Id: I7c8de4004bac6b750674d19e3e79c0695a42652e
| * Add myself to contributorsrefs/pull/3774/headSzymon Sobik2022-03-081-0/+1
| |
| * Adjust documentation regarding xml and json source file location otput.Szymon Sobik2022-03-082-6/+18
| |
| * Adjust xml and json unit tests to test for source file and line location.Szymon Sobik2022-03-084-32/+100
| |
| * Add support for testing of xml and json output of source file and line locationSzymon Sobik2022-03-082-0/+6
| |
| * Output source file path and line number in xml and json files.Szymon Sobik2022-03-081-5/+9
| | | | | | | | | | It outputs the location of testcase only. Standalone EXPECT will have no location.
* | Remove references to deleted script gen_gtest_pred_impl.pyDino Radakovic2022-03-142-6/+0
| | | | | | | | | | PiperOrigin-RevId: 434464892 Change-Id: Iaa2277f26d0049b3d792e87f64679774a28205c6
* | Mark ACTION_Pn()-generated functions as must-use-result, adding ↵Abseil Team2022-03-081-0/+3
|/ | | | | | | | | | | | | non-compilation tests. This catches when a client creates an action and discards it, thinking that the action has actually been applied to something. This will help people who make the mistake of defining, for example, both `void Use(Foo*, Bar)` and `ACTION_P(Use, bar) { Use(arg, bar); }` for later application to a Foo. With such an overload, a client may then write `Use(bar);`, selecting the Action<> overload and being confused why nothing happens. This also catches when a client defines their own action in terms of an ACTION_P()-generated one, invokes the Action<>'s builder, and then fails to invoke the resulting action, thinking it's operating on the outer action's parameters. PiperOrigin-RevId: 433197479 Change-Id: I98e4389150d01a5e753230113016d9fc38b1d260
* Update GCC/Clang Linux tests to use Bazel 5.0.0Derek Mauro2022-02-171-1/+1
| | | | | PiperOrigin-RevId: 429400664 Change-Id: I6454cf52cb2fc616a96663823de997591b63dfae
* Address conversion warning by explicitly casting to size_tAbseil Team2022-02-141-1/+3
| | | | | | | Closes #3762 PiperOrigin-RevId: 428593750 Change-Id: Ifac216568fbc7d999adb71996ec6a1bbe3b97412
* Add a 3-arg overload for ResultOf() matcher that takes a description string ↵Abseil Team2022-02-103-6/+48
| | | | | | | for better error messages. PiperOrigin-RevId: 427598749 Change-Id: I8c7a5d7b2dde017641534f1c7eed8dd56c33e845
* Merge pull request #3751 from noiseless:gtest-help-test-OpenBSDCopybara-Service2022-02-081-1/+2
|\ | | | | | | | | PiperOrigin-RevId: 427207981 Change-Id: Ib93f8f61bf568d4651f9148fbc522288308c0d71
| * Fix gtest-help-test failure on OpenBSDrefs/pull/3751/headAndrew Krasavin2022-02-051-1/+2
| |
* | Merge pull request #3746 from ↵Copybara-Service2022-02-081-3/+21
|\ \ | | | | | | | | | | | | | | | | | | IYP-Programer-Yeah:use-constant-time-lookup-for-exact-match PiperOrigin-RevId: 427179775 Change-Id: I9928be2421d559acf0e0f03643ce0b856b63f737
| * | Apply requested changes by using std::inserter with move.refs/pull/3746/headHossein Ghahramanzadeh2022-02-051-9/+11
| | |
| * | Do constant time matching for exact match filters.Hossein Ghahramanzadeh2022-01-311-3/+16
| | |
* | | Merge pull request #3617 from Bagira80:fix_3616Copybara-Service2022-02-072-4/+6
|\ \ \ | |_|/ |/| | | | | | | | PiperOrigin-RevId: 426976022 Change-Id: Ib9f49cf84775a50cbc806d331349e61a1060b498
| * | CMake: Fix values of INTERFACE_INCLUDE_DIRECTORIES propertyrefs/pull/3617/headDeniz Bahadir2021-10-192-4/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replace semicolon by `$<SEMICOLON>` in generator-expressions of target property `INTERFACE_INCLUDE_DIRECTORIES` of CMake targets `gtest`, `gtest_main`, `gmock` and `gmock_main`. Fixes: #3616 Signed-off-by: Deniz Bahadir <deniz@code.bahadir.email>
* | | Merge pull request #3724 from jjfvanderpol:mainCopybara-Service2022-02-041-0/+4
|\ \ \ | | | | | | | | | | | | | | | | PiperOrigin-RevId: 426450614 Change-Id: Iccbe56ddd81f4f3935b3d44aa5b835146758ff83
| * | | Set CMake Policy CMP0077 to NEWrefs/pull/3724/headJoris van der Pol2022-01-051-0/+4
| | | |
* | | | GetCurrentOsStackTraceExceptTop (both the method of UnitTestImpl and the ↵Abseil Team2022-02-013-3/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | wrapper function in gtest.cc) rely on the fact that the inner call is not getting optimized. This CL annotates them with the appropriate attributes. PiperOrigin-RevId: 425663217 Change-Id: Ib9ec2a69a7dd98d37640b56d4d7798572da66669
* | | | Merge pull request #3744 from danger89:patch-2Copybara-Service2022-02-011-1/+1
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | PiperOrigin-RevId: 425519462 Change-Id: I196b22675e471f66929c1cdd0f85a405ebac8484
| * | | Let me give a change to try it again - updating to latest versionrefs/pull/3744/headMelroy van den Berg2022-01-251-1/+1
| | | |
* | | | Finish some missed pieces of the TestCase to TestSuite MigrationDerek Mauro2022-01-283-13/+13
| | | | | | | | | | | | | | | | | | | | PiperOrigin-RevId: 424864779 Change-Id: Iac5cafa3568f5fe41c85c52d28f7d61845f76868
* | | | Change `ReturnArg` to use perfect forwarding of arguments (#3733)Dino Radakovic2022-01-262-3/+9
|/ / / | | | | | | | | | | | | PiperOrigin-RevId: 424355706 Change-Id: I618e5574b4b2c56a343905c20d8cc6d2a70cbcd1
* | | Make recreate_environments_when_repeating=false the default.Abseil Team2022-01-254-8/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | So that global test environments are by default set up and torn down once, regardless of the value of the repeat flag. The point of global environments is to be set up and torn down once, and shared by all tests in the process. There is no obvious reason why multiple runs of the same test should be treated distinctly from single runs of different tests. Having this be false by default means that repeats using a global environment run faster. It can still be set to true if it's desired that every repeat get a fresh environment, but this seems less important given the nature of a global environment. Every test I've seen using a global environment uses it to set up some expensive external resource, not something that can/should be set up for each test anew. (Again this is unsurprising, since the environment is a global.) PiperOrigin-RevId: 424003937 Change-Id: I9e8a825cb8900960dd65b85fe5ffcc0a337e57f3
* | | Merge pull request #3723 from ramkumar-kr:patch-1Copybara-Service2022-01-241-2/+2
|\ \ \ | | | | | | | | | | | | | | | | PiperOrigin-RevId: 423920083 Change-Id: I5e172813387f841d5a6c56eea155b876a465d208
| * | | Update example to reflect the tests belowrefs/pull/3723/headRam Kumar K R2022-01-041-2/+2
| |/ /
* | | Merge pull request #3732 from joshiayush:i2174Copybara-Service2022-01-241-4/+0
|\ \ \ | | | | | | | | | | | | | | | | PiperOrigin-RevId: 423906844 Change-Id: I1c34a3ce8a5c761ef9f1d7ce48bef8826742e97d
| * | | FIX #2174 -- remove `DEBUG_POSTFIX`refs/pull/3732/headAyush Joshi2022-01-131-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | while this is not synced with the pkg-config file, it only breaks things Fix #2174 Signed-off-by: Ayush Joshi <ayush854032@gmail.com>
* | | | Merge pull request #3615 from IYP-Programer-Yeah:fix-per-test-filter-processingCopybara-Service2022-01-211-51/+94
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | PiperOrigin-RevId: 423326942 Change-Id: I913f31960d7917b176c9f390424630708473837a
| * | | | Fix a typo in comments.refs/pull/3615/headHossein Ghahramanzadeh2022-01-061-1/+1
| | | | |