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

[Android][Java] Add ota test in java example app #36957

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yunhanw-google
Copy link
Contributor

-- Add Android OTA test and validate the firmware output and corresponding callback flow
-- Add CI running
-- fix incorrect manifest ordering for java-example-app

manual instruction for reference:
./scripts/build/build_examples.py --target linux-x64-java-matter-controller build
./scripts/build/build_examples.py --target linux-x64-ota-requestor build
server:
rm -rf /tmp/chip* & ./out/linux-x64-ota-requestor/chip-ota-requestor-app
client:
java -verbose:jni -Xcheck:jni -jar -Djava.library.path=out/linux-x64-java-matter-controller/lib/jni out/linux-x64-java-matter-controller/bin/java-matter-controller ota onnetwork-long 1 20202021 3840 1000

fixes: #36564 and #28709

@yunhanw-google yunhanw-google requested a review from a team as a code owner January 3, 2025 18:54
Copy link

semanticdiff-com bot commented Jan 3, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  .github/workflows/java-tests.yaml Unsupported file format
  examples/java-matter-controller/BUILD.gn Unsupported file format
  examples/java-matter-controller/Manifest.txt Unsupported file format
  examples/java-matter-controller/java/src/com/matter/controller/Main.kt Unsupported file format
  examples/java-matter-controller/java/src/com/matter/controller/commands/ota/PairOnNetworkLongOtaOverBdxCommand.kt Unsupported file format
  examples/ota-requestor-app/linux/main.cpp Unsupported file format
  scripts/tests/java/ota_test.py  0% smaller
  scripts/tests/java/test-firmware.bin Unsupported file format
  scripts/tests/java/test-firmware.ota Unsupported file format
  scripts/tests/run_java_test.py  0% smaller
  src/controller/java/CHIPDeviceController-JNI.cpp Unsupported file format

Copy link

github-actions bot commented Jan 3, 2025

PR #36957: Size comparison from 1b4c56c to 466c6a9

Full report (3 builds for cc32xx, stm32)
platform target config section 1b4c56c 466c6a9 change % change
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 540049 540049 0 0.0
RAM 205800 205800 0 0.0
lock CC3235SF_LAUNCHXL FLASH 574217 574217 0 0.0
RAM 205944 205944 0 0.0
stm32 light STM32WB5MM-DK FLASH 485080 485080 0 0.0
RAM 144912 144912 0 0.0

@yunhanw-google yunhanw-google force-pushed the feature/add_new_java_ota_test branch 2 times, most recently from f26ed94 to 068b9cb Compare January 3, 2025 19:32
Copy link

github-actions bot commented Jan 3, 2025

PR #36957: Size comparison from 1b4c56c to 068b9cb

Full report (3 builds for cc32xx, stm32)
platform target config section 1b4c56c 068b9cb change % change
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 540049 540049 0 0.0
RAM 205800 205800 0 0.0
lock CC3235SF_LAUNCHXL FLASH 574217 574217 0 0.0
RAM 205944 205944 0 0.0
stm32 light STM32WB5MM-DK FLASH 485080 485080 0 0.0
RAM 144912 144912 0 0.0

@yunhanw-google yunhanw-google force-pushed the feature/add_new_java_ota_test branch from 068b9cb to 6d55299 Compare January 3, 2025 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Test for Android OTA (or generally DynamicDispatcher)
2 participants