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

Documentation around bats-mock could be improved #80

Open
liamstevens opened this issue Aug 29, 2024 · 1 comment
Open

Documentation around bats-mock could be improved #80

liamstevens opened this issue Aug 29, 2024 · 1 comment

Comments

@liamstevens
Copy link

Hey team,

Raising this as I spent quite a while yesterday scratching my head over my stubs not behaving the way that I would have expected them to, based on both internal and external docs. Specifically this relates to the use of *, and the fact that it only matches one arg in the pattern. I know this isn't actually explicitly called out in the original repository's documentation, either.

I can see that this was brought up in a separate issue with a corresponding PR, but something a little clearer than this or with examples of usage of multiple * would be good. There's also some instances of these being escaped in the README, which doesn't appear to matter based on my testing.

It would be useful, I feel, to add a section to the BK Documentation around testing plugins that covers this. It would have saved me a lot of time and headache.

@mcncl
Copy link
Contributor

mcncl commented Aug 30, 2024

@liamstevens thanks for raising this. We definitely want the experience of contributing to be as smooth as possible. We'll take a look in to how we can best document this 🙂

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

No branches or pull requests

2 participants