Skip to content
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.

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

[TestDiscovery] Enable back TXT key for Vendor ID and Product ID (VP) #25017

Open
vivien-apple opened this issue Feb 13, 2023 · 2 comments
Open
Labels
yaml Missing features or bugs in the YAML test harness

Comments

@vivien-apple
Copy link
Contributor

Problem

The test outcomes for this step varies depending if the device broadcast the product id or not. There is no clear mechanism to express that in YAML and the test was made optional.
Then optional supports has been removed in #25016.
This test step needs to be enable again once there is a clear mechanism to express that the result can be 2 things or when the test plan is updated to reflect different capabilities for the DUT.

@bzbarsky-apple bzbarsky-apple added the yaml Missing features or bugs in the YAML test harness label Feb 13, 2023
@bzbarsky-apple
Copy link
Contributor

The test outcomes for this step varies depending if the device broadcast the product id or not

There is no PICS for this? Seems like there should be PICS for this, and two separate test steps, with different expected values, conditioned on that PICS...

@cjandhyala

@stale
Copy link

stale bot commented Aug 13, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the stale Stale issue or PR label Aug 13, 2023
@bzbarsky-apple bzbarsky-apple removed the stale Stale issue or PR label Aug 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
yaml Missing features or bugs in the YAML test harness
Projects
None yet
Development

No branches or pull requests

2 participants