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

Enable unit tests for Firefox on Windows #6895

Merged
merged 4 commits into from Mar 5, 2021
Merged

Conversation

whimboo
Copy link
Collaborator

@whimboo whimboo commented Feb 17, 2021

This will eventually fix issue #5673.

First I would like to see if tests might work again. If not further debugging is necessary.

@google-cla google-cla bot added the cla: yes label Feb 17, 2021
@whimboo whimboo force-pushed the ci_windows branch 6 times, most recently from 7829d4e to 276d4cc Compare February 17, 2021 10:49
@whimboo
Copy link
Collaborator Author

whimboo commented Feb 17, 2021

It's still busted so I'll have to take a look at https://bugzilla.mozilla.org/show_bug.cgi?id=1656962 first. We do not experience similar issues for other test jobs in our CI.

@whimboo whimboo force-pushed the ci_windows branch 6 times, most recently from 0f62a74 to 31147c3 Compare March 3, 2021 18:20
@whimboo whimboo changed the title chore: enable unit tests for Firefoxc on Windows chore: enable unit tests for Firefox on Windows Mar 3, 2021
@whimboo whimboo changed the title chore: enable unit tests for Firefox on Windows Enable unit tests for Firefox on Windows Mar 3, 2021
@whimboo
Copy link
Collaborator Author

whimboo commented Mar 3, 2021

This PR is ready to be reviewed now. I hope the way that I used is fine for you.

Copy link
Contributor

@jschfflr jschfflr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM % nit about types

src/node/BrowserRunner.ts Outdated Show resolved Hide resolved
@whimboo
Copy link
Collaborator Author

whimboo commented Mar 4, 2021

@jschfflr I updated the patch and all tests are passing. Would you mind merging the PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants