MLECO-4365: Add CI Test for insufficient tensor arena and update common tests.

Previously, execution path following failed tensor allocation
was not covered by tests.

Minor amendment added to make ~Model virtual.
During testing it was noticed that intentional errors simulated in
certain common tests would result in misleading output for subsequent tests
as the error messages appear in the console output of the next failing test.
Documentation and print statements added until a solution is found.

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

2 files changed, 15 insertions, 3 deletions

diff --git a/tests/common/ClassifierTests.cc b/tests/common/ClassifierTests.cc
index 0abaa15..7df4242 100644
--- a/tests/common/ClassifierTests.cc
+++ b/tests/common/ClassifierTests.cc
@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: Copyright 2021 Arm Limited and/or its affiliates <open-source-office@arm.com>
+ * SPDX-FileCopyrightText: Copyright 2021, 2023 Arm Limited and/or its affiliates <open-source-office@arm.com>
  * SPDX-License-Identifier: Apache-2.0
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
@@ -47,10 +47,15 @@ TEST_CASE("Common classifier")
 {
     SECTION("Test invalid classifier")
     {
+        /* 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("Invalid classifier common test output:\n");
         TfLiteTensor* outputTens = nullptr;
         std::vector <arm::app::ClassificationResult> resultVec;
         arm::app::Classifier classifier;
         REQUIRE(!classifier.GetClassificationResults(outputTens, resultVec, {}, 5, true));
+        printf("End of invalid classifier common test output. \nERROR messages above this line are "
+               "expected and can be ignored.\n\n");
     }
 
     SECTION("Test classification results")
diff --git a/tests/common/ProfilerTests.cc b/tests/common/ProfilerTests.cc
index 62c7347..0b487e5 100644
--- a/tests/common/ProfilerTests.cc
+++ b/tests/common/ProfilerTests.cc
@@ -1,5 +1,5 @@
 /*
- * SPDX-FileCopyrightText: Copyright 2021 Arm Limited and/or its affiliates <open-source-office@arm.com>
+ * SPDX-FileCopyrightText: Copyright 2021, 2023 Arm Limited and/or its affiliates <open-source-office@arm.com>
  * SPDX-License-Identifier: Apache-2.0
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
@@ -41,10 +41,17 @@ TEST_CASE("Common: Test Profiler")
         profilerValid.GetAllResultsAndReset(results);
         REQUIRE(results.size() == 1);
         REQUIRE(results[0].name == "test_valid");
-        /* Abuse should still fail: */
+
+        /* Improper usage should cause failure.
+         * 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("Invalid profiler usage common test output:\n");
         REQUIRE(false == profilerValid.StopProfiling()); /* We need to start it first */
+
         REQUIRE(true == profilerValid.StartProfiling()); /* Should be able to start it fine */
         REQUIRE(false == profilerValid.StartProfiling()); /* Can't restart it without resetting */
+        printf("End of Invalid profiler usage common test output. \nERROR messages above this line "
+               "are expected and can be ignored.\n\n");
         profilerValid.Reset();
         REQUIRE(true == profilerValid.StartProfiling()); /* Can start it again now.. */
         REQUIRE(true == profilerValid.StopProfiling());