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

Enhance test suite around zola serve #2448

Open
jamwil opened this issue Mar 3, 2024 · 3 comments
Open

Enhance test suite around zola serve #2448

jamwil opened this issue Mar 3, 2024 · 3 comments

Comments

@jamwil
Copy link
Contributor

jamwil commented Mar 3, 2024

Setting this up to discuss/track the addition of more thorough endpoint testing around the serve command. Better test coverage will support some refactoring efforts, including #2446 along with a better url path mounting implementation as discussed in #2311.

@jamwil
Copy link
Contributor Author

jamwil commented Apr 11, 2024

Apologies for the lack of action on this so far. It's still on my radar—energy level be damned.

@jamwil
Copy link
Contributor Author

jamwil commented Jun 27, 2024

Just marking here that the unit tests introduced in #2484 seem to fail intermittently in Windows CI. Will leave this issue and try to identify what's causing that.

@jamwil
Copy link
Contributor Author

jamwil commented Aug 30, 2024

Hmm. So I finally got around to testing these on a Windows box. All green. Unfortunately, this is CI-specific.

Edit: One thing I can try is to copy the test site into a temporary directory on each test invocation. That should, in theory, fix any concurrency issue.

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

1 participant