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

npm t fail #293

Closed
Berkmann18 opened this issue Apr 10, 2019 · 2 comments
Closed

npm t fail #293

Berkmann18 opened this issue Apr 10, 2019 · 2 comments

Comments

@Berkmann18
Copy link
Contributor

Berkmann18 commented Apr 10, 2019

- Do you want to request a feature or report a bug?
Bug.
- What is the current behaviour?
It goes all the way to dependency-check ./package.json ... and fails with some contradicting logs.

run-s test:*
> netlify-cli@2.11.4 test:ava /mnt/c/Users/max/Projects/netlify-cli
> nyc --reporter=lcov ava --verbose && nyc report
✔ camelCase to NETLIFY_ENV_CASE
✔ isDotProp detects doPropPaths
2 tests passed
----------|----------|----------|----------|----------|-------------------|
File      |  % Stmts | % Branch |  % Funcs |  % Lines | Uncovered Line #s |
----------|----------|----------|----------|----------|-------------------|
All files |      100 |      100 |      100 |      100 |                   |
 util.js  |      100 |      100 |      100 |      100 |                   |
----------|----------|----------|----------|----------|-------------------|
> netlify-cli@2.11.4 test:lint /mnt/c/Users/max/Projects/netlify-cli
> eslint src
> netlify-cli@2.11.4 test:deps /mnt/c/Users/max/Projects/netlify-cli
> dependency-check ./package.json --entry "src/**/!(*.test).js" --unused --missing --no-dev --no-peer -i @oclif/plugin-not-found -i @oclif/config -i @oclif/plugin-help -i @oclif/plugin-plugins
Fail! Modules in package.json not used in code: netlify-dev-plugin
Success! All dependencies used in the code are listed in package.json

- If the current behaviour is a bug, please provide the steps to reproduce.

git clone https://github.com/netlify/cli.git netlify-cli && cd netlify-cli
npm i
npm t

- What is the expected behaviour?
The test passed with no errors.

- Please mention your node.js, and operating system version.
Node: v11.12.0
OS: Ubuntu 18.10 (WSL)

@bcomnes
Copy link
Contributor

bcomnes commented Apr 10, 2019

Ah yeah, oclif has a bespoke dependency injection system, so it breaks standard dependency resolution algorithms. The solution is to add that plugin to the ignore list.

bcomnes added a commit that referenced this issue Apr 10, 2019
@bcomnes bcomnes mentioned this issue Apr 10, 2019
@bcomnes
Copy link
Contributor

bcomnes commented Apr 10, 2019

It also looks like Azure isn't catching failed tests, which is why I didn't notice this right away #295

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

No branches or pull requests

2 participants