search

apache / celix

Apache Celix is a framework for C and C++14 to develop dynamic modular software applications using component and in-process service-oriented programming.
https://celix.apache.org/
Apache License 2.0
158 stars 84 forks source link

Hotfix/Fix test_package error. #757

Closed PengZheng closed 1 week ago

PengZheng commented 1 week ago

As observed, trying to scheduling event to an inactive framework will lead to test_package failure:

celix/3.0.0 (test package): RUN: ./use_components_ready_check

Hello world from C bundle with id 2
Bundle 1: apache_celix_components_ready_check
Bundle 2: hello
Goodbye world from C bundle with id 2
[2024-06-24T09:03:00] [   info] [celix_framework] [framework_start:483] Celix framework started

ERROR: celix/3.0.0 (test package): Error in test() method, line 149
    self.run("./use_components_ready_check",
    ConanException: Error 1 while executing

This PR fixes this by only issuing a warning instead of returning error.

codecov-commenter commented 1 week ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.27%. Comparing base (39e2282) to head (f82223d).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #757 +/- ## ======================================= Coverage 90.26% 90.27% ======================================= Files 226 226 Lines 26319 26320 +1 ======================================= + Hits 23758 23760 +2 + Misses 2561 2560 -1 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.