[various] Upgrade googletest to v1.15.2 (#11586)

Gustl22 · web-flow · commit bf6d7db7f9a8 · 2026-04-29T02:46:22.000Z
Running e.g. pigeon tests (`packages\pigeon\tool\test.dart`) with Visual Studio 2026 gives the following error: ``` ############################## # Generating platform_test/ output [start time 22:58:25] Generation complete! ############################## # Running windows_unittests [start time 22:58:29] Building Windows application... CMake Error at D:/Tmp/Coding/git/flutter/packages/packages/pigeon/platform_tests/test_plugin/example/build/windows/x64/_deps/googletest-src/CMakeLists.txt:4 (cmake_minimum_required): Compatibility with CMake < 3.5 has been removed from CMake. Update the VERSION argument <min> value. Or, use the <min>...<max> syntax to tell CMake that the project requires at least <min> but has been updated to work with policies introduced by <max> or earlier. Or, add -DCMAKE_POLICY_VERSION_MINIMUM=3.5 to try configuring anyway. Building Windows application... 6,9s Unable to generate build files # Failed, exit code: 1 Process finished with exit code 1 ``` So upgrading the google tools to v1.15.2 resolves that issue as since [1.12.1](https://github.com/google/googletest/releases/tag/release-1.12.1) the minimum required CMake version is 3.5. Towards flutter/flutter#185597 ## Pre-Review Checklist **Note**: The Flutter team is currently trialing the use of [Gemini Code Assist for GitHub](https://developers.google.com/gemini-code-assist/docs/review-github-code). Comments from the `gemini-code-assist` bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed. [^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.
diff --git a/packages/camera/camera_windows/windows/CMakeLists.txt b/packages/camera/camera_windows/windows/CMakeLists.txt
@@ -68,7 +68,7 @@ enable_testing()
 include(FetchContent)
 FetchContent_Declare(
   googletest
-  URL https://github.com/google/googletest/archive/release-1.11.0.zip
+  URL https://github.com/google/googletest/archive/v1.15.2.zip
 )
 # Prevent overriding the parent project's compiler/linker settings
 set(gtest_force_shared_crt ON CACHE BOOL "" FORCE)
diff --git a/packages/file_selector/file_selector_linux/linux/CMakeLists.txt b/packages/file_selector/file_selector_linux/linux/CMakeLists.txt
@@ -38,7 +38,7 @@ enable_testing()
 include(FetchContent)
 FetchContent_Declare(
   googletest
-  URL https://github.com/google/googletest/archive/release-1.11.0.zip
+  URL https://github.com/google/googletest/archive/v1.15.2.zip
 )
 # Prevent overriding the parent project's compiler/linker settings
 set(gtest_force_shared_crt ON CACHE BOOL "" FORCE)
diff --git a/packages/file_selector/file_selector_windows/windows/CMakeLists.txt b/packages/file_selector/file_selector_windows/windows/CMakeLists.txt
@@ -50,7 +50,7 @@ enable_testing()
 include(FetchContent)
 FetchContent_Declare(
   googletest
-  URL https://github.com/google/googletest/archive/release-1.11.0.zip
+  URL https://github.com/google/googletest/archive/v1.15.2.zip
 )
 # Prevent overriding the parent project's compiler/linker settings
 set(gtest_force_shared_crt ON CACHE BOOL "" FORCE)
diff --git a/packages/local_auth/local_auth_windows/windows/CMakeLists.txt b/packages/local_auth/local_auth_windows/windows/CMakeLists.txt
@@ -87,7 +87,7 @@ enable_testing()
 # for a template, but not for a monorepo with many plugins.
 FetchContent_Declare(
   googletest
-  URL https://github.com/google/googletest/archive/release-1.11.0.zip
+  URL https://github.com/google/googletest/archive/v1.15.2.zip
 )
 # Prevent overriding the parent project's compiler/linker settings
 set(gtest_force_shared_crt ON CACHE BOOL "" FORCE)
diff --git a/packages/pigeon/platform_tests/test_plugin/linux/CMakeLists.txt b/packages/pigeon/platform_tests/test_plugin/linux/CMakeLists.txt
@@ -82,7 +82,7 @@ enable_testing()
 include(FetchContent)
 FetchContent_Declare(
   googletest
-  URL https://github.com/google/googletest/archive/release-1.11.0.zip
+  URL https://github.com/google/googletest/archive/v1.15.2.zip
 )
 # Prevent overriding the parent project's compiler/linker settings
 set(gtest_force_shared_crt ON CACHE BOOL "" FORCE)
diff --git a/packages/pigeon/platform_tests/test_plugin/windows/CMakeLists.txt b/packages/pigeon/platform_tests/test_plugin/windows/CMakeLists.txt
@@ -89,7 +89,7 @@ enable_testing()
 include(FetchContent)
 FetchContent_Declare(
   googletest
-  URL https://github.com/google/googletest/archive/release-1.11.0.zip
+  URL https://github.com/google/googletest/archive/v1.15.2.zip
 )
 # Prevent overriding the parent project's compiler/linker settings
 set(gtest_force_shared_crt ON CACHE BOOL "" FORCE)
diff --git a/packages/url_launcher/url_launcher_linux/linux/CMakeLists.txt b/packages/url_launcher/url_launcher_linux/linux/CMakeLists.txt
@@ -38,7 +38,7 @@ enable_testing()
 include(FetchContent)
 FetchContent_Declare(
   googletest
-  URL https://github.com/google/googletest/archive/release-1.11.0.zip
+  URL https://github.com/google/googletest/archive/v1.15.2.zip
 )
 # Prevent overriding the parent project's compiler/linker settings
 set(gtest_force_shared_crt ON CACHE BOOL "" FORCE)
diff --git a/packages/url_launcher/url_launcher_windows/windows/CMakeLists.txt b/packages/url_launcher/url_launcher_windows/windows/CMakeLists.txt
@@ -45,7 +45,7 @@ enable_testing()
 include(FetchContent)
 FetchContent_Declare(
   googletest
-  URL https://github.com/google/googletest/archive/release-1.11.0.zip
+  URL https://github.com/google/googletest/archive/v1.15.2.zip
 )
 # Prevent overriding the parent project's compiler/linker settings
 set(gtest_force_shared_crt ON CACHE BOOL "" FORCE)

Original file line number	Diff line number	Diff line change
`@@ -68,7 +68,7 @@ enable_testing()`
`68`	`68`	`include(FetchContent)`
`69`	`69`	`FetchContent_Declare(`
`70`	`70`	`googletest`
`71`		`- URL https://github.com/google/googletest/archive/release-1.11.0.zip`
	`71`	`+ URL https://github.com/google/googletest/archive/v1.15.2.zip`
`72`	`72`	`)`
`73`	`73`	`# Prevent overriding the parent project's compiler/linker settings`
`74`	`74`	`set(gtest_force_shared_crt ON CACHE BOOL "" FORCE)`
Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ enable_testing()`
`38`	`38`	`include(FetchContent)`
`39`	`39`	`FetchContent_Declare(`
`40`	`40`	`googletest`
`41`		`- URL https://github.com/google/googletest/archive/release-1.11.0.zip`
	`41`	`+ URL https://github.com/google/googletest/archive/v1.15.2.zip`
`42`	`42`	`)`
`43`	`43`	`# Prevent overriding the parent project's compiler/linker settings`
`44`	`44`	`set(gtest_force_shared_crt ON CACHE BOOL "" FORCE)`
Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,7 @@ enable_testing()`
`50`	`50`	`include(FetchContent)`
`51`	`51`	`FetchContent_Declare(`
`52`	`52`	`googletest`
`53`		`- URL https://github.com/google/googletest/archive/release-1.11.0.zip`
	`53`	`+ URL https://github.com/google/googletest/archive/v1.15.2.zip`
`54`	`54`	`)`
`55`	`55`	`# Prevent overriding the parent project's compiler/linker settings`
`56`	`56`	`set(gtest_force_shared_crt ON CACHE BOOL "" FORCE)`
Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ enable_testing()`
`87`	`87`	`# for a template, but not for a monorepo with many plugins.`
`88`	`88`	`FetchContent_Declare(`
`89`	`89`	`googletest`
`90`		`- URL https://github.com/google/googletest/archive/release-1.11.0.zip`
	`90`	`+ URL https://github.com/google/googletest/archive/v1.15.2.zip`
`91`	`91`	`)`
`92`	`92`	`# Prevent overriding the parent project's compiler/linker settings`
`93`	`93`	`set(gtest_force_shared_crt ON CACHE BOOL "" FORCE)`
Original file line number	Diff line number	Diff line change
`@@ -82,7 +82,7 @@ enable_testing()`
`82`	`82`	`include(FetchContent)`
`83`	`83`	`FetchContent_Declare(`
`84`	`84`	`googletest`
`85`		`- URL https://github.com/google/googletest/archive/release-1.11.0.zip`
	`85`	`+ URL https://github.com/google/googletest/archive/v1.15.2.zip`
`86`	`86`	`)`
`87`	`87`	`# Prevent overriding the parent project's compiler/linker settings`
`88`	`88`	`set(gtest_force_shared_crt ON CACHE BOOL "" FORCE)`
Original file line number	Diff line number	Diff line change
`@@ -89,7 +89,7 @@ enable_testing()`
`89`	`89`	`include(FetchContent)`
`90`	`90`	`FetchContent_Declare(`
`91`	`91`	`googletest`
`92`		`- URL https://github.com/google/googletest/archive/release-1.11.0.zip`
	`92`	`+ URL https://github.com/google/googletest/archive/v1.15.2.zip`
`93`	`93`	`)`
`94`	`94`	`# Prevent overriding the parent project's compiler/linker settings`
`95`	`95`	`set(gtest_force_shared_crt ON CACHE BOOL "" FORCE)`
Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ enable_testing()`
`45`	`45`	`include(FetchContent)`
`46`	`46`	`FetchContent_Declare(`
`47`	`47`	`googletest`
`48`		`- URL https://github.com/google/googletest/archive/release-1.11.0.zip`
	`48`	`+ URL https://github.com/google/googletest/archive/v1.15.2.zip`
`49`	`49`	`)`
`50`	`50`	`# Prevent overriding the parent project's compiler/linker settings`
`51`	`51`	`set(gtest_force_shared_crt ON CACHE BOOL "" FORCE)`