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

Add a Test Cluster in order to validate the generation/implementation of the Data Model #6445

Merged
merged 4 commits into from
May 4, 2021

Conversation

vivien-apple
Copy link
Contributor

Problem

Often while adding some features to ZAP or to into the Ember stack I need to set up a kind of "fake" cluster that uses what needs to be implemented.
Also, it may be useful to have a cluster written just to check that either the ZCL or the IM messages dispatching works properly.

This PR add a Test Cluster. This cluster contains a few commands only, and a list of attributes that are supported by the stack.

Summary of Changes

  • Add a Test Cluster zcl definition (first commit)
  • Add a server implementation for the Test Cluster (second commit)
  • Activate this cluster into multiples .zap files (third commit)
  • Use this Test Cluster into the Darwin CI (fourth commit)
  • Update gen/ folders (last commit)

@vivien-apple vivien-apple self-assigned this May 4, 2021
@vivien-apple vivien-apple changed the title Test add test cluster Add a Test Cluster in order to validate the generation/implementation of the Data Model May 4, 2021
@github-actions
Copy link

github-actions bot commented May 4, 2021

Size increase report for "esp32-example-build" from d66c807

File Section File VM
chip-all-clusters-app.elf .flash.rodata 640 640
chip-all-clusters-app.elf .flash.text 236 236
chip-all-clusters-app.elf .dram0.bss 0 144
Full report output
BLOAT REPORT

Files found only in the build output:
    report.csv

Comparing ./master_artifact/chip-pigweed-app.elf and ./pull_artifact/chip-pigweed-app.elf:

sections,vmsize,filesize

Comparing ./master_artifact/chip-all-clusters-app.elf and ./pull_artifact/chip-all-clusters-app.elf:

sections,vmsize,filesize
.debug_info,0,65149
.debug_line,0,25091
.debug_loc,0,20024
.debug_str,0,4606
.debug_abbrev,0,2791
.debug_frame,0,1072
.flash.rodata,640,640
.debug_ranges,0,400
.strtab,0,391
.debug_aranges,0,368
.flash.text,236,236
.dram0.bss,144,0
.symtab,0,112
.xt.prop._ZTVN4chip24LifetimePersistedCounterE,0,-1
.shstrtab,0,-3
[Unmapped],0,-640


@woody-apple
Copy link
Contributor

@mspang @saurabhst @msandstedt ?

@woody-apple woody-apple merged commit dab17fb into project-chip:master May 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants