Skip to content

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
@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