-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve cellular unittests #11495
Improve cellular unittests #11495
Conversation
Also added checks for pointer validity.
As shutdown was overridden in test class, original shutdown() method was not called at all.
Pushed fix for astyle failures. |
CI started |
Test run: FAILEDSummary: 1 of 4 test jobs failed Failed test jobs:
|
@0xc0170 Build failure does not seem to relate to this change. Some CI problem? |
@kivaisan you are right. Problem is not related to this |
CI restarted |
Test run: FAILEDSummary: 1 of 4 test jobs failed Failed test jobs:
|
CI restarted (again internal error) |
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
Description
Improved UT test coverage of cellular components.
No functional changes to actual code except some pointer validity checks on some methods. Also clarifiied documentation of
CellularNetwork::is_active_context
method.Pull request type
Reviewers
@OPpuolitaival @AnttiKauppila
Release Notes