Skip to content

feat: add multi-invokers #251

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

Merged
merged 3 commits into from
May 10, 2024
Merged

feat: add multi-invokers #251

merged 3 commits into from
May 10, 2024

Conversation

ceejatec
Copy link
Contributor

Multi-invokers are a new feature to complement data providers, with greater flexibility: allowing for multiple parameters to be passed to the test function, and for parameters containing whitespace.

📚 Description

Multi-invokers are a new feature to complement data providers, with greater flexibility: allowing for multiple parameters to be passed to the test function, and for parameters containing whitespace.

🔖 Changes

Extended the core runner:run_test function to accept arbitrary arguments to pass to test function.
Always use "$@" to pass through arguments so they are properly quoted!
Created new "parameterized tests" doc page, which documents multi-invokers and incorporates the doc for data providers which was previously at the bottom of the test-files page.

✅ To-do list

  • I updated the CHANGELOG.md to reflect the new feature or fix
  • I updated the documentation to reflect the changes

Multi-invokers are a new feature to complement data providers,
with greater flexibility: allowing for multiple parameters to be
passed to the test function, and for parameters containing whitespace.

Created new "parameterized tests" doc page, which documents
multi-invokers and incorporates the doc for data providers which
was previously at the bottom of the test-files page.
@ceejatec
Copy link
Contributor Author

ceejatec commented May 8, 2024

@Chemaclass Can this be merged?

@Chemaclass Chemaclass merged commit 86c7738 into TypedDevs:main May 10, 2024
7 checks passed
@ceejatec
Copy link
Contributor Author

Thank you!

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