aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | Googletest exportAbseil Team2020-01-091-1/+1
| | | | | | | | | | | | | | | | | | | | | Close #2658 Mark test case destrucors as override. PiperOrigin-RevId: 288507991
* | | Googletest exportAbseil Team2020-01-093-134/+42
| | | | | | | | | | | | | | | | | | | | | | | | Use C++11 variadic templates for Invoke in gmock-generated-actions.h. Replace InvokeArgumentAdl with Invoke that uses C++11 variadic templates. PiperOrigin-RevId: 288449236
* | | Googletest exportAbseil Team2020-01-021-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Explicitly default copy constructor in BoundSecondMatcher Since C++11, implicit defaulting of copy constructors is deprecated for types with user-defined copy assignment operators, so we should explicitly default the copy constructor of BoundSecondMatcher. PiperOrigin-RevId: 287587847
* | | Merge pull request #2632 from Kravlalala:cmake/mingw_msys_buildAndy Soffer2020-01-021-1/+1
|\ \ \ | | | | | | | | | | | | PiperOrigin-RevId: 287587126
| * | | Activate GNU extensions in case of MSYS generatorIlya Kravchuk2019-12-131-1/+1
| | | |
* | | | Googletest exportAbseil Team2020-01-021-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Add missing period. PiperOrigin-RevId: 287553550
* | | | Googletest exportAbseil Team2020-01-021-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Fix typo in example. PiperOrigin-RevId: 287212448
* | | | Merge pull request #2639 from trzecieu:trzeci/move_ctor_assignAndy Soffer2020-01-022-5/+21
|\ \ \ \ | | | | | | | | | | | | | | | PiperOrigin-RevId: 286896167
| * | | | Make move operation noexcept.Piotr Paczkowski (trzeci.eu)2019-12-201-2/+2
| | | | |
| * | | | Define default destructor for test classesPiotr Paczkowski (trzeci.eu)2019-12-201-0/+1
| | | | |
| * | | | Deleted functions as part of public interfacePiotr Paczkowski (trzeci.eu)2019-12-201-4/+4
| | | | |
| * | | | Review notes: Return T& from assignment operatorsPiotr Paczkowski (trzeci.eu)2019-12-201-2/+2
| | | | |
| * | | | Disable move constructor and assignment operator for test classes.trzeci2019-12-172-2/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Disable move operations for TEST() and TEST_F() macros. Previous implementation disabled only copy ctor and assing operator, but this was violating rule of 5[1], which was captured by static code analysis tools like clang-tidy `cppcoreguidelines-special-member-functions`. [1]: https://github.com/isocpp/CppCoreGuidelines/blob/master/CppCoreGuidelines.md#c21-if-you-define-or-delete-any-default-operation-define-or-delete-them-all
* | | | | Googletest exportAbseil Team2020-01-0210-14/+145
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add option (default to disabled) to make C++ type parameterized tests (TYPED_TEST_P) fail when they're not instantiated. When an un-instantiated TYPED_TEST_P is found, a new test will be inserted that emits a suitable message. For now, that is just a notice, but the hope it to flip the bit to make it fail by default. PiperOrigin-RevId: 286408038
* | | | | Merge pull request #2624 from ShabbyX:masterAndy Soffer2020-01-026-21/+34
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | PiperOrigin-RevId: 286397298
| * | | | | Revert "Googletest export": disallow empty prefixShahbaz Youssefi2019-12-164-20/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 20b5b8ecc7a81d23b4716e22a2b35fd53379c8c6. Reason for revert: Breaks existing applications, such as ANGLE (angleproject.org), requires adding an extra prefix that needs to be typed for no reason (when testing from command line) and increases the logs' sizes by a non-trivial amount due to the very large number of tests ANGLE runs. Original commit message: Add a compile time check to ensure that the parameters to TEST_P and INSTANTIATE_TEST_SUITE_P are not empty. Some compilers already fail in that case and, even where it works, it's likely to result in technically invalid code by virtue of creating reserved identifiers: https://en.cppreference.com/w/cpp/language/identifiers First, every project is perfectly capable of adding a prefix if they want to support such a compiler. This change penalizes every project. Second, using a prefix such as `_p` also results in reserved identifiers, so this change is not really solving the problem. For that matter, instead of generating `gtest_##prefix##...`, you can generate `gtest_x##prefix##...` to correctly fix the issue, including when empty prefixes are used.
| * | | | | Revert "Googletest export": Remove test for empty prefixShahbaz Youssefi2019-12-163-4/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 4c25e2b87fcb78abcfdee2739db9ed7a19754cd3. Reason for revert: Reverting the commit that disallows empty prefixes. Original commit message: Remove a test case rendered obsolete by disallowing empty argument for INSTANTIATE_TEST_SUITE_P. Remove the code that it was testing.
| * | | | | Workaround VS bug w.r.t empty arguments to macrosShahbaz Youssefi2019-12-161-1/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Empty arguments can be passed to macros per C99 and C++11 specs, which can then be forwarded to other macros. Visual Studio's compiler has a bug in the following situation: #define A(x) #x #define B(x, y) A(x) B(, b) In the above case, Visual Studio first expands x to nothing, then complains that A is not invoked with the right amount of arguments. However, x in A(x) is still one argument, even if it expands to no preprocessing tokens. See also https://stackoverflow.com/a/7674214.
* | | | | | Googletest exportAbseil Team2020-01-023-17/+98
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use C++11 variadic templates for InvokeArgumentAdl in gmock-generated-actions.h. Make InvokeArgumentAdl use variadic templates to generate its overloads instead of using pump.py syntax. PiperOrigin-RevId: 286267615
* | | | | | Googletest exportAbseil Team2020-01-023-98/+17
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use C++11 variadic templates for InvokeArgumentAdl in gmock-generated-actions.h. Make InvokeArgumentAdl use variadic templates to generate its overloads instead of using pump.py syntax. PiperOrigin-RevId: 286148805
* | | | | Merge pull request #2590 from kuzkry:remove-workaround_g++-stale-commentsCJ Johnson2019-12-161-12/+0
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | PiperOrigin-RevId: 285812343
| * | | | | remove stale comments about older GCC versionsKrystian Kuzniarek2019-11-221-12/+0
| | | | | |
* | | | | | Googletest exportkrzysio2019-12-161-18/+17
| |/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't use fully qualified ::std types in code examples. Having a nested user-defined 'std' namespace anywhere in a program is a minefield and shouldn't be either explicitly supported or implicitly condoned. PiperOrigin-RevId: 285790182
* | | | | Merge pull request #2589 from ↵Matt Calabrese2019-12-131-9/+3
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | kuzkry:remove-workaround_g++-no-space-after-first-macro-argument PiperOrigin-RevId: 285255373
| * | | | | remove g++ 2.95.0 workaround: no space after first comma in macrosKrystian Kuzniarek2019-11-221-9/+3
| |/ / / /
* | | | | Googletest exportAbseil Team2019-12-131-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add GTEST_API_ to correct a build failure under MSCV. PiperOrigin-RevId: 285208464
* | | | | Googletest exportAbseil Team2019-12-133-175/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use C++11 variadic templates for ActionHelper in gmock-generated-actions.h. Make ActionHelper use variadic templates to generate Perform static member function specializations instead of using pump.py syntax. PiperOrigin-RevId: 284988441
* | | | | Googletest exportAbseil Team2019-12-135-4/+96
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Detect when C++ parametric tests (TEST_P) are not instantiated. When an un-instantiated TEST_P is found, a new test will be inserted that will emit a warning message. This can be made to error with minor code edits. In the future, that is intended to be the default. PiperOrigin-RevId: 284901666
* | | | | Merge pull request #2595 from kuzkry:remove-workaround_msvc-warning-4355Matt Calabrese2019-12-131-8/+0
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | PiperOrigin-RevId: 284234675
| * | | | | remove MSVC workaround: warning 4355Krystian Kuzniarek2019-11-221-8/+0
| |/ / / /
* | | | | Merge pull request #2592 from kuzkry:remove-workaround_msvc-error-C2665Matt Calabrese2019-12-131-9/+0
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | PiperOrigin-RevId: 284207090
| * | | | | remove MSVC workaround: error C2665Krystian Kuzniarek2019-11-221-9/+0
| |/ / / /
* | / / / Googletest exportmisterg2019-12-131-0/+2
| |/ / / |/| | | | | | | | | | | | | | | | | | | Require all appveyor googletest windows builds to work on Pull Requests to increase CI coverage PiperOrigin-RevId: 284206759
* | | | Merge pull request #2609 from kuzkry:revert-2596Mark Barolak2019-12-062-11/+13
|\ \ \ \ | | | | | | | | | | | | | | | PiperOrigin-RevId: 284041013
| * | | | Revert "remove MSVC workaround: wmain link error in the static library"Krystian Kuzniarek2019-11-271-2/+9
| | | | | | | | | | | | | | | | | | | | This reverts commit 298a40f023e6813d2bf73847c3a38ceaf5e46320.
| * | | | Revert "unify googletest and googlemock main functions"Krystian Kuzniarek2019-11-272-9/+4
| | | | | | | | | | | | | | | | | | | | This reverts commit a909becdc599c46bcb57346b6123cb57cd07d15d.
* | | | | Googletest exportAbseil Team2019-12-066-31/+152
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change googletest to notice failures during SetUpTestSuite() and TearDownTestSuite(). Previously, errors that occurred during those functions were logged but otherwise ignored. After this change, such failures will cause the test to fail and a brief summary will be printed at the bottom of the test log. See https://github.com/google/googletest/issues/2330. PiperOrigin-RevId: 284033342
* | | | | Googletest exportAbseil Team2019-12-051-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update example Environment class destructor annotation PiperOrigin-RevId: 283986183
* | | | | Merge pull request #2594 from ↵Mark Barolak2019-12-051-17/+4
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | kuzkry:remove-workaround_msvc-unneeded-const-dropping PiperOrigin-RevId: 283979494
| * | | | | remove MSVC workaround: cease const droppingKrystian Kuzniarek2019-11-221-14/+3
| | |/ / / | |/| | |
* | | | | Googletest exportAbseil Team2019-12-051-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Internal change PiperOrigin-RevId: 283948137
* | | | | Googletest exportAbseil Team2019-12-021-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Clarify use of IsTrue and IsFalse matchers. These matchers are subtle and confusing: what are they for? The docs in the code are clear, but not very accessible. googletest/googlemock/include/gmock/gmock-more-matchers.h PiperOrigin-RevId: 283393275
* | | | | Merge pull request #2603 from maximilianschwab:patch-1Mark Barolak2019-12-021-1/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | PiperOrigin-RevId: 283349646
| * | | | | Fixed typoMaximilian Schwab2019-11-251-1/+1
| | | | | |
* | | | | | Merge pull request #2607 from oyefremov:patch-1Mark Barolak2019-12-021-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | PiperOrigin-RevId: 283349621
| * | | | | | Rename test case to test suiteOleksandr Yefremov2019-11-271-1/+1
| | |_|/ / / | |/| | | |
* | | | | | Merge pull request #2583 from ChristophStrehle:masterGennadiy Rozental2019-11-271-1/+2
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | PiperOrigin-RevId: 282817206
| * | | | | Fix compile break for Microsoft Visual Studio 2017 v141Christoph Strehle2019-11-211-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is a workaround, for those who have to compile with v141 build tools, for a bug in msvc that the compiler can't compile the WithArgsAction. see the following link for more details: https://developercommunityapi.westus.cloudapp.azure.com/content/problem/420339/googlemocks-withargs-doesnt-compile-with-permissiv.html
* | | | | | Merge pull request #2604 from matepek:readme-update-with-opensource-projGennadiy Rozental2019-11-261-0/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | PiperOrigin-RevId: 282620313
| * | | | | | README.md: added Catch2 and Google Test ExplorerMate Pek2019-11-251-0/+4
| | |/ / / / | |/| | | |