pnoltes commented on a change in pull request #332:
URL: https://github.com/apache/celix/pull/332#discussion_r601151012
##########
File path: libs/framework/gtest/src/DependencyManagerTestSuite.cc
##########
@@ -638,4 +639,12 @@ TEST_F(DependencyManagerTestSuite, ExceptionsInLifecycle) {
//required service -> should stop, but fails at stop and should become
inactive (component will disable itself)
dm.clear(); //dm clear will deinit component and this should fail, but
not deadlock
}
+}
+
+TEST_F(DependencyManagerTestSuite, ComponentContext) {
Review comment:
same
##########
File path: libs/framework/gtest/src/CxxBundleContextTestSuite.cc
##########
@@ -561,3 +565,21 @@ TEST_F(CxxBundleContextTestSuite,
setServicesWithTrackerWhenMultipleRegistration
//Also look into why this is not happening in the C service tracker test.
EXPECT_EQ(3, count.load());
}
+
+TEST_F(CxxBundleContextTestSuite, WaitForAllEvents) {
+ long svcId = ctx->findService<TestInterface>();
+ EXPECT_EQ(svcId, -1L);
+
+ {
+ auto impl = std::make_shared<TestImplementation>();
+ auto svcReg = ctx->registerService<TestInterface>(impl).build();
+ svcReg->wait();
+ svcId = ctx->findService<TestInterface>();
+ EXPECT_GE(svcId, 0L);
+ }
+
+ ctx->waitForAllEvents();
+ ctx->waitIfAbleForAllEvents();
Review comment:
same
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
