Skip to content

test: each test case for browser#3100

Merged
sheremet-va merged 7 commits intovitest-dev:mainfrom
Aslemammad:fix/test-each-browser
Apr 8, 2023
Merged

test: each test case for browser#3100
sheremet-va merged 7 commits intovitest-dev:mainfrom
Aslemammad:fix/test-each-browser

Conversation

@Aslemammad
Copy link
Member

@Aslemammad Aslemammad commented Mar 29, 2023

This tries to resolve issue #3085 since it cannot be reproduced in linux and it works as expected, but adding a test case for more safety!
Resolves #3124

@Aslemammad
Copy link
Member Author

We need to add a browser + windows ci!

@Aslemammad
Copy link
Member Author

The solution for #3124 was that vite didn't allow accessing the dist/utils.js file because of config.fs.strict, so I made it false so the files can be accessed.

@Aslemammad
Copy link
Member Author

For the each case, I tested it in different platforms and it couldn't be reproduced, same with the tests.

@Aslemammad Aslemammad requested a review from sheremet-va April 8, 2023 11:49
@sheremet-va sheremet-va merged commit ddbda10 into vitest-dev:main Apr 8, 2023
@Aslemammad Aslemammad deleted the fix/test-each-browser branch April 8, 2023 11:54
@veaba
Copy link

veaba commented Apr 9, 2023

Not yet fixed on windows.

image

@sheremet-va
Copy link
Member

sheremet-va commented Apr 9, 2023

Not yet fixed on windows.

image

This is why the issue is not closed. And the fix is not yet released.

@veaba
Copy link

veaba commented Apr 9, 2023

Sorry, I thought this PR would fix the problem.

I just clone this repo and use pnpm link to import vitest into my project.

it working!

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.

Browser mode hanging on blank white screen, failing to load vitest/utils

3 participants