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

Tests failing #838

Closed
collinsauve opened this issue May 18, 2017 · 6 comments
Closed

Tests failing #838

collinsauve opened this issue May 18, 2017 · 6 comments

Comments

@collinsauve
Copy link
Contributor

When running npm run test from master I get 50 errors that look like this:

45) no-unresolved syntax verification valid import json from "./data.json":

      AssertionError: Should have no errors but had 1: [ { ruleId: 'no-unresolved syntax verification',
    severity: 1,
    message: 'Casing of ./data.json does not match the underlying filesystem.',
    line: 1,
    column: 18,
    nodeType: 'Literal',
    source: 'import json from "./data.json"' } ]
      + expected - actual

      -1
      +0

      at testValidTemplate (node_modules\eslint\lib\testers\rule-tester.js:423:20)
      at Context.RuleTester.it (node_modules\eslint\lib\testers\rule-tester.js:548:25)

I am running:

  • Windows 10 (v1607 build 14393.447)
  • NPM v4.1.2
  • Node v7.7.4
  • GNU bash, version 4.3.46(2)-release (x86_64-pc-msys)
@benmosher
Copy link
Member

can you confirm that data.json case does actually match?

@collinsauve
Copy link
Contributor Author

Is it supposed to be all lower-case?

$ find . -iname data.json
./resolvers/node/test/data.json
./tests/files/data.json

@benmosher
Copy link
Member

yep. super weird. I wonder if it's a Node 7 issue? I have not tested past Node v6

@benmosher
Copy link
Member

benmosher commented May 25, 2017

oh actually: does your shell pwd path match the filesystem? there was an old Windows issue where if you navigated case-insensitive to your code folder it would fail because the absolute path (based on pwd) didn't match FS.

@benmosher
Copy link
Member

If #846 passes I will close, assuming it's that.

@collinsauve
Copy link
Contributor Author

does your shell pwd path match the filesystem?

That was it! All tests are passing now - thank you so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants