Skip to content

Add gtest-tesults v1.0.1#52154

Open
ajeetd wants to merge 1 commit into
microsoft:masterfrom
ajeetd:add-gtest-tesults
Open

Add gtest-tesults v1.0.1#52154
ajeetd wants to merge 1 commit into
microsoft:masterfrom
ajeetd:add-gtest-tesults

Conversation

@ajeetd
Copy link
Copy Markdown
Contributor

@ajeetd ajeetd commented Jun 2, 2026

Summary

This PR adds the gtest-tesults port — a header-only Google Test event listener that reports test results to Tesults.

Port details

Field Value
Name gtest-tesults
Version 1.0.1
Type Header-only (INTERFACE library)
Dependencies gtest, tesults-cpp

Usage

Set TESULTS_TARGET and register the listener before running tests:

#include <gtest_tesults/gtest_tesults.h>
// auto-registers from TESULTS_TARGET env var — no code changes needed

Or register manually:

auto cfg = gtest_tesults::config_from_args(argc, argv);
testing::UnitTest::GetInstance()->listeners().Append(new gtest_tesults::TesultsListener(cfg));

Testing

Port was tested locally on macOS (Apple Silicon) with:

  • CMake 3.14+
  • AppleClang 21
  • vcpkg install gtest-tesults

Checklist

  • Port files follow vcpkg conventions
  • vcpkg x-add-version run and versioning files committed
  • SHA512 verified against GitHub archive
  • vcpkg_install_copyright included
  • Header-only: set(VCPKG_BUILD_TYPE release) and debug/lib dirs removed
  • find_dependency declarations in installed CMake config
  • Single version in versions file

@@ -0,0 +1,19 @@
{
"name": "gtest-tesults",
Copy link
Copy Markdown
Member

@vicroms vicroms Jun 2, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm worried that the gtest- prefix may confuse people into thinking that this project is related to Google or upstream GTest.

My recommendation is to rename the port as tesults-gtest.

@vicroms vicroms added the requires:vcpkg-team-review This PR or issue requires someone on the vcpkg team to take a further look. label Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

requires:vcpkg-team-review This PR or issue requires someone on the vcpkg team to take a further look.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants