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

[TC-TSTATS-4.2] - Script Ignores Configured Endpoint #373

Closed
ZainZhongUL opened this issue Sep 25, 2024 · 1 comment · Fixed by project-chip/connectedhomeip#35771
Closed

Comments

@ZainZhongUL
Copy link

ZainZhongUL commented Sep 25, 2024

After side-loading the update from PR 35607, TC-TSTATS-4.2 continues to fail. The test script appears to be ignoring the endpoint specified in the test configuration. The DUT's Thermostat cluster is located on endpoint 5, but the script is still sending test commands to endpoint 1.

We have verified that the test case passes successfully when the Thermostat cluster is located on endpoint 1.

This suggests that the script has a hardcoded endpoint value (endpoint 1), preventing it from adapting to the actual endpoint configuration of the TH.

Fail-TSTAT-4.2-UI-Python.log

@cecille
Copy link
Contributor

cecille commented Sep 25, 2024

Test uses a defined parameter for the endpoint rather than using the endpoint from the config. To use the test as-is, you would add --int-arg endpoint:5 or "int-arg": "endpoint:5" to the UI parameters.

Alternately, the test can be corrected to use the endpoint parameter a s expected, which makes more sense to me, IMO. I have a potential fix up here:project-chip/connectedhomeip#35771. However, i would need someone else to validate this as I do not have a TSTAT device with a TSTAT on a different endpoint,

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

Successfully merging a pull request may close this issue.

2 participants