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

[SVE] TC-TCCM-1.1 unable to detect TCCM.S.A0003 #29243

Open
pankore opened this issue Sep 14, 2023 · 3 comments
Open

[SVE] TC-TCCM-1.1 unable to detect TCCM.S.A0003 #29243

pankore opened this issue Sep 14, 2023 · 3 comments
Labels

Comments

@pankore
Copy link
Contributor

pankore commented Sep 14, 2023

Reproduction steps

When disable TCCM.S.A0003, Step 4b reading atrributelist failed
CHIP:TOO AttributeList: 8 entries
CHIP:TOO [1]: 0
CHIP:TOO [2]: 1
CHIP:TOO [3]: 2
CHIP:TOO [4]: 65528
CHIP:TOO [5]: 65529
CHIP:TOO [6]: 65531
CHIP:TOO [7]: 65532
CHIP:TOO [8]: 65533
Test Failure: The response ([0, 1, 2, 65528, 65529, 65531, 65532, 65533]) is missing [3].

Test logs:
UI_Test_Run_2023_09_14_12_52_25.log

Bug prevalence

always

GitHub hash of the SDK that was being used

5ccb774

Platform

ameba

Platform Version(s)

No response

Type

Test Improvement

Testing

YAML tested

(Optional) If manually tested please explain why this is only manually tested

No response

Anything else?

No response

@pankore
Copy link
Contributor Author

pankore commented Sep 19, 2023

@bzbarsky-apple any comments on this issue?
In my case, TCCM.S.F00 and TCCM.S.A0003 is disabled but the TH is still asking for A0003 to be included

@bzbarsky-apple
Copy link
Contributor

I don't know that there is much to comment on. The test plan is correct. The YAML is clearly wrong. For the PICS_EVENT_LIST_ENABLED case it has:

              contains: [0, 1, 65528, 65529, 65530, 65531, 65532, 65533]

and for the case when EventList is disabled it has:

              contains: [0, 1, 2, 3, 65528, 65529, 65531, 65532, 65533]

which should not have the "2" and "3" in there. @manjunath-grl

@manjunath-grl
Copy link
Contributor

@pankore fixed in PR 29179 please verify it and close the ticket.

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

No branches or pull requests

3 participants