MLECO-4490: Upgrade MLEK Catch Unit tests

*Removed default behaviour of unit-test executables to redirect output
to file.
*Accompanying change made to CI script to generate xml files there instead.
*Switched to newer minor version of Catch.
*Note: Support for simultaneous output to console and file is available in new major version (Catch 3).

Other unit test cleanup:
* Redundant warnings about common tests output removed
* Static cast added to silence compiler warning
* Removed additional printing in RNNoiseModelTests

Signed-off-by: Liam Barry <liam.barry@arm.com>
Change-Id: If9d7f366ec205a7cd49a45198a15867ef7b75afd

1 files changed, 0 insertions, 6 deletions

diff --git a/tests/use_case/inference_runner/InferenceTestInferenceRunner.cc b/tests/use_case/inference_runner/InferenceTestInferenceRunner.cc
index 7988ae1..974d2c8 100644
--- a/tests/use_case/inference_runner/InferenceTestInferenceRunner.cc
+++ b/tests/use_case/inference_runner/InferenceTestInferenceRunner.cc
@@ -33,9 +33,6 @@ namespace app {
 
 TEST_CASE("Testing Init failure due to insufficient tensor arena inf runner", "[inf runner]")
 {
-    /* Note: Errors or warnings generated by this test will appear in output of any subsequent
-     * failing tests causing misleading output. Give warning until solution is found */
-    printf("Insufficient tensor arena test output:\n");
     arm::app::TestModel model{};
     REQUIRE_FALSE(model.IsInited());
     size_t insufficientTensorArenaSz = 1000;
@@ -43,8 +40,5 @@ TEST_CASE("Testing Init failure due to insufficient tensor arena inf runner", "[
                              insufficientTensorArenaSz,
                              arm::app::inference_runner::GetModelPointer(),
                              arm::app::inference_runner::GetModelLen()));
-
-    printf("End of insufficient tensor arena test output. \nERROR messages above this line are "
-           "expected and can be ignored.\n\n");
     REQUIRE_FALSE(model.IsInited());
 }