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

chip-tool : Currently in the YAML scripts the PICS checks in each step are case-sensitive , can this be made case-insensitive please? #21425

Closed
raju-apple opened this issue Jul 29, 2022 · 1 comment · Fixed by #21432

Comments

@raju-apple
Copy link
Contributor

raju-apple commented Jul 29, 2022

This issue was a major pain-point in SVE1 from a lot of participants

I'll explain with an example here :
Currently in YAML scripts , if a step is something like this :

    - label: "DUT executes a blink effect"
      cluster: "LogCommands"
      command: "UserPrompt"
      PICS: PICS_USER_PROMPT && I.S.C40.Rsp
      arguments:
          values:
              - name: "message"
                value: "Please enter 'y' for success"
              - name: "expectedValue"
                value: "y"

The PICS value I.S.C40.Rsp is case sensitive here , if the PICS tool or the test plan or spec changes it due to any reason or a manual error to either I.S.c40.Rsp or I.S.C40.RSP , the test case wouldn't execute as desired. We are trying to get all the scripts in sync here but would want the chip-tool to make any PICS values to be case - insensitive so that we don't run into these issues.

@woody-apple
Copy link
Contributor

Cert Blocker Review: Marking this as 'needs provisional' unless we have a resolution to this issue today. Possible this is an error in the test case as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants