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

Testing/Debugging Refinements #41

Merged
merged 5 commits into from
Oct 30, 2024
Merged

Testing/Debugging Refinements #41

merged 5 commits into from
Oct 30, 2024

Conversation

noelforte
Copy link
Owner

@noelforte noelforte commented Oct 20, 2024

This pull tracks refinements to tests for this plugin, notably addressing further issues with caching templates.

Goals:

  • Move all basic functionality tests to /tests/stubs/
  • Refactor test files that cover the above functionality into a single test suite, keep serve, autotrim, ignore-tag and other custom features seperate for erganomics
  • Add dependency on debug package (perhaps as a peer, to pull in from eleventy) (added in b8f0a03)

@noelforte noelforte self-assigned this Oct 20, 2024
@noelforte noelforte linked an issue Oct 20, 2024 that may be closed by this pull request
@noelforte noelforte marked this pull request as draft October 22, 2024 21:39
@noelforte noelforte force-pushed the testing-refinements branch from 176852f to a4d9af5 Compare October 24, 2024 03:49
@noelforte noelforte closed this Oct 25, 2024
@noelforte noelforte force-pushed the testing-refinements branch from a4d9af5 to bb128ce Compare October 25, 2024 02:51
@noelforte noelforte reopened this Oct 29, 2024
@noelforte noelforte marked this pull request as ready for review October 29, 2024 04:47
@noelforte noelforte force-pushed the testing-refinements branch from 9dd5be8 to 6c6c974 Compare October 29, 2024 04:51
@noelforte
Copy link
Owner Author

Decided to forgo testing serve mode since it's a bit involved to set up and it's simpler to rely instead on emperical observations. If there are more issues with caching then I can add test cases to handle it.

@noelforte noelforte merged commit 92c6000 into main Oct 30, 2024
3 checks passed
@noelforte noelforte deleted the testing-refinements branch October 30, 2024 03:02
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.

1 participant