Skip to content

Commit

Permalink
Update README
Browse files Browse the repository at this point in the history
  • Loading branch information
slonopotamus committed Aug 2, 2024
1 parent e26c3e4 commit 3853be6
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions README.adoc
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
= UEST (Unreal Engine Suckless Testing)
:icons: font

CAUTION: This is pre-alpha software, not ready for production!

Expand Down Expand Up @@ -128,9 +129,10 @@ TEST_CASE("MyGame.MyTest", "[ApplicationContextMask][ProductFilter]")
Well, no.
+
--
The caveat is that `TEST_CASE` macro uses a broken way to generate unique class names.
The caveat is that `TEST_CASE` macro uses a very broken way to generate unique class names.
They collide across compilation units and namespaces, and you end up asking yourself "why my test doesn't register at all".
Instead, Epics tell users to use `TEST_CASE_NAMED`, where you need to write test name _twice_.
That way, you end up with the same test class name collision chances as other approaches.
--

.The bad
Expand Down Expand Up @@ -172,7 +174,7 @@ TEST(MyTest, "MyGame")
.The bad
* Not understood by Rider (https://youtrack.jetbrains.com/issue/RSCPP-36039/Support-Unreal-Engine-CQTest-framework)[RSCPP-36039]).
Not sure about VS, would not be surprised if situation is the same.
* Async execution degraded to Automation Test style
* Async execution is as bad as in Automation Test style
* `clang-format` is unable to properly indent `TEST_CLASS` with nested `TEST_METHOD`

.The ugly
Expand Down

0 comments on commit 3853be6

Please sign in to comment.