Skip to content

Commit

Permalink
all-clusters-app: Disable OTA Provider Server (#13454)
Browse files Browse the repository at this point in the history
* all-clusters-app: Disable OTA Provider Server

Having same cluster's client and server in same app is not supported in
the zap. Removed OTA provider server from all-clusters-app

* Remove OTA Provider server test cases

* Remove OTA Provider files from all-clusters-app/mbed CMakeLists.txt

* Disable OtaSoftwareUpdateProvider cluster tests for Darwin
  • Loading branch information
shubhamdp authored and pull[bot] committed Dec 5, 2023
1 parent 769d6db commit 1536571
Show file tree
Hide file tree
Showing 11 changed files with 111 additions and 233 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1408,7 +1408,7 @@
"mfgCode": null,
"define": "OTA_PROVIDER_CLUSTER",
"side": "server",
"enabled": 1,
"enabled": 0,
"commands": [
{
"name": "QueryImageResponse",
Expand Down
1 change: 0 additions & 1 deletion examples/all-clusters-app/esp32/main/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,6 @@ set(SRC_DIRS_LIST
"${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/src/app/clusters/keypad-input-server"
"${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/src/app/clusters/media-playback-server"
"${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/src/app/clusters/ota-requestor"
"${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/src/app/clusters/ota-provider"
"${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/src/app/clusters/target-navigator-server"
"${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/src/app/clusters/thermostat-server"
"${CMAKE_SOURCE_DIR}/third_party/connectedhomeip/src/app/clusters/thermostat-user-interface-configuration-server"
Expand Down
2 changes: 0 additions & 2 deletions examples/all-clusters-app/mbed/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@ target_include_directories(${APP_TARGET} PRIVATE
${CHIP_ROOT}/src/app
${CHIP_ROOT}/third_party/nlio/repo/include
${GEN_DIR}/all-clusters-app
${GEN_DIR}/ota-provider-app
${GEN_DIR}/app-common
)

Expand Down Expand Up @@ -118,7 +117,6 @@ target_sources(${APP_TARGET} PRIVATE
${APP_CLUSTERS}/network-commissioning-old/network-commissioning-ember.cpp
${APP_CLUSTERS}/network-commissioning-old/network-commissioning-old.cpp
${APP_CLUSTERS}/on-off-server/on-off-server.cpp
${APP_CLUSTERS}/ota-provider/ota-provider.cpp
${APP_CLUSTERS}/scenes/scenes.cpp
${APP_CLUSTERS}/switch-server/switch-server.cpp
${APP_CLUSTERS}/target-navigator-server/target-navigator-server.cpp
Expand Down
1 change: 0 additions & 1 deletion src/app/tests/suites/TestDescriptorCluster.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,6 @@ tests:
0x001E, # Binding
0x001F, # Access Control
0x0028, # Basic Information
0x0029, # OTA Software Update Provider
0x002A, # OTA Software Update Requestor
0x002B, # Localization Configuration
0x002E, # Power Source Configuration
Expand Down
2 changes: 2 additions & 0 deletions src/darwin/Framework/CHIP/templates/clusters-tests.zapt
Original file line number Diff line number Diff line change
Expand Up @@ -210,6 +210,7 @@ CHIPDevice * GetConnectedDevice(void)
{{#unless (isStrEqual "Access Control" name)}}
{{#unless (isStrEqual "Basic" name)}}
{{#unless (isStrEqual "Thermostat" name)}}
{{#unless (isStrEqual "OTA Software Update Provider" name)}}
{{#chip_server_cluster_attributes}}
{{#unless (isStrEqual chipCallback.name "Unsupported")}}
- (void)testSendCluster{{asUpperCamelCase parent.name}}ReadAttribute{{asUpperCamelCase name}}WithCompletionHandler
Expand Down Expand Up @@ -266,6 +267,7 @@ CHIPDevice * GetConnectedDevice(void)
{{/unless}}
{{/unless}}
{{/unless}}
{{/unless}}
{{/chip_client_clusters}}

@end
87 changes: 18 additions & 69 deletions src/darwin/Framework/CHIPTests/CHIPClustersTests.m

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 1536571

Please sign in to comment.