PengZheng commented on PR #588: URL: https://github.com/apache/celix/pull/588#issuecomment-1632225435
The crash of `CelixFrameworkUtilsErrorInjectionTestSuite.testIsBundleUrlValid` is very interesting. It was caused by `celix_utils_strdup` failure in `celix_framework_registerServiceAsync`. Of course, `CELIX_EI_UNKNOWN_CALLER` should be avoided at the first place. This crash tells us more: an unexpected service registration together with corresponding events process could interact with tests more subtle than we thought it to be. If we remove "components.ready" for now, addming `celix_framework_waitForEmptyEventQueue` after a framework instance is created should protect us from most hazards in tests. In production, we shall never do that since it will hurt startup speed. -- 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. To unsubscribe, e-mail: dev-unsubscr...@celix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
