From 7ee260c54921571b18b15049304426fe151c1265 Mon Sep 17 00:00:00 2001 From: Abseil Team Date: Mon, 3 Apr 2023 14:00:54 -0700 Subject: [PATCH] Rolled back due to breaking existing tests. PiperOrigin-RevId: 521555658 Change-Id: I09742faceb82b2b7ceb423e850a8b50d532ad6ff --- googletest/src/gtest.cc | 20 -------------------- 1 file changed, 20 deletions(-) diff --git a/googletest/src/gtest.cc b/googletest/src/gtest.cc index c2581fa0..7832492d 100644 --- a/googletest/src/gtest.cc +++ b/googletest/src/gtest.cc @@ -44,7 +44,6 @@ #include // NOLINT #include #include -#include #include #include #include @@ -2982,25 +2981,6 @@ void TestSuite::Run() { TestEventListener* repeater = UnitTest::GetInstance()->listeners().repeater(); - // Ensure our tests are in a deterministic order. - // - // We do this by sorting lexicographically on (file, line number), providing - // an order matching what the user can see in the source code. - // - // In the common case the line number comparison shouldn't be necessary, - // because the registrations made by the TEST macro are executed in order - // within a translation unit. But this is not true of the manual registration - // API, and in more exotic scenarios a single file may be part of multiple - // translation units. - std::stable_sort(test_info_list_.begin(), test_info_list_.end(), - [](const TestInfo* const a, const TestInfo* const b) { - if (const int result = std::strcmp(a->file(), a->file())) { - return result < 0; - } - - return a->line() < b->line(); - }); - // Call both legacy and the new API repeater->OnTestSuiteStart(*this); // Legacy API is deprecated but still available