Remove description param from test result collecting function

Change-Id: I75f757abd6350456cbf572833b68a7a4aa8647e5

diff --git a/tests/framework/include/dpl/test/test_results_collector.h b/tests/framework/include/dpl/test/test_results_collector.h
index ac568e4..cfd9227 100644 (file)
--- a/tests/framework/include/dpl/test/test_results_collector.h
+++ b/tests/framework/include/dpl/test/test_results_collector.h
@@ -63,7 +63,6 @@ class TestResultsCollectorBase :
 
     virtual void CollectTestsCasesList(const TestCaseIdList& /*list*/) {}
     virtual void CollectResult(const std::string& id,
-                               const std::string& description,
                                const FailStatus status = FailStatus::NONE,
                                const std::string& reason = "",
                                const bool& isPerformanceTest = false,

diff --git a/tests/framework/include/dpl/test/test_results_collector_console.h b/tests/framework/include/dpl/test/test_results_collector_console.h
index 06d9467..332763b 100644 (file)
--- a/tests/framework/include/dpl/test/test_results_collector_console.h
+++ b/tests/framework/include/dpl/test/test_results_collector_console.h
@@ -42,7 +42,6 @@ private:
 
     virtual void CollectCurrentTestGroupName(const std::string& name);
     virtual void CollectResult(const std::string& id,
-                               const std::string& /*description*/,
                                const FailStatus status = FailStatus::NONE,
                                const std::string& reason = "",
                                const bool& isPerformanceTest = true,

diff --git a/tests/framework/include/dpl/test/test_results_collector_html.h b/tests/framework/include/dpl/test/test_results_collector_html.h
index 4a7b764..90d635e 100644 (file)
--- a/tests/framework/include/dpl/test/test_results_collector_html.h
+++ b/tests/framework/include/dpl/test/test_results_collector_html.h
@@ -47,7 +47,6 @@ private:
     virtual void Start();
     virtual void CollectCurrentTestGroupName(const std::string& name);
     virtual void CollectResult(const std::string& id,
-                               const std::string& /*description*/,
                                const FailStatus status = FailStatus::NONE,
                                const std::string& reason = "",
                                const bool& isPerformanceTest = false,

diff --git a/tests/framework/include/dpl/test/test_results_collector_summary.h b/tests/framework/include/dpl/test/test_results_collector_summary.h
index 35e7ba9..f0edfcc 100644 (file)
--- a/tests/framework/include/dpl/test/test_results_collector_summary.h
+++ b/tests/framework/include/dpl/test/test_results_collector_summary.h
@@ -46,7 +46,6 @@ private:
     virtual bool ParseCollectorSpecificArg(const std::string& arg);
     virtual void Start();
     virtual void CollectResult(const std::string& id,
-                               const std::string& description,
                                const FailStatus status = FailStatus::NONE,
                                const std::string& reason = "",
                                const bool& isPerformanceTest = false,

diff --git a/tests/framework/include/dpl/test/test_results_collector_xml.h b/tests/framework/include/dpl/test/test_results_collector_xml.h
index 422af1c..01b9802 100644 (file)
--- a/tests/framework/include/dpl/test/test_results_collector_xml.h
+++ b/tests/framework/include/dpl/test/test_results_collector_xml.h
@@ -45,7 +45,6 @@ private:
     virtual void Start();
     virtual void CollectCurrentTestGroupName(const std::string& name);
     virtual void CollectResult(const std::string& id,
-                               const std::string& /*description*/,
                                const FailStatus status = FailStatus::NONE,
                                const std::string& reason = "",
                                const bool& isPerformanceTest = false,

diff --git a/tests/framework/include/dpl/test/test_runner.h b/tests/framework/include/dpl/test/test_runner.h
index 4b41cfb..a2694de 100644 (file)
--- a/tests/framework/include/dpl/test/test_runner.h
+++ b/tests/framework/include/dpl/test/test_runner.h
@@ -136,7 +136,6 @@ class TestRunner
     void RunTests();
 
     void CollectResult(const std::string& id,
-                       const std::string& description,
                        const TestResultsCollectorBase::FailStatus status
                            = TestResultsCollectorBase::FailStatus::NONE,
                        const std::string& reason = std::string(),

diff --git a/tests/framework/src/test_results_collector_console.cpp b/tests/framework/src/test_results_collector_console.cpp
index 6c92db5..09030a8 100644 (file)
--- a/tests/framework/src/test_results_collector_console.cpp
+++ b/tests/framework/src/test_results_collector_console.cpp
@@ -60,7 +60,6 @@ void ConsoleCollector::Finish()
 }
 
 void ConsoleCollector::CollectResult(const std::string& id,
-                                     const std::string& /*description*/,
                                      const FailStatus status,
                                      const std::string& reason,
                                      const bool& isPerformanceTest,

diff --git a/tests/framework/src/test_results_collector_html.cpp b/tests/framework/src/test_results_collector_html.cpp
index c34922a..93698cc 100644 (file)
--- a/tests/framework/src/test_results_collector_html.cpp
+++ b/tests/framework/src/test_results_collector_html.cpp
@@ -106,7 +106,6 @@ bool HtmlCollector::ParseCollectorSpecificArg(const std::string& arg)
 }
 
 void HtmlCollector::CollectResult(const std::string& id,
-                                  const std::string& /*description*/,
                                   const FailStatus status,
                                   const std::string& reason,
                                   const bool& isPerformanceTest,

diff --git a/tests/framework/src/test_results_collector_summary.cpp b/tests/framework/src/test_results_collector_summary.cpp
index 05b9cc7..74cb680 100644 (file)
--- a/tests/framework/src/test_results_collector_summary.cpp
+++ b/tests/framework/src/test_results_collector_summary.cpp
@@ -63,7 +63,6 @@ void SummaryCollector::Start()
 }
 
 void SummaryCollector::CollectResult(const std::string& id,
-                                     const std::string& description,
                                      const FailStatus status,
                                      const std::string& reason,
                                      const bool& isPerformanceTest,
@@ -71,7 +70,6 @@ void SummaryCollector::CollectResult(const std::string& id,
                                      const std::chrono::system_clock::duration& performanceMaxTime)
 {
     DPL_UNUSED_PARAM(id);
-    DPL_UNUSED_PARAM(description);
     DPL_UNUSED_PARAM(reason);
     DPL_UNUSED_PARAM(isPerformanceTest);
     DPL_UNUSED_PARAM(performanceTime);

diff --git a/tests/framework/src/test_results_collector_xml.cpp b/tests/framework/src/test_results_collector_xml.cpp
index f65d6fd..7eea2b7 100644 (file)
--- a/tests/framework/src/test_results_collector_xml.cpp
+++ b/tests/framework/src/test_results_collector_xml.cpp
@@ -126,7 +126,6 @@ bool XmlCollector::ParseCollectorSpecificArg(const std::string& arg)
 }
 
 void XmlCollector::CollectResult(const std::string& id,
-                                 const std::string&,
                                  const FailStatus status,
                                  const std::string& reason,
                                  const bool& isPerformanceTest,

diff --git a/tests/framework/src/test_runner.cpp b/tests/framework/src/test_runner.cpp
index 7818ab2..aece8e4 100644 (file)
--- a/tests/framework/src/test_runner.cpp
+++ b/tests/framework/src/test_runner.cpp
@@ -251,7 +251,6 @@ TestRunner::Status TestRunner::RunTestCase(const TestCaseStruct& testCase)
     } catch (const TestFailed &e) {
         // Simple test failure
         CollectResult(testCase.name,
-                      "",
                       TestResultsCollectorBase::FailStatus::FAILED,
                       e.GetMessage());
 
@@ -261,7 +260,6 @@ TestRunner::Status TestRunner::RunTestCase(const TestCaseStruct& testCase)
         if (m_runIgnored) {
             // Simple test have to be implemented
             CollectResult(testCase.name,
-                          "",
                           TestResultsCollectorBase::FailStatus::IGNORED,
                           e.GetMessage());
         }
@@ -271,7 +269,6 @@ TestRunner::Status TestRunner::RunTestCase(const TestCaseStruct& testCase)
     } catch (const std::exception &) {
         // std exception failure
         CollectResult(testCase.name,
-                      "",
                       TestResultsCollectorBase::FailStatus::FAILED,
                       "std exception");
 
@@ -280,7 +277,6 @@ TestRunner::Status TestRunner::RunTestCase(const TestCaseStruct& testCase)
     } catch (...) {
         // Unknown exception failure
         CollectResult(testCase.name,
-                      "",
                       TestResultsCollectorBase::FailStatus::FAILED,
                       "unknown exception");
 
@@ -289,7 +285,6 @@ TestRunner::Status TestRunner::RunTestCase(const TestCaseStruct& testCase)
     }
 
     CollectResult(testCase.name,
-                  "",
                   TestResultsCollectorBase::FailStatus::NONE,
                   "",
                   testCase.m_isPerformanceTest,
@@ -420,7 +415,6 @@ void TestRunner::setCurrentTestCasePerformanceResult(bool isPerformanceTest,
 
 void TestRunner::CollectResult(
     const std::string& id,
-    const std::string& description,
     const TestResultsCollectorBase::FailStatus status,
     const std::string& reason,
     const bool& isPerformanceTest,
@@ -432,7 +426,6 @@ void TestRunner::CollectResult(
                   [&](const TestResultsCollectors::value_type & collector)
                   {
                       collector.second->CollectResult(id,
-                                                      description,
                                                       status,
                                                       reason,
                                                       isPerformanceTest,