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

Test suite fails #37

Closed
chrisknuteson-qumulex opened this issue Aug 12, 2019 · 5 comments
Closed

Test suite fails #37

chrisknuteson-qumulex opened this issue Aug 12, 2019 · 5 comments

Comments

@chrisknuteson-qumulex
Copy link

chrisknuteson-qumulex commented Aug 12, 2019

Greetings,

It looks like the automated tests fail when trying to hit the typedefsUrl.

⠧  Syncing type definitions in package.json...events.js:173
      throw er; // Unhandled 'error' event
      ^

Error: incorrect header check
    at Zlib.zlibOnError [as onerror] (zlib.js:162:17)
Emitted 'error' event at:
    at errorOrDestroy (internal/streams/destroy.js:98:12)
    at Gunzip.onerror (_stream_readable.js:717:7)
    at Gunzip.emit (events.js:197:13)
    at Zlib.zlibOnError [as onerror] (zlib.js:165:8)
error Command failed with exit code 1.

Simple reproduction case: Pull the repo; npm install; npm test

@jeffijoe
Copy link
Owner

jeffijoe commented Aug 12, 2019

Sounds like a transient error with the type definitions source. Try again?

Edit: yeah it's a type def source issue. Should probably be resolved soon. I'd imagine they are updating it or something.

@chrisknuteson-qumulex
Copy link
Author

Maybe. I've had failing statuses for about 10 min now so I thought to run it across your bow

Is it possible some header change from the source is affecting the fetch?
https://typespublisher.blob.core.windows.net/typespublisher/data/search-index-min.json

@jeffijoe
Copy link
Owner

The error is from the unzip. I'd imagine a new file is being uploaded and it's not complete yet? Maybe they stopped zipping it? Not sure, will have to wait and see.

@chrisknuteson-qumulex
Copy link
Author

Thanks!

@jeffijoe
Copy link
Owner

Apparently unzipping is no longer needed. Shipped 0.6.1 to fix this.

This issue was closed.
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