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

[epic] Experiment with mockable operations #482

Draft
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

josephjclark
Copy link
Collaborator

Experimenting with:

  • Alternative adaptor stucture
  • Mockable functions for unit and integration tests
  • job code as unit tests and example documentation

See tutorial.md for a guide to the changes.

@josephjclark josephjclark marked this pull request as draft March 13, 2024 05:59
@josephjclark josephjclark changed the title Fancy mocks experiements Experiment with mockable operations Mar 13, 2024
@josephjclark
Copy link
Collaborator Author

One possible benefit of the operation/impl split is that you could call the impl directly from job code, allowing you to do nested operations more easily. Although even as I type this it feels like bad practice. Would it make women-for-women any cleaner?

@mtuchi mtuchi mentioned this pull request Mar 21, 2024
@josephjclark josephjclark changed the title Experiment with mockable operations [epic] Experiment with mockable operations May 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: New Issues
Development

Successfully merging this pull request may close these issues.

2 participants