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

fix: Fix Windows tests #402

Merged
merged 1 commit into from
Feb 13, 2023
Merged

fix: Fix Windows tests #402

merged 1 commit into from
Feb 13, 2023

Conversation

sansyrox
Copy link
Member

get conditional global hosts as windows does not allow 0.0.0.0 host

Description

This PR fixes #372

Fixes test on windows

@sansyrox sansyrox marked this pull request as ready for review February 11, 2023 00:16
@netlify
Copy link

netlify bot commented Feb 11, 2023

Deploy Preview for robyn canceled.

Name Link
🔨 Latest commit a68d94b
🔍 Latest deploy log https://app.netlify.com/sites/robyn/deploys/63ea6477387c3d000814055d

Copy link
Collaborator

@AntoineRR AntoineRR left a comment

Choose a reason for hiding this comment

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

Thanks for fixing the remaining test for Windows @sansyrox ! I have a few suggestions before we can merge this.

integration_tests/conftest.py Outdated Show resolved Hide resolved
integration_tests/test_base_url.py Outdated Show resolved Hide resolved
@sansyrox
Copy link
Member Author

These are great suggestions @AntoineRR 😄

I will implement them tomorrow haven't slept in two days haha 🤣

@sansyrox sansyrox force-pushed the fix/windows-tests branch 2 times, most recently from 707cc30 to 35ed50b Compare February 13, 2023 16:23
get conditional global hosts as windows does not allow 0.0.0.0 host
@sansyrox
Copy link
Member Author

@AntoineRR , all done! 😄

Copy link
Collaborator

@AntoineRR AntoineRR left a comment

Choose a reason for hiding this comment

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

LGTM! The helpers directory is a nice touch, thanks!

@sansyrox sansyrox merged commit 3b74468 into main Feb 13, 2023
@sansyrox sansyrox deleted the fix/windows-tests branch February 13, 2023 19:54
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.

[BUG] All tests are not passing on windows
2 participants