Skip to content

Add Alternate Mocking Framework (3-of-3 / New unit tests) #119

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.

Sign up for GitHub

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

Jump to bottom
Open
sappenin wants to merge 5 commits into
base: df/mock-all/2-update-existing-tests
Choose a base branch
from
Open

Add Alternate Mocking Framework (3-of-3 / New unit tests) #119

sappenin wants to merge 5 commits into from

Conversation

@sappenin
Copy link
Collaborator

@sappenin sappenin commented Dec 17, 2025 •
edited
Loading

High Level Overview of Change

  • Add new unit tests using the new mocking framework.

Context of Change

This PR is number-3 in a series of three PRs that offers an alternative design to the proposal in #102. In particular:

  • PR #117 (this PR): Adds initial mocking support using the mockall framework.
  • PR #118: Updates existing unit tests to use the new mocking framework, where appropriate.
  • PR #119: Adds new unit test coverage to a variety of host-function usages in the library.

@sappenin sappenin self-assigned this Dec 17, 2025
@sappenin sappenin changed the title Add new unit tests Add Alternate Mocking Framework (3-of-3 / New unit tests) Dec 17, 2025
This was referenced Dec 17, 2025
Open
Merged
Closed
sappenin
sappenin commented Dec 17, 2025
View reviewed changes
@sappenin sappenin requested a review from mvadari December 17, 2025 16:02
@sappenin sappenin marked this pull request as ready for review December 17, 2025 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mvadari mvadari Awaiting requested review from mvadari

Assignees

@sappenin sappenin

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@sappenin