commit 398dd19f66f66bea150c78425f96122027180aab
author Yao Chen <yaochen@google.com> Mon Oct 01 14:49:03 2018 -0700
committer Yao Chen <yaochen@google.com> Mon Oct 01 14:49:03 2018 -0700
tree 14b920a5b357291f430bbf5ef314897b71e24386
parent 043a3a713de8864afb04b3fc029bfc1155c334b3

Add unit tests for ShellSubscriber and fix a bug

Test: statsd_test
Change-Id: Iaf0558ec2a2dc190bedb240da8019868266ec8f5

cmds/statsd/src/shell/ShellSubscriber.cpp

diff --git a/cmds/statsd/src/shell/ShellSubscriber.cpp b/cmds/statsd/src/shell/ShellSubscriber.cpp
index 3cd49d7..1306a46 100644
--- a/cmds/statsd/src/shell/ShellSubscriber.cpp
+++ b/cmds/statsd/src/shell/ShellSubscriber.cpp
@@ -113,12 +113,12 @@
 
     for (const auto& matcher : mPushedMatchers) {
         if (matchesSimple(*mUidMap, matcher, event)) {
+            event.ToProto(mProto);
             // First write the payload size.
             size_t bufferSize = mProto.size();
             write(mOutput, &bufferSize, sizeof(bufferSize));
 
             // Then write the payload.
-            event.ToProto(mProto);
             mProto.flush(mOutput);
             mProto.clear();
             break;
@@ -137,4 +137,4 @@
 
 }  // namespace statsd
 }  // namespace os
-}  // namespace android
\ No newline at end of file
+}  // namespace android