summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | Googletest exportAbseil Team2020-12-021-2/+0
| | | | | | | | | | | | | | | | Remove `status` from our internal Subprocess utility. This facility is unused, so better to just remove it than figure out what its cross-OS semantics should be. PiperOrigin-RevId: 345279290
* | Googletest exportAbseil Team2020-12-021-2/+2
| | | | | | | | | | | | | | | | Check for a high bit to see if a return value is a signal or an error code in googletest/test. This is needed because for subprocess under python3 windows, a return value representing a C signal (such as 0x80000003) is represented as a large positive integer rather than a negative one. PiperOrigin-RevId: 345270460
* | Googletest exportAbseil Team2020-12-021-0/+1
| | | | | | | | | | | | | | | | Include TargetConditionals.h before checking TARGET_OS_IPHONE New versions of Clang require that you include TargetConditionals.h before checking the TARGET_OS_* macros. Include it where appropriate to keep the compiler happy. PiperOrigin-RevId: 345106443
* | Googletest exportdmauro2020-12-024-24/+13
| | | | | | | | | | | | Fix the TravisCI build by updating the required tools PiperOrigin-RevId: 344882646
* | Googletest exportAbseil Team2020-12-021-1/+1
| | | | | | | | | | | | Put "value" in SetArgReferee's description between code backticks. PiperOrigin-RevId: 344054658
* | Googletest exportAbseil Team2020-11-241-0/+2
| | | | | | | | | | | | Internal change: Add self-shortlink PiperOrigin-RevId: 343960442
* | Googletest exportAbseil Team2020-11-241-0/+12
| | | | | | | | | | | | | | | | Update note on static const data members for C++17. Using `constexpr` provides a tidier solution, where applicable. PiperOrigin-RevId: 343276402
* | Merge pull request #3114 from marbre:FindPythonGennadiy Rozental2020-11-241-1/+7
|\ \ | | | | | | | | | PiperOrigin-RevId: 342977338
| * | Refactor finding pythonMarius Brehler2020-11-111-1/+7
| | | | | | | | | | | | | | | Use `find_package(Python ..)` if available, instead of the deprecated `find_package(PythonInterp)` CMake module.
* | | Merge pull request #3118 from vinsonlee:initialize-TestInfo-is_in_another_shard_Derek Mauro2020-11-171-0/+1
|\ \ \ | | | | | | | | | | | | PiperOrigin-RevId: 342731771
| * | | Initialize TestInfo member is_in_another_shard_ in constructor.Vinson Lee2020-11-131-0/+1
|/ / / | | | | | | | | | Signed-off-by: Vinson Lee <vlee@freedesktop.org>
* | | Googletest exportAbseil Team2020-11-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Use override instead of virtual for destructor https://google.github.io/styleguide/cppguide.html says: "Explicitly annotate overrides of virtual functions or virtual destructors with exactly one of an override or (less frequently) final specifier. Do not use virtual when declaring an override". The mocked class _should_ have a virtual destructor most of the times. PiperOrigin-RevId: 342082140
* | | Googletest exportAbseil Team2020-11-121-5/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | Use environment variable TEST_TMPDIR in Linux environments for temporary directory if available otherwise use /tmp/. Bazel sets the environment variable TEST_TMPDIR when launching tests. PiperOrigin-RevId: 342058921
* | | Googletest exportAbseil Team2020-11-124-15/+17
| | | | | | | | | | | | | | | | | | | | | Use a tagged constructor for FlatTuple instead. Some versions of MSVC are getting confused with that constructor and generating invalid code. PiperOrigin-RevId: 342050957
* | | Googletest exportdmauro2020-11-121-0/+20
| | | | | | | | | | | | | | | | | | | | | Disable the wide character printing tests that were discovered to fail under MSVC. Created a bug to debug this. PiperOrigin-RevId: 341886318
* | | Googletest exportdmauro2020-11-121-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | Fix *_THROW tests under MSVC, which produce slightly different type names from std::type_info (e.g. "class std::runtime_error" instead of "std::runtime_error") PiperOrigin-RevId: 341874178
* | | Googletest exportdmauro2020-11-123-64/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rollback change from https://github.com/google/googletest/pull/1836. This change generates a script on Windows to actually run each test, but the script itself doesn't correctly report if the test passed. This change will "break tests" that were already broken on Windows, but weren't being reported as such. PiperOrigin-RevId: 341850671
* | | Googletest exportAbseil Team2020-11-111-3/+1
|/ / | | | | | | | | | | Remove unreachable code when GTEST_CUSTOM_TEMPDIR_FUNCTION_ is defined. PiperOrigin-RevId: 341810264
* | Merge pull request #3109 from hyukmyeong:typoMark Barolak2020-11-104-16/+16
|\ \ | | | | | | | | | PiperOrigin-RevId: 341604515
| * | fix typosHyuk Myeong2020-11-084-16/+16
| | |
* | | Merge pull request #3090 from knutpett:working_dir_on_disklessMark Barolak2020-11-101-0/+2
|\ \ \ | | | | | | | | | | | | PiperOrigin-RevId: 341604408
| * | | Only save original working directory if death tests are enabledKnut Petter Svendsen2020-11-091-0/+2
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | On a diskless system you cannot get the current directory. So if death tests are disabled anyway, there is no point trying to get current directory. Without this fix, running tests on diskless systems will fail, even when death tests are disabled.
* | | Googletest exportAbseil Team2020-11-093-219/+210
| | | | | | | | | | | | | | | | | | | | | | | | Change ACTION{,_Pn,_TEMPLATE} macros to build functors rather than ActionInterface<> subclasses, thus changing the Action<> wrappers they create to use the modernized (non-const) argument tuple type, allowing these macros to mutate their arguments. Functor-based Action<>s deep-copy the implementing object, so have the functors use a shared_ptr to the non-trivial state of bound value parameters. No longer specialize that shared state to the particular action signature, encoding that information instead only in the instantiation of the implementation function. PiperOrigin-RevId: 341116208
* | | Googletest exportAbseil Team2020-11-092-1/+9
| |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | GMock: Make Truly explain when it fails I just wrote a test that had a matcher of the form Optional(AllOf( SomeMatcher, SomeOtherMatcher, Truly(SomePredicate))) The predicate failed, the other two matchers succeeded, and I got a hard-to-interpret message saying that the value in the optional "didn't match". Didn't match what? This change improves situations like that slightly by having Truly explain to its result listener when it fails. When there are multiple Trulys in an AllOf, there will be some ambiguity, but it will at least provide more information than right now. PiperOrigin-RevId: 341105141
* | Googletest exportAbseil Team2020-11-063-28/+118
| | | | | | | | | | | | Add support for ref qualifiers in MOCK_METHOD. PiperOrigin-RevId: 341047839
* | Googletest exportofats2020-11-053-103/+76
| | | | | | | | | | | | Rewrite InvokeArgument action without using pump. PiperOrigin-RevId: 340861582
* | Merge pull request #3008 from hyukmyeong:update_tutorialAndy Getz2020-11-023-61/+79
|\ \ | | | | | | | | | PiperOrigin-RevId: 340286884
| * | Remove spaces between Google Test and Google MockHyuk Myeong2020-09-163-58/+58
| | |
| * | Add follow-up patch for more natural readingHyuk Myeong2020-09-151-1/+2
| | |
| * | Apply the reviewed commentHyuk Myeong2020-09-151-13/+17
| | |
| * | Remove a spaceHyuk Myeong2020-09-151-1/+1
| | |
| * | Improve the tutorial that may be confusingHyuk Myeong2020-09-152-22/+30
| | |
* | | Merge pull request #2714 from kuzkry:remove-repeated-includeAndy Getz2020-11-021-4/+0
|\ \ \ | | | | | | | | | | | | PiperOrigin-RevId: 340266002
| * | | remove a duplicated includeKrystian Kuzniarek2020-06-121-4/+0
| | | | | | | | | | | | | | | | | | | | | | | | Detection of string_view type (whether it's std or Abseil) is done in googletest/include/gtest/internal/gtest-port.h with GTEST_INTERNAL_HAS_STRING_VIEW.
* | | | Merge pull request #3086 from sfc-gh-ebrossard:masterAndy Getz2020-11-021-0/+29
|\ \ \ \ | | | | | | | | | | | | | | | PiperOrigin-RevId: 340226421
| * | | | Add instructions for sanitizer integrationElliott Brossard2020-10-271-0/+29
| | |_|/ | |/| |
* / | | Googletest exportAbseil Team2020-10-301-3/+0
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove obsolete comment about non-const references. They used to be banned by the C++ style guide, but positions shifted, and this comment is no longer up to date. There's another reference (heh) on lines 816-819 to how gmock is a "general framework", with the possible implication that support for non-const references was weird in some way and only there to be "general", but I left it alone because I don't really feel I understand what it's saying. PiperOrigin-RevId: 339323428
* | | Merge pull request #3044 from manavrion:improve_file_path_normalizevslashg2020-10-271-7/+5
|\ \ \ | | | | | | | | | | | | PiperOrigin-RevId: 339242159
| * | | Improve FilePath::Normalize methodRuslan Manaev2020-10-071-7/+5
| | | |
* | | | Googletest exportAbseil Team2020-10-271-53/+53
| | | | | | | | | | | | | | | | | | | | | | | | I'm not sure how this relates to the GitHub repo. Could you please advise? PiperOrigin-RevId: 339060470
* | | | Merge pull request #3073 from ArthurSonzogni:fix-typoGennadiy Rozental2020-10-222-2/+2
|\ \ \ \ | | | | | | | | | | | | | | | PiperOrigin-RevId: 338093064
| * | | | Fix typo "definedin in" => "defined in"Arthur Sonzogni2020-10-202-2/+2
|/ / / / | | | | | | | | | | | | | | | | | | | | Replace "definedin in" by "defined in" in files: - googletest/src/gtest.cc - googletest/test/googletest-output-test-golden-lin.txt
* | | | Googletest exportAbseil Team2020-10-191-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | Fix typo in the "Assertion Placement" section PiperOrigin-RevId: 337435223
* | | | Googletest exportAbseil Team2020-10-191-5/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix some issues when running fuse_gmock_files. The module path should be updated before importing `fuse_gtest_files`, since the script may not run from the googletest repo root. We also need a non-frozen set in order to track progress. PiperOrigin-RevId: 337380466
* | | | Googletest exportdmauro2020-10-153-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes build warnings from previous CL Add CMake to internal presubmit to prevent these PiperOrigin-RevId: 337325504
* | | | Merge pull request #3042 from ↵Derek Mauro2020-10-151-9/+9
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | Aralox:Issue-2135-modify-mock-template-args-AX-to-TArgX PiperOrigin-RevId: 337217118
| * | | | Issue 2135: Change template args in NiceMock, NaggyMock and StrictMock from ↵Aralox2020-10-071-9/+9
| |/ / / | | | | | | | | | | | | A1, A2, ... to TArg1, TArg2,... to avoid clash with legacy header files
* | | | Googletest exportAbseil Team2020-10-144-3/+364
| | | | | | | | | | | | | | | | | | | | Add ::testing::FieldsAre matcher for objects that support get<> and structured bindings. PiperOrigin-RevId: 337165285
* | | | Googletest exportdmauro2020-10-143-4/+60
| | | | | | | | | | | | | | | | | | | | | | | | Disable warnings on code that intentionally tests a suboptimal syntax PiperOrigin-RevId: 337138442
* | | | Googletest exportdmauro2020-10-141-11/+18
| | | | | | | | | | | | | | | | | | | | | | | | Stop using master.zip to make the build reproducible PiperOrigin-RevId: 337102716
OpenPOWER on IntegriCloud