Skip to content

Allow loading snippets with the same package but different user IDs #996

Merged
mhaoli merged 8 commits intogoogle:masterfrom
mhaoli:change_sc_identifier_to_package_name_and_user_id
Dec 19, 2025
Merged

Allow loading snippets with the same package but different user IDs #996
mhaoli merged 8 commits intogoogle:masterfrom
mhaoli:change_sc_identifier_to_package_name_and_user_id

Conversation

@mhaoli
Copy link
Copy Markdown
Collaborator

@mhaoli mhaoli commented Dec 10, 2025

Changes:

  1. Add a property identifier to SnippetClientV2 class.
  2. Use the client identifier instead of package name for logging and dup registration check.

This PR allows loading snippets with the same package for different Android users.

Comment thread mobly/controllers/android_device_lib/snippet_client_v2.py Outdated
Comment thread mobly/controllers/android_device_lib/services/snippet_management_service.py Outdated
Comment thread mobly/controllers/android_device_lib/snippet_client_v2.py Outdated
Copy link
Copy Markdown
Collaborator

@xpconanfan xpconanfan left a comment

Choose a reason for hiding this comment

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

The title of this PR should focus on the actual functional goal, not the implementation detail

Comment thread mobly/controllers/android_device_lib/services/snippet_management_service.py Outdated
@mhaoli mhaoli changed the title Change snippet client identifier from pacakge to package+user_id Allow loading snippets with the same package but different user IDs Dec 12, 2025
@mhaoli mhaoli force-pushed the change_sc_identifier_to_package_name_and_user_id branch from b3c0828 to 99883e9 Compare December 19, 2025 06:06
@mhaoli mhaoli merged commit e6b0cc5 into google:master Dec 19, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants