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

[Test-TC-DIASW-1.1] ThreadMetrics attribute seems to be missing from the Software Diagnostics cluster #11185

Closed
kvikrambhat opened this issue Oct 29, 2021 · 4 comments · Fixed by #14806
Closed

[Test-TC-DIASW-1.1] ThreadMetrics attribute seems to be missing from the Software Diagnostics cluster #11185

kvikrambhat opened this issue Oct 29, 2021 · 4 comments · Fixed by #14806
Labels
cert V1.0

Comments

@kvikrambhat
Copy link
Contributor

Problem

The optional attribute ThreadMetrics seems to be missing from the Software Diagnostics cluster.

Spec Ref
SoftwareDiagnosticsAttributes

YAML Test Script Error
TC-DIAGSW-1 1
@kvikrambhat kvikrambhat mentioned this issue Oct 29, 2021
Closed
@holbrookt
Copy link
Contributor

@kvikrambhat is this still an issue?

@kowsisoundhar12
Copy link
Contributor

@holbrookt ThreadMetrics attribute added in yaml
2022-02-01_13-16

@holbrookt
Copy link
Contributor

Can this issue be closed now then?

@bzbarsky-apple
Copy link
Contributor

This works fine, but the test is trying to read it from the wrong endpoint, so failing... Fix coming up.

bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Feb 5, 2022
@bzbarsky-apple
Add correct checking for whether a command is supported.
d7717cc 
This also fixes the status reporting for unsupported endpoints and
clusters.  And fixes some tests to use the right endpoint, now that we
are checking that sort of thing.

Fixes project-chip#7612
Fixes project-chip#13920
Fixes project-chip#14022
Fixes project-chip#11578
Fixes project-chip#11185
@bzbarsky-apple bzbarsky-apple mentioned this issue Feb 5, 2022
Merged
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Feb 5, 2022
@bzbarsky-apple
Add correct checking for whether a command is supported.
df971f0 
This also fixes the status reporting for unsupported endpoints and
clusters.  And fixes some tests to use the right endpoint, now that we
are checking that sort of thing.

Fixes project-chip#7612
Fixes project-chip#13920
Fixes project-chip#14022
Fixes project-chip#11578
Fixes project-chip#11185
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Feb 5, 2022
@bzbarsky-apple
Add correct checking for whether a command is supported.
6eaf69a 
This also fixes the status reporting for unsupported endpoints and
clusters.  And fixes some tests to use the right endpoint, now that we
are checking that sort of thing.

Fixes project-chip#7612
Fixes project-chip#13920
Fixes project-chip#14022
Fixes project-chip#11578
Fixes project-chip#11185
bzbarsky-apple added a commit to bzbarsky-apple/connectedhomeip that referenced this issue Feb 5, 2022
@bzbarsky-apple
Add correct checking for whether a command is supported.
606d9d2 
This also fixes the status reporting for unsupported endpoints and
clusters.  And fixes some tests to use the right endpoint, now that we
are checking that sort of thing.

Fixes project-chip#7612
Fixes project-chip#13920
Fixes project-chip#14022
Fixes project-chip#11578
Fixes project-chip#11185
bzbarsky-apple added a commit that referenced this issue Feb 7, 2022
@bzbarsky-apple
Add correct checking for whether a command is supported. (#14806)
26fa3fa 
This also fixes the status reporting for unsupported endpoints and
clusters.  And fixes some tests to use the right endpoint, now that we
are checking that sort of thing.

Fixes #7612
Fixes #13920
Fixes #14022
Fixes #11578
Fixes #11185
mkardous-silabs pushed a commit to mkardous-silabs/connectedhomeip that referenced this issue Feb 7, 2022
@bzbarsky-apple @mkardous-silabs
Add correct checking for whether a command is supported. (project-chi…
9684468 
…p#14806)

This also fixes the status reporting for unsupported endpoints and
clusters.  And fixes some tests to use the right endpoint, now that we
are checking that sort of thing.

Fixes project-chip#7612
Fixes project-chip#13920
Fixes project-chip#14022
Fixes project-chip#11578
Fixes project-chip#11185
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
cert V1.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add correct checking for whether a command is supported. bzbarsky-apple/connectedhomeip
6 participants
@andy31415 @kvikrambhat @holbrookt @woody-apple @kowsisoundhar12 @bzbarsky-apple