-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Implement kotlin setup-payload phase II #26963
Conversation
@yufengwangca in https://github.com/project-chip/connectedhomeip/blob/master/.github/workflows/java-tests.yaml#L93 we now have the ability to run java unit tests. Would we be able to add unit tests for these classes and functionality as well? |
src/controller/java/src/chip/onboardingpayload/ManualOnboardingPayloadGenerator.kt
Outdated
Show resolved
Hide resolved
src/controller/java/src/chip/onboardingpayload/ManualOnboardingPayloadGenerator.kt
Outdated
Show resolved
Hide resolved
src/controller/java/src/chip/onboardingpayload/ManualOnboardingPayloadGenerator.kt
Outdated
Show resolved
Hide resolved
src/controller/java/src/chip/onboardingpayload/ManualOnboardingPayloadGenerator.kt
Outdated
Show resolved
Hide resolved
src/controller/java/src/chip/onboardingpayload/ManualOnboardingPayloadGenerator.kt
Outdated
Show resolved
Hide resolved
src/controller/java/src/chip/onboardingpayload/OnboardingPayload.kt
Outdated
Show resolved
Hide resolved
src/controller/java/src/chip/onboardingpayload/OnboardingPayload.kt
Outdated
Show resolved
Hide resolved
src/controller/java/src/chip/onboardingpayload/OnboardingPayload.kt
Outdated
Show resolved
Hide resolved
src/controller/java/src/chip/onboardingpayload/ManualOnboardingPayloadGenerator.kt
Outdated
Show resolved
Hide resolved
PR #26963: Size comparison from 28e0423 to 4879135 Increases (9 builds for bl602, bl702, esp32, psoc6, telink)
Decreases (10 builds for bl602, bl702, cyw30739, psoc6, telink)
Full report (58 builds for bl602, bl702, cc32xx, cyw30739, efr32, esp32, k32w, linux, mbed, nrfconnect, psoc6, qpg, telink)
|
Per sync, we plan to add unit tests after all kotlin conversion is complete so that we don't need to worry about how to link and load native lib to JUnit framework |
This is the second phase of setuppayload implementation in kotlin, which convert ManualSetupPayloadGenerator and ManualSetupPayloadParser to kotlin, the final goal is to implement setup payload completely in kotlin and deprecate the existing src/setup_payload implemented in java/c++.
Testing:
Leverage the existing CI tests using java_matter_controller, unit tests will be added later after convert all to kotlin so that we don't need to figure out how to load native lib to JUnit