Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

add additional test cases for 'no_run' #926

Closed
wants to merge 1 commit into from

Conversation

danieleades
Copy link
Contributor

@danieleades danieleades commented Jan 4, 2022

adds additional test cases to check that the 'no_run' attribute on doc-tests is properly respected even when there is also a rust tag

see #924

@danieleades
Copy link
Contributor Author

these additional test cases confirm that the tests don't run when there are multiple code-block attributes. Unfortunately they don't test that they do run if they have multiple attributes that don't include "no_run"...

@xd009642 xd009642 marked this pull request as ready for review January 4, 2022 19:44
@xd009642
Copy link
Owner

xd009642 commented Jan 4, 2022

You can add some tests to the normal doc tests crate with some attributes that aren't no_run or look at maybe unit testing in the cargo.toml 🤔

@danieleades danieleades closed this Jan 5, 2022
@danieleades danieleades deleted the multiple-doc-tags branch January 5, 2022 08:24
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants