Adjustment needed for performance tests in wrt-commons

The change "Add summary view to all security tests" colids with DPL
performance test in wrt-commons (commit: Add RUNNER_PERF_TEST_BEGIN()
and RUNNER_PERF_TEST_END() macros for performance tests). Without
this patch a build-break in security-test will occur after applying
performance tests feature to wrt-commons repository.
This patch fix the build break - security-tests can be build with
and without performance test in wrt-commons.

Verify:
    Build wrt-commons with patch "Add RUNNER_PERF_TEST_BEGIN()
and RUNNER_PERF_TEST_END() macros for performance tests". Then build
security-tests. Run all tests - there should be no changes in test
results.

Change-Id: If49834dcdd5396434a782e936d9906035b02fd32
Signed-off-by: Janusz Kozerski <j.kozerski@samsung.com>

diff --git a/tests/common/summary_collector.cpp b/tests/common/summary_collector.cpp
index b4d69ac..229b073 100644 (file)
--- a/tests/common/summary_collector.cpp
+++ b/tests/common/summary_collector.cpp
@@ -59,6 +59,26 @@ void SummaryCollector::Finish()
 void SummaryCollector::CollectResult(const std::string& /*id*/,
                                       const std::string& /*description*/,
                                       const FailStatus::Type status,
+                                      const std::string& /*reason = ""*/,
+                                      const bool& isPerformanceTest,
+                                      const std::chrono::system_clock::duration& performanceTime,
+                                      const std::chrono::system_clock::duration& performanceMaxTime)
+{
+    (void)isPerformanceTest;
+    (void)performanceTime;
+    (void)performanceMaxTime;
+    switch (status) {
+        case FailStatus::IGNORED: ++m_ignored; break;
+        case FailStatus::INTERNAL: // internal error count as fail
+        case FailStatus::FAILED: ++m_failed; break;
+        case FailStatus::NONE: ++m_succeeded; break;
+    };
+    ++m_total;
+}
+
+void SummaryCollector::CollectResult(const std::string& /*id*/,
+                                      const std::string& /*description*/,
+                                      const FailStatus::Type status,
                                       const std::string& /*reason = ""*/)
 {
     switch (status) {

diff --git a/tests/common/summary_collector.h b/tests/common/summary_collector.h
index 3a1cdf9..3f2eb3c 100644 (file)
--- a/tests/common/summary_collector.h
+++ b/tests/common/summary_collector.h
@@ -25,6 +25,7 @@
 #define _RAW_RESULTS_COLLECTOR_H_
 
 #include <dpl/test/test_results_collector.h>
+#include <chrono>
 
 /*
  * Custom test runner results collector. The results (total test cases, failed, etc.) are
@@ -42,6 +43,15 @@ class SummaryCollector : public DPL::Test::TestResultsCollectorBase
                        const std::string& /*description*/,
                        const FailStatus::Type status,
                        const std::string& /*reason = ""*/);
+    void CollectResult(const std::string& /*id*/,
+                       const std::string& /*description*/,
+                       const FailStatus::Type status,
+                       const std::string& /*reason = ""*/,
+                       const bool& isPerformanceTest = false,
+                       const std::chrono::system_clock::duration&
+                               performanceTime = std::chrono::microseconds::zero(),
+                       const std::chrono::system_clock::duration&
+                               performanceMaxTime = std::chrono::microseconds::zero());
 
 public:
     static TestResultsCollectorBase* Constructor();