Re: [edk2-devel] [Patch 3/7] UnitTestFrameworkPkg: Add googletest submodule and GoogleTestLib

["Michael D Kinney" <michael.d.kinney@intel.com>]

Hi Michael,

Here is a first pass at a feature matrix.  Can you think of other features to add?

### Framework and GoogleTest Feature Comparison

| Feature                   | Framework | GoogleTest |
|:--------------------------|:---------:|:----------:|
| Host based unit tests     |    YES    |    YES     |
| Target based unit tests   |    YES    |     NO     |
| Unit test source language |     C     |    C++     |
| Register Test Suite       |    YES    |    Auto    |
| Register Test Case        |    YES    |    Auto    |
| Setup/Teardown Hooks      |    YES    |    YES     |
| Value-Parameterized Tests |    NO     |    YES     |
| Typed Tests               |    NO     |    YES     |
| Type-Parameterized Tests  |    NO     |    YES     |
| Mocking Support           |   Cmocka  |     NO     |
| JUNIT XML Reports         |    YES    |    YES     |
| Execute subset of tests   |    NO     |    YES     |
| VS Code Extensions        |    NO     |    YES     |

Thanks,

Mike



From: Michael Kubacki <mikuback@linux.microsoft.com> 
Sent: Friday, November 4, 2022 7:45 AM
To: Kinney, Michael D <michael.d.kinney@intel.com>; devel@edk2.groups.io
Subject: Re: [edk2-devel] [Patch 3/7] UnitTestFrameworkPkg: Add googletest submodule and GoogleTestLib

Reviewed-by: Michael Kubacki <mailto:michael.kubacki@microsoft.com>

There's a typo in UnitTestFrameworkPkg/ReadMe.md. You'll find it by searching for "ued".

The ReadMe.md updates are very helpful. Something else to consider would be a brief feature matrix at the top for the "framework" vs GoogleTest. I think it would be a little easier to parse which to choose than picking apart the key differences in the text.