Skip to content

Refactor utests: replace internal functions with API and clean up#2529

Merged
michalvasko merged 5 commits into
CESNET:develfrom
HanzlikPetr:unit_test_change
Jul 13, 2026
Merged

Refactor utests: replace internal functions with API and clean up#2529
michalvasko merged 5 commits into
CESNET:develfrom
HanzlikPetr:unit_test_change

Conversation

@HanzlikPetr

Copy link
Copy Markdown

No description provided.

@HanzlikPetr HanzlikPetr changed the title Refactor utests to replace internal functions with API and clean up Refactor utests: replace internal functions with API and clean up Jun 10, 2026

@Roytak Roytak left a comment

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.

Noticed a couple of possible places for improvement, but very good overall.

Comment thread tests/utests/basic/test_common.c Outdated
Comment thread tests/utests/types/instanceid.c Outdated
Comment thread tests/utests/types/union.c Outdated
Comment thread tests/utests/utests.h Outdated
Comment thread tests/utests/utests.h Outdated
Comment thread tests/utests/schema/test_schema.c
Comment thread tests/utests/schema/test_tree_schema_compile.c
Comment thread tests/utests/types/inet_types.c
Comment thread tests/utests/data/test_tree_data_sorted.c Outdated
Comment thread tests/utests/utests.h

@michalvasko michalvasko left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please rebase the current devel changes.

Comment thread tests/utests/basic/test_context.c
@HanzlikPetr HanzlikPetr force-pushed the unit_test_change branch 3 times, most recently from 535eff7 to f4b412a Compare July 13, 2026 12:20
Comment thread src/ly_common.h Outdated
@michalvasko michalvasko merged commit e7881c3 into CESNET:devel Jul 13, 2026
37 of 39 checks passed
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.

4 participants