ipfs-gui icon indicating copy to clipboard operation
ipfs-gui copied to clipboard

remove js-ipfs related deps

Open SgtPooki opened this issue 2 years ago • 2 comments

### Tasks
- [ ] remove ipfs
- [ ] remove ipfs-core-utils
- [ ] remove ipfs-provider
- [ ] remove is-ipfs

SgtPooki avatar Jun 12 '23 04:06 SgtPooki

the IPNS publishing fails when I remove the js-ipfs support from ipfsdCtl in ipfs-backend.js for e2e tests, and pass when I leave it alone:

results of npx playwright test -c ./test/e2e --repeat-each 10

 5 failed
    ipns.test.js:34:5 › IPNS publishing › Settings screen › should support adding new keys ─────────
    ipns.test.js:29:5 › IPNS publishing › Settings screen › should list IPNS keys ──────────────────
    ipns.test.js:85:5 › IPNS publishing › Files screen › should have functional "Publish to IPNS" context action
    ipns.test.js:85:5 › IPNS publishing › Files screen › should have functional "Publish to IPNS" context action
    ipns.test.js:85:5 › IPNS publishing › Files screen › should have functional "Publish to IPNS" context action
  355 passed (3.0m)

SgtPooki avatar Jun 13 '23 17:06 SgtPooki

I reverted changes to ipfs-backend.js and re-ran npx playwright test -c ./test/e2e --repeat-each 10 and got

  5 failed
    explore.test.js:181:5 › Explore screen › Inspecting CID › should open dag-pb unixFS XKCD Archives
    explore.test.js:181:5 › Explore screen › Inspecting CID › should open dag-pb unixFS XKCD Archives
    ipns.test.js:85:5 › IPNS publishing › Files screen › should have functional "Publish to IPNS" context action
    ipns.test.js:85:5 › IPNS publishing › Files screen › should have functional "Publish to IPNS" context action
    ipns.test.js:85:5 › IPNS publishing › Files screen › should have functional "Publish to IPNS" context action

SgtPooki avatar Jun 13 '23 17:06 SgtPooki