Skip to content

Stop using npm link in tests #3345

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.

Sign up for GitHub

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

Merged
merged 28 commits into from
Oct 30, 2017
Merged

Stop using npm link in tests #3345

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
28 commits
Select commit Hold shift + click to select a range
594b02a
Try a custom function
Timer Oct 28, 2017
85a1992
fix path
Timer Oct 28, 2017
0c62386
install rsync
Timer Oct 28, 2017
e6a696d
Test with node 8
Timer Oct 28, 2017
05d060f
Ensure package is removed from the tree
Timer Oct 28, 2017
b6e6f1c
Remove ours
Timer Oct 28, 2017
c567509
matching functions
Timer Oct 28, 2017
a736e39
try cygwin
Timer Oct 28, 2017
d9e2b34
try c:/
Timer Oct 28, 2017
2fe0106
sigh
Timer Oct 28, 2017
4a19ea9
Try /cygdrive/
gaearon Oct 29, 2017
2f5ffaf
Try /cygdrive/
gaearon Oct 29, 2017
e5df78b
Revert "Try /cygdrive/"
Timer Oct 29, 2017
7f6a4e0
Use sub-optimal copy
Timer Oct 29, 2017
4f1f598
Remove rsync install
Timer Oct 29, 2017
b926eba
Fix variable
Timer Oct 29, 2017
6e3c2a7
Continue on err
Timer Oct 29, 2017
810c806
Don't link react-scripts, lol
Timer Oct 29, 2017
c2c9db0
run yarn after eject
Timer Oct 29, 2017
3a2130e
re-enable cleanup
Timer Oct 29, 2017
e14903e
uninstall yarn
Timer Oct 29, 2017
47c056a
try with force
Timer Oct 29, 2017
2758cab
appveyor ... seriously?
Timer Oct 29, 2017
ce0c223
swallow bad eject
Timer Oct 29, 2017
6c7cdeb
remove eject error swallow
Timer Oct 29, 2017
2ca4c84
Revert "run yarn after eject"
Timer Oct 29, 2017
dea7cd7
Run yarn after eject
Timer Oct 29, 2017
e05bbbe
add a comment
Timer Oct 29, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Don't link react-scripts, lol
  • Loading branch information
@Timer
Timer committed Oct 29, 2017
commit 810c80686018c416c88117aba91aefd5d4594549
3 changes: 1 addition & 2 deletions tasks/e2e-kitchensink.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ function cleanup {
ps -ef | grep 'react-scripts' | grep -v grep | awk '{print $2}' | xargs kill -9
cd "$root_path"
# TODO: fix "Device or resource busy" and remove ``|| $CI`
rm -rf "$temp_cli_path" "$temp_app_path" "$temp_module_path" || $CI
# rm -rf "$temp_cli_path" "$temp_app_path" "$temp_module_path" || $CI
}

# Error messages are redirected to stderr
Expand Down Expand Up @@ -252,7 +252,6 @@ install_package "$root_path"/packages/babel-preset-react-app
install_package "$root_path"/packages/eslint-config-react-app
install_package "$root_path"/packages/react-error-overlay
install_package "$root_path"/packages/react-dev-utils
install_package "$root_path"/packages/react-scripts

# Link to test module
install_package "$temp_module_path/node_modules/test-integrity"
Expand Down
3 changes: 1 addition & 2 deletions tasks/e2e-simple.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ function cleanup {
cd "$root_path"
# Uncomment when snapshot testing is enabled by default:
# rm ./packages/react-scripts/template/src/__snapshots__/App.test.js.snap
rm -rf "$temp_cli_path" $temp_app_path
# rm -rf "$temp_cli_path" $temp_app_path
}

# Error messages are redirected to stderr
Expand Down Expand Up @@ -335,7 +335,6 @@ echo yes | npm run eject
install_package "$root_path"/packages/babel-preset-react-app
install_package "$root_path"/packages/eslint-config-react-app
install_package "$root_path"/packages/react-dev-utils
install_package "$root_path"/packages/react-scripts

# Test the build
npm run build
Expand Down