Local test runs busted about graphql-language-service HOT 6 CLOSED

Updated the title because I saw that the tests actually do run in Travis CI (example: https://travis-ci.org/graphql/graphql-language-service/builds/265356781), so there is something funky with my local install (this after an rm -r * and a clean git checkout . and yarn).

If I can figure out what is locally wrong with my install, can close this one, but will still need to deal with the other issues (the Flow errors, for example).

from graphql-language-service.

It's been a while since I worked in this repo and a fair bit has changed since then, so quite possible that i am Doing It Wrong™.

from graphql-language-service.

lerna bootstrap --hoist maybe? And I'm aware of the flow errors with not finding packages correctly - this is a known issue in lerna and can be worked around with including flow-typed packages in the top directory. I just didn't find time to do that yet. @wincent

from graphql-language-service.

It happened because yarn automatically links all graphql-langauge-service-* packages to the local package folders and as package.json points to dist/index.js the important thing is to build all packages before running tests, as otherwise an outdated or no dist/index.js file is there. This was fixed in #166 by running lerna run build before the tests.
Now some flow errors appear, but the tests and flow was working when i tried #166, so maybe flow 0.55 changed something.

from graphql-language-service.

With [email protected] there is a bug with Object.create and the flowtype for object prototype not matching. We're waiting for a fix in 0.56.0 and will fix graphql package accordingly to properly fix the flow issues, which will mitigate the flow failures in this library (and others such as codemirror-graphql).

from graphql-language-service.

Tests are passing now - closing this in favor.

from graphql-language-service.