Skip to content

RDK-56291 - [RDKE] Increase L2 Test Coverage For Remote Debugger : Target 80% [ Phase 2 ] #147

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

RDK-56291 - [RDKE] Increase L2 Test Coverage For Remote Debugger : Target 80% [ Phase 2 ] #147

wants to merge 1 commit into from
+2,305 −28

Conversation

Abhinavpv28
Copy link
Contributor

@Abhinavpv28 Abhinavpv28 commented Aug 15, 2025 •
edited
Loading

Reason for change: Increase L2 Test Coverage For Remote Debugger
Test Procedure: Focused Regression
Risks: Low
Signed-off-by: Abhinav P V [email protected]

@Abhinavpv28 Abhinavpv28 requested review from a team as code owners August 15, 2025 10:03
@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

9 similar comments
@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@CLAassistant
Copy link

CLAassistant commented Aug 15, 2025 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

16 similar comments
@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

11 similar comments
@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@Abhinavpv28 Abhinavpv28 changed the title Feature/deepsleepl2 RDK-56291 - [RDKE] Increase L2 Test Coverage For Remote Debugger : Target 80% [ Phase 2 ] Aug 18, 2025
venkat0557
venkat0557 reviewed Aug 19, 2025
View reviewed changes
remote_debugger.json
@@ -39,5 +39,30 @@
"Commands": "systemctl list-units --type=service --all",
"Timeout" : 10
}
}
},
"DeepSleep": {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The DeepSleep config is added only for L2? Does it stay in the device root FS even if it's not required? Also need to check any security concerns.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This file wont be available in rootfs. this file is only for test purpose. the real json file installations are done in meta layer.

@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

@Abhinavpv28
Create Makefile
b345239 
Create deepsleep_main.c

Update Makefile

Create power_controller.h

Update run_l2.sh

Update cov_build.sh

Create pwrMgr.h

Update cov_build.sh

Update cov_build.sh

Update Makefile

Rename deepsleep_main.c to main.c

Update Makefile

update Makefile

Delete test/functional-tests/tests/rrdIarmEvents.c

Update run_l2.sh

Update remote_debugger.json

Update remote_debugger.json

Create test_deepsleep_static.py

Update run_l2.sh

Update run_l2.sh

Create test_deepsleep_dynamic.py

Update run_l2.sh

Update rrdInterface.c

Update create_json.sh

Create test_append.py

Create test_category.py

Update test_rrd_dynamic_profile_missing_report.py

Create test_rrd_dynamic_with_download_harmful.py

Create test_rrd_negative.py

Update configure.ac

Update cov_build.sh

Update run_l2.sh

Update main.c

Rename main.c to deepsleep_main.c

Update Makefile

Create rrd_append_report.feature

Create rrd_deepsleep_static.feature

Create rrd_deepsleep_dynamic.feature

Create rrd_dynamic_with_download_harmful_report.feature

Update rrd_dynamic_profile_missing_report.feature

Update test_rrd_dynamic_profile_missing_report.py

Update cov_build.sh
@github-actions GitHub Actions
Copy link

Code Coverage Summary

                               Total:|86.6%   3800|98.1%  1079|    -      0

shibu-kv
shibu-kv requested changes Aug 27, 2025
View reviewed changes
Copy link
Contributor

@shibu-kv shibu-kv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please make sure that the L2 tests executes successfully and reliably .
CI worklow shows that there are test failures.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@venkat0557 venkat0557 venkat0557 left review comments

@shibu-kv shibu-kv shibu-kv requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Abhinavpv28 @CLAassistant @venkat0557 @shibu-kv