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

Refactor tests to not assume name of project folder #25

Merged
merged 3 commits into from
Mar 12, 2024

Conversation

kapouer
Copy link
Contributor

@kapouer kapouer commented Mar 11, 2024

These three tests fail when the parent directory is not "import-meta-resolve".

Signed-off-by: Jérémy Lal <kapouer@melix.org>
@wooorm
Copy link
Owner

wooorm commented Mar 11, 2024

🤔 but i want to test up-down too 🤔

@kapouer
Copy link
Contributor Author

kapouer commented Mar 11, 2024

This test is not supposed to test pathToFileURL, but resolve, doesn't it ?
Changing an argument of pathToFileURL won't change the test intent.

@wooorm
Copy link
Owner

wooorm commented Mar 12, 2024

I missed that, thanks for clarifying

@wooorm wooorm changed the title test: do not assume project dirname Refactor tests to not assume name of project folder Mar 12, 2024
@wooorm wooorm merged commit 66b952b into wooorm:main Mar 12, 2024
4 checks passed
@wooorm
Copy link
Owner

wooorm commented Mar 12, 2024

Thanks!

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.

2 participants