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-BRBINFO-1.1 - Feature-map attribute should be implemented #20731

Closed
sowmyassp opened this issue Jul 14, 2022 · 5 comments · Fixed by #21061
Closed

TC-BRBINFO-1.1 - Feature-map attribute should be implemented #20731

sowmyassp opened this issue Jul 14, 2022 · 5 comments · Fixed by #21061

Comments

@sowmyassp
Copy link

Issue :
While Executing the behavior of the global mandatory feature-map attribute of the Bridged Basic Information cluster, throws error code as 0x86(unsupported attribute).

Commit used : 0390a8d

  • Chip-tool - RPI-4, 8GB RAM
  • DUT - RPI - RPI-4, 8GB RAM
  • Platform : linux platform
  • Dut application: bridge-app
  • Expected behavior : Pass with status(0x0)

Actual behavior : Fail with status 0x86(unsupported attribute)
image

Steps to reproduce :
./chip-tool bridgeddevicebasic read feature-map 1 3

Test-plan reference :
https://github.com/CHIP-Specifications/chip-test-plans/blob/master/src/cluster/bridged_device_basic_information.adoc#tc-brbinfo-1-1-global-attributes-for-bridged-device-basic-information-cluster-cluster-dut-server

@andy31415
Copy link
Contributor

I think we need to explain/justify cert blocker status a bit here.

Cert blocker in my mind means "Certification cannot prove that the SDK can be used to build an application that can be certified". In this case, we have feature map attributes working in other apps, so the SDK can be used to build certifiable applications.

Bridge app in particular seems not to be updated fully, so this is an issue and we may chose to do so before 1.0, however in terms of certification blocking, I do not believe this is so.

@andy31415
Copy link
Contributor

In generally I am trying to avoid code SDK developers being on the hook to develop fully certifiable applications for all possible applications (we have a lot of them already: light, lock, all-clusters, all-clusters-minimal, several tv bits, bridge, ota provider and requestor, shell, thermostat, temperature measurement, window, pump .... etc).

@cjandhyala
Copy link
Contributor

@andy31415 Cert blocker is also for something that is failing a certification test case that need fix in SDK and example app, so that CSG can verify the test ones. In this case its blocking a cert TC TC-BRBINFO-1.1.

@woody-apple
Copy link
Contributor

Cert Blocker Review: Given bridge app is used for verification, we need to make sure the feature map is correct.

@franck-apple
Copy link

@jwinder-ca is looking at this.

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.

6 participants