Skip to content
This repository has been archived by the owner on Mar 10, 2020. It is now read-only.

chore: test against go-ipfs 0.4.13 #638

Merged
merged 3 commits into from
Dec 1, 2017
Merged

Conversation

daviddias
Copy link
Contributor

No description provided.

@ghost ghost assigned daviddias Dec 1, 2017
@ghost ghost added the in progress label Dec 1, 2017
@daviddias
Copy link
Contributor Author

daviddias commented Dec 1, 2017

A ton of tests is failing when testing js-ipfs-api against go-ipfs@0.4.13. @keks do you happen the know why? Could this be related to cmds changes?

Also, this is why #486 is a P0 //cc @victorbjelkholm

@keks
Copy link

keks commented Dec 1, 2017

We only merged the cmds stuff right after 0.4.13, so that can't really be the reason.

@daviddias
Copy link
Contributor Author

@keks any other HTTP-API changes that came in 0.4.12 or 0.4.13?

@keks
Copy link

keks commented Dec 1, 2017

Not sure - 0.4.13 was a hotfix release right after 0.4.12, I think there were very few changes. Do the tests pass for 0.4.12?

@richardschneider
Copy link
Contributor

@diasdavid installed 0.4.13 and used my net-http-api client against it. All tests passed.

@daviddias
Copy link
Contributor Author

@richardschneider do you make any use of the HTTP Headers in your library? Especially the x-stream and x-chunked?

@daviddias daviddias mentioned this pull request Dec 1, 2017
11 tasks
@richardschneider
Copy link
Contributor

@diasdavid I don't use any special headers, except asking for GZIP response.

package.json Outdated
@@ -43,7 +43,7 @@
"peer-id": "~0.10.3",
"peer-info": "~0.11.3",
"promisify-es6": "^1.0.3",
"pump": "^2.0.0",
"pump": "^1.0.3",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@keks @richardschneider this was the culprit! Apologies for taking from your time

@codecov
Copy link

codecov bot commented Dec 1, 2017

Codecov Report

Merging #638 into master will decrease coverage by 0.23%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #638      +/-   ##
==========================================
- Coverage   84.25%   84.02%   -0.24%     
==========================================
  Files         115      115              
  Lines        1747     1746       -1     
==========================================
- Hits         1472     1467       -5     
- Misses        275      279       +4
Impacted Files Coverage Δ
src/utils/send-files-stream.js 88.6% <100%> (-0.15%) ⬇️
src/pubsub.js 74.32% <0%> (-5.41%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 62d733e...960dd79. Read the comment docs.

@daviddias daviddias merged commit 2fc7d0a into master Dec 1, 2017
@daviddias daviddias deleted the test/go-ipfs-v0.4.13 branch December 1, 2017 21:00
@ghost ghost removed the in progress label Dec 1, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants