Skip to content

[RDKE] L2 Tests And Integration With CI for Remote Debugger Dynamic Updates #98

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
Merged
merged 50 commits into from
Apr 7, 2025
Merged

[RDKE] L2 Tests And Integration With CI for Remote Debugger Dynamic Updates #98

merged 50 commits into from
Apr 7, 2025

Conversation

Abhinavpv28
Copy link
Contributor

@Abhinavpv28 Abhinavpv28 commented Mar 14, 2025 •
edited
Loading

Reason for change: L2 tests integration for remote debugger for dynamic profile
Test Procedure: Run L2 test woekflow
Risks: Low
Signed-off-by: Abhinav P V [email protected]

@Abhinavpv28 Abhinavpv28 requested review from a team as code owners March 14, 2025 10:48
@CLAassistant
Copy link

CLAassistant commented Mar 14, 2025 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@Abhinavpv28
Update run_l2.sh
04fe87b
@Abhinavpv28
Update run_l2.sh
0167ba1
@Abhinavpv28
Update run_l2.sh
868a111
@Abhinavpv28
Update helper_functions.py
7b2de0c
@Abhinavpv28
Update test_rrd_dynamic_profile_report.py
7ea9844
@Abhinavpv28
Update helper_functions.py
63a7543
@Abhinavpv28
Create test_rrd_dynamic_profile_missing_report.py
a56351c
@Abhinavpv28
Update run_l2.sh
bdd1359
@Abhinavpv28
Create create_json.sh
f4a8b18
@Abhinavpv28
Update create_json.sh
318d743
@Abhinavpv28
Update helper_functions.py
70dea17
@Abhinavpv28
Create test_rrd_dynamic_profile_harmful_report.py
eb467af
@Abhinavpv28
Update helper_functions.py
f6b946b
@Abhinavpv28
Create test_rrd_append_dynamic_profile_static_notfound.py
9b29df9
@Abhinavpv28
Merge pull request #107 from rdkcentral/develop
087effd 
Rebase
@Abhinavpv28
Update helper_functions.py
78d4035
@Abhinavpv28
Update run_l2.sh
e59f42b
@Abhinavpv28
Update run_l2.sh
89abf51
@Abhinavpv28
Update run_l2.sh
3bcd951
@Abhinavpv28
Update test_rrd_dynamic_profile_missing_report.py
1b111ab
@Abhinavpv28
Update test_rrd_dynamic_profile_missing_report.py
814029e
@Abhinavpv28
Update run_l2.sh
c3bcc93
@Abhinavpv28
Update run_l2.sh
dd3730d
@Abhinavpv28
Update test_rrd_dynamic_profile_report.py
1245e9c
@Abhinavpv28
Update test_rrd_append_dynamic_profile_static_notfound.py
d37d6ea
@nhanasi
Copy link
Contributor

nhanasi commented Apr 1, 2025

@Abhinavpv28 - please make sure we have the feature file and test file created for all the testcases and the same name is used.

@Abhinavpv28 Abhinavpv28 changed the title L2 test [RDKE] L2 Tests And Integration With CI for Remote Debugger Dynamic Updates Apr 3, 2025
nhanasi
nhanasi reviewed Apr 3, 2025
View reviewed changes
Copy link
Contributor

@nhanasi nhanasi left a comment

Choose a reason for hiding this comment

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

test/functional-tests/features/rrd_append_dynamic_profile_static_not_found.feature is not correct

nhanasi
nhanasi reviewed Apr 3, 2025
View reviewed changes
Copy link
Contributor

@nhanasi nhanasi left a comment

Choose a reason for hiding this comment

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

test/functional-tests/tests/test_rrd_dynamic_profile_missing_report.py
In this first the request need to be check in static profile and if not found only it has to go for dynamic and then if media/apps present it should use else go for download

Abhinavpv28 added 18 commits April 4, 2025 15:38
@Abhinavpv28
Create test_rrd_append_dynamic_profile_report.py
1a3e00d
@Abhinavpv28
Update run_l2.sh
42e5bb3
@Abhinavpv28
Update run_l2.sh
16f4e45
@Abhinavpv28
Update helper_functions.py
679a40c
@Abhinavpv28
Rename rrd_dynamic_profile_missing_report .feature to rrd_dynamic_pro…
cb33408 
…file_missing_report.feature
@Abhinavpv28
Delete test/functional-tests/tests/test_rrd_append_dynamic_profile_re…
1b4901c 
…port.py
@Abhinavpv28
Update run_l2.sh
33eaa67
@Abhinavpv28
Update rrd_append_dynamic_profile_static_not_found.feature
123b8b9
@Abhinavpv28
Update rrd_append_dynamic_profile_static_not_found.feature
76d5c29
@Abhinavpv28
Update rrd_append_dynamic_profile_static_not_found.feature
5500e7a
@Abhinavpv28
Update rrd_dynamic_profile_missing_report.feature
0505d9b
@Abhinavpv28
Update rrd_dynamic_profile_report.feature
101598e
@Abhinavpv28
Update test_rrd_dynamic_profile_harmful_report.feature
55a99c0
@Abhinavpv28
Update create_json.sh
b0cc22e
@Abhinavpv28
Update test_rrd_append_dynamic_profile_static_notfound.py
ca2968b
@Abhinavpv28
Update test_rrd_dynamic_profile_report.py
0cdf3b9
@Abhinavpv28
Update test_rrd_dynamic_profile_missing_report.py
0fc400f
@Abhinavpv28
Update test_rrd_dynamic_profile_harmful_report.py
ae04cbb
@nhanasi nhanasi merged commit 0a9ef31 into develop Apr 7, 2025
6 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nhanasi nhanasi nhanasi approved these changes

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

Successfully merging this pull request may close these issues.
3 participants
@Abhinavpv28 @CLAassistant @nhanasi