Skip to content

chore: refactor mocking in tests that need caluma responses#997

Merged
open-dynaMIX merged 2 commits intoadfinis:mainfrom
open-dynaMIX:cleanup_graphql_tests
Jan 11, 2024
Merged

chore: refactor mocking in tests that need caluma responses#997
open-dynaMIX merged 2 commits intoadfinis:mainfrom
open-dynaMIX:cleanup_graphql_tests

Conversation

@open-dynaMIX
Copy link
Copy Markdown
Member

@open-dynaMIX open-dynaMIX commented Jan 11, 2024

This commit streamlines almost duplicate code in the conftest files of the accounting and case apps into the main conftest file.

Drive-by commit:

  • docs: add docstring & fix typo in comment

This commit streamlines almost duplicate code in the conftest files of
the accounting and case apps into the main conftest file.
@open-dynaMIX open-dynaMIX force-pushed the cleanup_graphql_tests branch from 5c1f469 to 9fae862 Compare January 11, 2024 11:37
@open-dynaMIX open-dynaMIX requested a review from winged January 11, 2024 11:40
Copy link
Copy Markdown
Member

@winged winged left a comment

Choose a reason for hiding this comment

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

Awesome! And thanks for the docstring :-)

@open-dynaMIX open-dynaMIX merged commit 31a5e18 into adfinis:main Jan 11, 2024
@open-dynaMIX open-dynaMIX deleted the cleanup_graphql_tests branch January 11, 2024 13:46
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.

2 participants