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

dedupe tests #573

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

dedupe tests #573

wants to merge 5 commits into from

Conversation

hugomrdias
Copy link
Contributor

@hugomrdias hugomrdias commented Dec 12, 2019

This PR supports the deduping of tests in ipfs-http-client and js-ipfs.

Tests will be added here and removed in both repos.

changes to subsystems

  • dag.get
  • ping
  • mfs.ls
  • add

refs:
ipfs-inactive/js-ipfs-http-client#1200
ipfs/js-ipfs#2669

@hugomrdias hugomrdias marked this pull request as ready for review December 13, 2019 11:11
@achingbrain
Copy link
Collaborator

Is there some way of comparing coverage before/after to make sure nothing is missed in these PRs?

@hugomrdias
Copy link
Contributor Author

Yes check codecov in the http-client PR apparently we lost these 2 bits https://codecov.io/gh/ipfs/js-ipfs-http-client/compare/23421f397e4ae80ac9e9264ee6662b2cc0e34791...2e30f994459a038fc4665890101a23beafcd0f71/changes

@hugomrdias
Copy link
Contributor Author

In js-ipfs
codecov/project
Absolute coverage decreased by -0.82% but relative coverage increased by +11.8% compared to 6395730

And the follow up cli tests PRs will increase the cov more.

@hugomrdias
Copy link
Contributor Author

added a new test for the validation in dag.get

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

Successfully merging this pull request may close these issues.

2 participants