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 <[email protected]> 
Sent: Friday, November 4, 2022 7:45 AM
To: Kinney, Michael D <[email protected]>; [email protected]
Subject: Re: [edk2-devel] [Patch 3/7] UnitTestFrameworkPkg: Add googletest 
submodule and GoogleTestLib

Reviewed-by: Michael Kubacki <mailto:[email protected]>

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. 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#95962): https://edk2.groups.io/g/devel/message/95962
Mute This Topic: https://groups.io/mt/94799427/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to