commit bcf045a471227c72b636b526e70d6df36f1a3395
author Szymon Dompke <szymon.dompke@intel.com> Fri Sep 16 15:23:30 2022 +0200
committer Szymon Dompke <szymon.dompke@intel.com> Fri Sep 16 16:08:47 2022 +0200
tree b194e30b355941e2ffb4bbf5bd440f38ec879315
parent a8182beb1129b0608d2ddfb82895194ea3485288

Fix clang errors

- structs used in emplace_back now have defined constructor
- removed dead code from ensure

Testing done:
- CI tests are passing
- UTs are passing

Signed-off-by: Szymon Dompke <szymon.dompke@intel.com>
Change-Id: I3f56a3796e45d608c7a5d3da1f8c2d674d62f567

tests/src/test_ensure.cpp

diff --git a/tests/src/test_ensure.cpp b/tests/src/test_ensure.cpp
index d252bc3..0e300ad 100644
--- a/tests/src/test_ensure.cpp
+++ b/tests/src/test_ensure.cpp
@@ -8,44 +8,22 @@
 class TestEnsure : public Test
 {
   public:
-    utils::Ensure<std::function<void()>> makeEnsure()
-    {
-        return [this] { ++executed; };
-    }
-
     utils::Ensure<std::function<void()>> sut;
+
     size_t executed = 0u;
 };
 
 TEST_F(TestEnsure, executesCallbackOnceWhenDestroyed)
 {
-    sut = makeEnsure();
+    sut = [this] { ++executed; };
     sut = nullptr;
 
     EXPECT_THAT(executed, Eq(1u));
 }
 
-TEST_F(TestEnsure, executesCallbackOnceWhenMoved)
-{
-    sut = makeEnsure();
-    auto copy = std::move(sut);
-    copy = nullptr;
-
-    EXPECT_THAT(executed, Eq(1u));
-}
-
-TEST_F(TestEnsure, executesCallbackTwiceWhenReplaced)
-{
-    sut = makeEnsure();
-    sut = makeEnsure();
-    sut = nullptr;
-
-    EXPECT_THAT(executed, Eq(2u));
-}
-
 TEST_F(TestEnsure, executesCallbackTwiceWhenNewCallbackAssigned)
 {
-    sut = makeEnsure();
+    sut = [this] { ++executed; };
     sut = [this] { executed += 10; };
     sut = nullptr;
 

tests/src/test_report.cpp

diff --git a/tests/src/test_report.cpp b/tests/src/test_report.cpp
index 579c714..d05620b 100644
--- a/tests/src/test_report.cpp
+++ b/tests/src/test_report.cpp
@@ -85,7 +85,8 @@
 
         std::vector<MetricValue> readings{{MetricValue{"a", "b", 17.1, 114},
                                            MetricValue{"aa", "bb", 42.0, 74}}};
-        readings.resize(metricParameters.size());
+
+        ASSERT_THAT(readings.size(), Ge(metricParameters.size()));
 
         for (size_t i = 0; i < metricParameters.size(); ++i)
         {