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.

Sign up for GitHub

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

Jump to bottom

[BUG] TC-IDM-6.2 subscribe request returns INVALID_ACTION #28981

Closed
pankore opened this issue Aug 31, 2023 · 2 comments · Fixed by #28989
Closed

[BUG] TC-IDM-6.2 subscribe request returns INVALID_ACTION #28981

pankore opened this issue Aug 31, 2023 · 2 comments · Fixed by #28989
Labels
bug Something isn't working needs triage

Comments

@pankore
Copy link
Contributor

pankore commented Aug 31, 2023

Reproduction steps

Using TE2 SHA:5b4f8004662d00bdb111367fec7d3ea978c23372 built chip-all-cluster-app and tested TC-IDM-6.2.
When executing step 2, the response shows INVALID_ACTION.

tc-idm-6.2.txt

Bug prevalence

always

GitHub hash of the SDK that was being used

5b4f800

Platform

other

Platform Version(s)

No response

Anything else?

No response
@pankore pankore added bug Something isn't working needs triage labels Aug 31, 2023
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Aug 31, 2023
@bzbarsky-apple
Fix incorrect loop condition in HasValidEventPathForEndpointAndCluster.
f10b959 
The test was backwards, so when CHIP_CONFIG_ENABLE_EVENTLIST_ATTRIBUTE was
defined for the "wildcard event" case we always ended up treating the list of
events for the cluster as empty.

Fixes project-chip#28981
@bzbarsky-apple bzbarsky-apple mentioned this issue Aug 31, 2023
Merged
@mergify mergify bot closed this as completed in #28989 Sep 1, 2023
mergify bot pushed a commit that referenced this issue Sep 1, 2023
@bzbarsky-apple
Fix incorrect loop condition in HasValidEventPathForEndpointAndCluste…
5deb59b 
…r. (#28989)

The test was backwards, so when CHIP_CONFIG_ENABLE_EVENTLIST_ATTRIBUTE was
defined for the "wildcard event" case we always ended up treating the list of
events for the cluster as empty.

Fixes #28981
abpoth pushed a commit to abpoth/connectedhomeip that referenced this issue Sep 12, 2023
@bzbarsky-apple @abpoth
Fix incorrect loop condition in HasValidEventPathForEndpointAndCluste…
88efe0d 
…r. (project-chip#28989)

The test was backwards, so when CHIP_CONFIG_ENABLE_EVENTLIST_ATTRIBUTE was
defined for the "wildcard event" case we always ended up treating the list of
events for the cluster as empty.

Fixes project-chip#28981
@pankore
Copy link
Contributor Author

pankore commented Sep 13, 2023 •
edited
Loading

@bzbarsky-apple I found that after applying the commit 5deb59b

In Step 13 and 14, I am still getting INVALID_ACTION after ACL command.
TC-IDM-6.2-step13-14.txt

I have created a new issue #29203

@bzbarsky-apple
Copy link
Contributor

@pankore Thanks! That's a bug in the test plan....

HunsupJung pushed a commit to HunsupJung/connectedhomeip that referenced this issue Oct 23, 2023
@bzbarsky-apple @HunsupJung
Fix incorrect loop condition in HasValidEventPathForEndpointAndCluste…
ec55735 
…r. (project-chip#28989)

The test was backwards, so when CHIP_CONFIG_ENABLE_EVENTLIST_ATTRIBUTE was
defined for the "wildcard event" case we always ended up treating the list of
events for the cluster as empty.

Fixes project-chip#28981
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working needs triage
Projects
None yet
Milestone
No milestone
2 participants
@bzbarsky-apple @pankore