TDK-11955: MVS framework cleanup & fixes for app loading issue#191
TDK-11955: MVS framework cleanup & fixes for app loading issue#191asuresh977 wants to merge 4 commits into
Conversation
…t M149] Reason for change: MVS Framework Updates Test Procedure: Verify with TDK TM Risks: None Signed-off-by: Aromal Suresh <aromal_suresh@comcast.com>
There was a problem hiding this comment.
Pull request overview
This PR cleans up framework/fileStore/MediaValidationUtility.py by removing a number of legacy RDKShell-based helper functions (launchPlugin, destroyPlugin, moveToFrontClient, sendKeysToClient, checkRDKShellClients, checkClientZOrder, exclude_from_zorder, checkWebkitReadyState), normalizes whitespace, fixes a few typos in comments/messages, and reworks launchApp to retry the LoadedApps check (max 2 attempts with a 3-second sleep between them) to address an app-loading reliability issue.
Changes:
- Remove legacy RDKShell helper functions and trim blank lines.
- Refactor
launchAppto pollrdkservice_get_loaded_apps()with a retry instead of a single 2-second wait. - Fix typos ("webnspect" → "webinspect", "webki webinspect" → "webinspect") and minor print formatting.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
…t M149] Reason for change: MVS Framework Updates Test Procedure: Verify with TDK TM Risks: None Signed-off-by: Aromal Suresh <aromal_suresh@comcast.com>
…t M149] Reason for change: MVS Framework Updates Test Procedure: Verify with TDK TM Risks: None Signed-off-by: Aromal Suresh <aromal_suresh@comcast.com>
|
b'## WARNING: A Blackduck scan failure has been waived A prior failure has been upvoted
|
MVS framework cleanup & fixes for app loading issue [Sprint M149]
Reason for change: MVS Framework Updates
Test Procedure: Verify with TDK TM
Risks: None