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

[Platform] [mbed] Re-enable Unit Testing Firmware for mbed #33978

Closed
Alami-Amine opened this issue Jun 18, 2024 · 1 comment
Closed

[Platform] [mbed] Re-enable Unit Testing Firmware for mbed #33978

Alami-Amine opened this issue Jun 18, 2024 · 1 comment

Comments

@Alami-Amine
Copy link
Contributor

Alami-Amine commented Jun 18, 2024

Context

  • since the PR [mbed] Disable unit test build #15966, Unit Testing for Mbed platform was disabled because the unit-testing firmware stopped fitting in the flash (all unit tests are compiled into one big binary).

  • during the migration of unit testing framework from nl_unit to pw_unit_test framework, Mbed platform was not migrated to pigweed as there is no way for testing.

  • This was discussed in:

Is there an github issues to track to re-enable this after we split tests to multiple smaller binaries? If so please include a TODO with the issue number. If not, please create one and add the issue number here

Originally posted by @tehampson in #33943 (comment)

Actions

  • After splitting unit tests to multiple smaller binaries, it should be possible re-enable unit testing for Mbed and to test that.
  • However, there is no platform maintainer for Mbed; therefore, the feasibility of this issue can be questioned.

Step to Re-enable:

  1. Add support for running pw_unit_test in the mbed Test_driver: src/test_driver/mbed/unit_tests/main/main.cpp
  2. Enable it in the Github Actions workflow:
@Alami-Amine
Copy link
Contributor Author

mbed is actually deprecated and will stop being supported in Matter, so ticket is no longer relevant

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

1 participant