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

Fix NetworkCommissioning Cluster issues #32156

Merged
merged 6 commits into from
Feb 16, 2024

Commits on Feb 15, 2024

  1. Fix NetworkCommissioning Cluster issues

    This PR addresses several NetworkCommissioning issues
    which are all inter-related:
    
    - Attributes are never reported on change.
    - ScanNetworks used to modify attributes that should not be modified
    - LastNetworkStatus was not set where required
    - Thread network scanning did not report error on SSID field provided
    - All ConstraintErrors actually were reported as InvalidCommand
    - Lacking results of directed scanning did not report NetworkNotFound
    
    - Fixes project-chip#32024
    - Fixes project-chip#32022
    - Fixes project-chip#32021
    - Fixes project-chip#32019
    - Fixes project-chip#32018
    - Fixes project-chip#31870
    
    Testing done:
    - TC-CNET-4.4 pass on ESP32 and Linux
    - Commissioning works on ESP32 and Linux
    - Manual testing of all attribute changes validated with chip-repl
      - Automated test beyond TC-CNET-4.4 coming as a follow-up
    tcarmelveilleux committed Feb 15, 2024
    Configuration menu
    Copy the full SHA
    4fbe075 View commit details
    Browse the repository at this point in the history
  2. Restyled by clang-format

    restyled-commits authored and tcarmelveilleux committed Feb 15, 2024
    Configuration menu
    Copy the full SHA
    5775cd6 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    b0ba574 View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    42308b6 View commit details
    Browse the repository at this point in the history
  5. Apply review comments

    tcarmelveilleux committed Feb 15, 2024
    Configuration menu
    Copy the full SHA
    9f72265 View commit details
    Browse the repository at this point in the history
  6. Restyled by clang-format

    restyled-commits authored and tcarmelveilleux committed Feb 15, 2024
    Configuration menu
    Copy the full SHA
    34d7b9f View commit details
    Browse the repository at this point in the history