IVGCVSW-3721 Add support for startup sequence (Mock Gatord service).
authorColm Donelan <Colm.Donelan@arm.com>
Fri, 18 Oct 2019 15:49:28 +0000 (16:49 +0100)
committerColm Donelan <Colm.Donelan@arm.com>
Fri, 18 Oct 2019 15:53:36 +0000 (16:53 +0100)
Fixing logic error in PeriodicCounterCaptureCommandHandler. Default
quiet operation should be false and the unit tests should set it to
true.

Signed-off-by: Colm Donelan <Colm.Donelan@arm.com>
Change-Id: Icabf30819985811e42168a98909b44a6c5f5d4df

tests/profiling/gatordmock/PeriodicCounterCaptureCommandHandler.hpp
tests/profiling/gatordmock/tests/GatordMockTests.cpp

index 05539a4..4135a2f 100644 (file)
@@ -37,7 +37,7 @@ public:
     PeriodicCounterCaptureCommandHandler(uint32_t familyId,
                                          uint32_t packetId,
                                          uint32_t version,
-                                         bool quietOperation = true)
+                                         bool quietOperation = false)
         : CommandHandlerFunctor(familyId, packetId, version)
         , m_QuietOperation(quietOperation)
     {}
index eb4b178..f5e60f8 100644 (file)
@@ -103,7 +103,7 @@ BOOST_AUTO_TEST_CASE(CounterCaptureHandlingTest)
     profiling::Packet packet2(headerWord1, dataLength, uniqueData2);
 
     uint32_t version = 1;
-    gatordmock::PeriodicCounterCaptureCommandHandler commandHandler(0, 4, version, false);
+    gatordmock::PeriodicCounterCaptureCommandHandler commandHandler(0, 4, version, true);
 
     // Simulate two separate packets coming in to calculate period
     commandHandler(packet1);
@@ -129,7 +129,7 @@ BOOST_AUTO_TEST_CASE(GatorDMockEndToEnd)
     profiling::CommandHandlerRegistry registry;
 
     // Update with derived functors
-    gatordmock::PeriodicCounterCaptureCommandHandler counterCaptureCommandHandler(0, 4, version, false);
+    gatordmock::PeriodicCounterCaptureCommandHandler counterCaptureCommandHandler(0, 4, version, true);
 
     // Register different derived functors
     registry.RegisterFunctor(&counterCaptureCommandHandler);