Skip to content

Conversation

@thymikee
Copy link
Member

Summary:

An attempt to fix the tests on Windows.

Test Plan:

Green CI

@thymikee thymikee force-pushed the fix/windows branch 5 times, most recently from 4173154 to f3c75d1 Compare July 18, 2019 10:02
@thymikee
Copy link
Member Author

@Esemesek feel free to jump on the last 2 failing tests:

  • one is about glob returning forward slashes even on windows (so either convert path separators in source code or adjust this test to additionally replace forward slashes on windows)
  • second one kinda worries me, because it seems like the files are not copied recursively, however Windows users didn't report anything like it, so it's probably something in our testing setup

@thymikee
Copy link
Member Author

thymikee commented Aug 2, 2019

@theweavrs it would be lovely if you could help me finish this PR up! <3

@thecodrr
Copy link
Contributor

thecodrr commented Aug 2, 2019

@thymikee sure, how can I help?

@thymikee
Copy link
Member Author

thymikee commented Aug 2, 2019

By fixing the tests that are still failing and sending a PR to this branch (fix/windows) 😄

thecodrr and others added 2 commits August 4, 2019 18:14
* use stringify with replacer

* use slash

* fix: rnpm deprecation test

* fix: copyFiles test in windows

* fix: e2e tests on windows

* fix: init from local template

* fix: cli getting stuck on replacing project dir

* fix: properly escape regexp when copying template

* refactor: use escape func from jest-regex-util

* refactor: copy escape func from jest-regex-util
@thymikee thymikee changed the title WIP: chore: fix tests on Windows chore: fix tests on Windows Aug 4, 2019
@thymikee thymikee merged commit f976b4c into master Aug 4, 2019
@thymikee
Copy link
Member Author

thymikee commented Aug 4, 2019

Thank you @theweavrs ❤️

@thymikee thymikee deleted the fix/windows branch August 4, 2019 17:03
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.

3 participants