commit 77c481c40c825a7ae161481dadee3e89a245f213
author Ben Zhang <benz@fb.com> Wed Jun 07 13:22:00 2017 -0700
committer Facebook Github Bot <facebook-github-bot@users.noreply.github.com> Wed Jun 07 13:33:08 2017 -0700
tree 36bf5218d4bfea8a404afc1e1f0724dc174be0c3
parent 98825d1323bdc8732398cd9c18441949f3702e37

Fixed flaky observerTest.TestNotifyAfterDetach

Summary: Not resetting counter matters when running binary directly.

Reviewed By: bwasti

Differential Revision: D5202723

fbshipit-source-id: 7cc5c9e4d5c6db0f79fa3950454556bc26ea4914

caffe2/core/observer_test.cc

diff --git a/caffe2/core/observer_test.cc b/caffe2/core/observer_test.cc
index 44f4ce4..7a086b2 100644
--- a/caffe2/core/observer_test.cc
+++ b/caffe2/core/observer_test.cc
@@ -126,6 +126,7 @@
       make_unique<DummyObserver<SimpleNet>>(
           *(caffe2::dynamic_cast_if_rtti<SimpleNet*>(net.get())));
   net_ob.get()->Deactivate();
+  counter = 0;
   net.get()->Run();
   EXPECT_EQ(0, counter.load());
 }