-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat: Allow post-id only by using profile
from metadata
#6
Conversation
7a11abd
to
1800e42
Compare
We potentially might want to mirror the main Quarto approach for tests, e.g. @cderv were there ever any notes on best practices for testing Quarto extensions outside of the |
No there is not yet for extension. I would say it depends here what you want to test. Testing quarto with this extension
This is easy to do in this repo so maybe the latter is too much yet. Testing your extension lua code
So as you see there is not yet one way, and I have given thought about it. So we could also start comming up with some advice and nice helper for that. Hope it helps |
My thinking was that eventually we could use playwright for client-side testing. For now, keeping the test cases in the docs site is a cheap suite of smoke tests in that we'll get CI failures if the docs site fails to build. It's also easy for manual verification since the tests are deployed with the docs (and in a way serves as additional examples). For stuff that is supposed to break with a quarto render, or for testing in a project context, we'd need to use |
Part 2 of addressing the changes described in this comment.
This PR also adds a set of unlisted tests to
docs/
that can be viewed at{site-preview}/tests
. The goal is to have a set of pages for quick manual checks (that we might automate at some point).