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

Zephyr native tests #551

Merged
merged 2 commits into from
Feb 12, 2025
Merged

Conversation

adigie
Copy link
Member

@adigie adigie commented Feb 11, 2025

Run Zephyr native tests in CI.

Following line configures workflow to use NCS from specified PR:
NCS PR: nrfconnect/sdk-nrf/pull/20287

Copy link
Collaborator

@LuDuda LuDuda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

Btw. Not sure.. but could we squash noups to single commit for maintenance reasons?

@adigie
Copy link
Member Author

adigie commented Feb 11, 2025

Btw. Not sure.. but could we squash noups to single commit for maintenance reasons?

Sure.

adigie and others added 2 commits February 11, 2025 14:10
Enable examples-nrfconnect workflow.

Signed-off-by: Adrian Gielniewski <[email protected]>

[nrf noup] Use image with `act` fix

Use image based on NCS 2.9.0 with `act` cache fix.

Signed-off-by: Adrian Gielniewski <[email protected]>

[nrf noup] Disable some steps in examples-nrfconnect

Disable size reports and examples build.

Signed-off-by: Adrian Gielniewski <[email protected]>

[nrf noup] Update NCS to recommended version

Update NCS version instead of just checking it.

Signed-off-by: Adrian Gielniewski <[email protected]>

[nrf noup] Update NCS to PR revision

Signed-off-by: Adrian Gielniewski <[email protected]>

[nrf noup] Temporary fix for link issue

Fix duplicated arguments during linking.

Signed-off-by: Adrian Gielniewski <[email protected]>
* [nrfconnect] Workaround for Zephyr native link issue

Add temporary workaround for Zephyr native link issue.
This will be fixed with the next NCS release.

Signed-off-by: Adrian Gielniewski <[email protected]>

* [nrfconnect] Fix PSA configuration in tests

Signed-off-by: Adrian Gielniewski <[email protected]>

* Add missing PSA initialization in some tests

Signed-off-by: Adrian Gielniewski <[email protected]>

* Fix hmac key handle validation when PSA is used

Signed-off-by: Adrian Gielniewski <[email protected]>

* Fix heap memory check in TestChipCryptoPAL

Wipe all key slots during each test case SetUp to avoid reusing slot
previously used by other tests. This affects heap memory usage
calculation as when the slot is reused it frees the memory.

Signed-off-by: Adrian Gielniewski <[email protected]>

* Fix TestEventLoopHandler for Zephyr native

Expect correct sequence for different LayerImplSelect::Signal
implementations. When CHIP_SYSTEM_CONFIG_POSIX_LOCKING is not defined,
WakeEvent::Notify is called and loop has to spin one more time to
process the event.

Signed-off-by: Adrian Gielniewski <[email protected]>

---------

Signed-off-by: Adrian Gielniewski <[email protected]>
Signed-off-by: Adrian Gielniewski <[email protected]>
(cherry picked from commit 7786bb639fe8e8778e82668dfbac26698477015e)
@adigie adigie force-pushed the zephyr-native-tests branch from 66ea406 to 0625976 Compare February 11, 2025 13:11
@maciejbaczmanski maciejbaczmanski merged commit 9dc1fc2 into nrfconnect:master Feb 12, 2025
10 checks passed
@adigie adigie deleted the zephyr-native-tests branch February 12, 2025 07:36
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.

3 participants