Skip to content

tests: unit: add tests for bm_scheduler#756

Open
lemrey wants to merge 2 commits intonrfconnect:mainfrom
lemrey:event-sched-test
Open

tests: unit: add tests for bm_scheduler#756
lemrey wants to merge 2 commits intonrfconnect:mainfrom
lemrey:event-sched-test

Conversation

@lemrey
Copy link
Copy Markdown
Contributor

@lemrey lemrey commented Apr 20, 2026

Add unit tests.

@lemrey lemrey requested review from a team as code owners April 20, 2026 08:41
@github-actions github-actions Bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Apr 20, 2026
@github-actions
Copy link
Copy Markdown

You can find the documentation preview for this PR here.

@lemrey lemrey force-pushed the event-sched-test branch from ed623af to 982751e Compare April 20, 2026 08:54
Comment thread lib/bm_scheduler/bm_scheduler.c
Don't copy data if not present, avoid UB in memcpy.

Signed-off-by: Emanuele Di Santo <emdi@nordicsemi.no>
@lemrey lemrey force-pushed the event-sched-test branch from 982751e to a117e08 Compare April 21, 2026 11:41
TEST_ASSERT_EQUAL(1, call_count);
TEST_ASSERT_EQUAL(sizeof(payload), call_log[0].len);

const uint8_t expected[] = {0x55, 0x66, 0x77, 0x88};
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Move to top?

Add unit tests.

Signed-off-by: Emanuele Di Santo <emdi@nordicsemi.no>
@lemrey lemrey force-pushed the event-sched-test branch from a117e08 to 03e003d Compare April 22, 2026 07:42
@lemrey lemrey removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Apr 30, 2026
@lemrey
Copy link
Copy Markdown
Contributor Author

lemrey commented Apr 30, 2026

Adding tests, no changelog required

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants