Skip to content

Commit 2a63ed6

Browse files
gregmagolanalexeagle
authored andcommitted
fix(builtin): add test case for @bazel/hide-bazel-files bug
1 parent 7e777c7 commit 2a63ed6

4 files changed

Lines changed: 11 additions & 32 deletions

File tree

e2e/symlinked_node_modules_npm/package-lock.json

Lines changed: 3 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
{
22
"dependencies": {
3-
"typescript": "2.9.1",
4-
"rxjs": "6.4.0"
3+
"rxjs": "6.4.0",
4+
"typescript": "2.9.1"
55
},
66
"devDependencies": {
7-
"@bazel/hide-bazel-files": "file:../../dist/npm_bazel_hide-bazel-files"
7+
"@bazel/hide-bazel-files": "0.32.1"
88
},
99
"scripts": {
10-
"postinstall": "hide-bazel-files",
11-
"test": "bazel test ... && rm -rf node_modules && bazel test ... && rm -rf node_modules && yarn install && bazel test ..."
10+
"//": "we use fixed version of @bazel/hide-bazel-files above as npm makes symlinks for `file:` deps and @bazel/hide-bazel-files doesn't work as a symlink because that would require --preserve-symlinks-main",
11+
"test": "bazel test ... && rm -rf node_modules && bazel test ... && rm -rf node_modules && npm install && npm add @schematics/angular@8.0.0-beta.15 && bazel test ... && npm remove @schematics/angular"
1212
}
1313
}

e2e/symlinked_node_modules_npm/yarn.lock

Lines changed: 0 additions & 23 deletions
This file was deleted.
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,12 @@
11
{
22
"dependencies": {
3-
"typescript": "2.9.1",
4-
"rxjs": "6.4.0"
3+
"rxjs": "6.4.0",
4+
"typescript": "2.9.1"
55
},
66
"devDependencies": {
77
"@bazel/hide-bazel-files": "file:../../dist/npm_bazel_hide-bazel-files"
88
},
99
"scripts": {
10-
"test": "bazel test ... && rm -rf node_modules && bazel test ... && rm -rf node_modules && yarn install && bazel test ..."
10+
"test": "bazel test ... && rm -rf node_modules && bazel test ... && rm -rf node_modules && yarn install && yarn add @schematics/angular@8.0.0-beta.15 && bazel test ... && yarn remove @schematics/angular"
1111
}
1212
}

0 commit comments

Comments
 (0)