[tizen_app_control] Update integration tests#1068
Conversation
- Relax the 'Can find matching applications' assertion from isNotEmpty to greaterThanOrEqualTo(0): app_control_foreach_app_matched returns APP_CONTROL_ERROR_NONE with an empty list when no apps match, so an empty result is not an error. - Add 'Can send and receive request with uri and mime': verifies that uri and mime fields are correctly transmitted and received (the existing test only covered the null case). - Strengthen the String list extra data assertion in 'Omit invalid extra data' to check exact values instead of just isNotEmpty. Validated on RPI4 and TV emulator (T-samsung-10.0-x86_64): 7 tests, all pass. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the integration tests for tizen_app_control by relaxing the matched app IDs assertion, adding a test for URI and MIME round-trips, and verifying exact string list extra data. Feedback highlights a tautological assertion on list length that should be replaced with a type check, and a potential race condition in the new test where the broadcast stream should be listened to before triggering the launch request.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
- Replace the tautological `matches.length >= 0` check with `isA<List<String>>()` so the assertion actually verifies the result. - Subscribe to `onAppControl` before calling `sendLaunchRequest()` to avoid a potential race where broadcast-stream events could be missed; applied to the existing send/receive tests as well for consistency.
1 participant
Validated on RPI4 and TV emulator (T-samsung-10.0-x86_64): 7 tests, all pass.