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

Added Mock module #154

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from
Open

Conversation

szabo137
Copy link
Member

This moves the test implementation from the test directory to the src and exposes it as a submodule. It can be excesses elegantly:

using QEDbase.Mocks

proc = MockProcess(...)

Furthermore, the notion of Test as prefixes is replaced by Mock to be more explicit with the meaning of the module Mocks.

solves #151

Summary of changes

  • moved test implementation to QEDbase.Mocks
  • replaced 'test' by 'mock' where appropriate
  • adjusted unit tests

@szabo137
Copy link
Member Author

Would be nice, if we could release QEDbase after this is merged to expose QEDbase.Mocks to the other packages of the framework.

AntonReinhard
AntonReinhard previously approved these changes Jan 29, 2025
@AntonReinhard
Copy link
Member

Would be nice, if we could release QEDbase after this is merged to expose QEDbase.Mocks to the other packages of the framework.

Maybe we should do that together with the current PSDEF removal. Otherwise, we'll have two breaking releases right after each other. I don't think a release should be necessary before then because the non-release pipelines all work with the dev versions anyway.

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.

Making TestImplementation available for the whole framework
2 participants