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

Add CI for Node.js and Browser tests #57

Closed
daviddias opened this issue Sep 15, 2015 · 10 comments
Closed

Add CI for Node.js and Browser tests #57

daviddias opened this issue Sep 15, 2015 · 10 comments

Comments

@daviddias
Copy link
Contributor

No description provided.

@victorb
Copy link
Contributor

victorb commented Sep 15, 2015

Starting looking into this, just got to this page before having other things to attend to: http://docs.travis-ci.com/user/gui-and-headless-browsers/ FYI

@harlantwood
Copy link
Contributor

Circle CI is running tests, tried and failed to create a PR just now from my phone to add the status badge to the homepage:

Circle CI

Code:

[![Circle CI](https://circleci.com/gh/ipfs/node-ipfs-api.svg?style=svg)](https://circleci.com/gh/ipfs/node-ipfs-api)

@harlantwood
Copy link
Contributor

...but perhaps this thread is suggesting we start running additional types of tests in CI...

daviddias added a commit that referenced this issue Sep 16, 2015
@daviddias
Copy link
Contributor Author

Thank you @harlantwood , just PR'ed with the badged you added.

For browser tests CI, I was hoping to have something like:

image

@harlantwood
Copy link
Contributor

Ah! Thanks for PR, and great idea!

@daviddias
Copy link
Contributor Author

Let's do this! #81

@daviddias
Copy link
Contributor Author

Following up on #81, we need to Tweak timeouts so the tests pass on sauce labs every time, so that we have CI browser tests on node-ipfs-api :)

API

  • before tests run
  • .send
    • node.js
    • browser
  • .add
    • .add a file
      • node.js
      • browser
    • .add a buffer
      • node.js
      • browser
    • .add a path
      • node.js
      • browser
    • .add a folder (nested)
      • node.js
      • browser
  • .cat
    • node.js
    • browser
  • .ls
    • node.js
    • browser
  • .config
    • .config.get
      • node.js
      • browser
    • .config.set
      • node.js
      • browser
    • .config.show
      • node.js
      • browser
    • .config.replace
      • node.js
      • browser
  • .update (Currently disabled, wait for version 0.4.0 release)
  • .version
    • node.js
    • browser
  • .commands
    • node.js
    • browser
  • .mount (Requires FUSE, not practical for testing in node-ipfs-api)
  • .diag
    • .diag.net
      • node.js
      • browser
  • .block
    • .block.get
      • node.js
      • browser
    • .block.put
      • node.js
      • browser
  • .object
    • .object.get
      • node.js
      • browser
    • .object.put
      • node.js
      • browser
    • .object.data
      • node.js
      • browser
    • .object.stat
      • node.js
      • browser
    • .object.links
      • node.js
      • browser
  • .swarm
    • .swarm.peers
      • node.js
      • browser
    • .swarm.connect
      • node.js
      • browser
  • .ping
    • node.js
    • browser
  • .id
    • node.js
    • browser
  • .pin
    • .pin.add
      • node.js
      • browser
    • .pin.remove
      • node.js
      • browser
    • .pin.list
      • node.js
      • browser
  • .gateway this doesn't exist anymore on the http api
    • .gateway.enable
    • .gateway.disable
  • log
    • log.tail (needs 0.3.9 ndjson stuff)
  • .name
    • .name.publish
      • node.js
      • browser
    • .name.resolve
      • node.js
      • browser
  • .Buffer (do we still need to expose Buffer? Node.js now uses TypedArrays, so it shouldn't be necessary)
    • node.js
    • browser
  • .refs
    • .refs.local
      • node.js
      • browser
  • .dht
    • .dht.findprovs
      • node.js
      • browser
    • .dht.get
      • node.js
      • browser
    • .dht.put
      • node.js
      • browser

@daviddias
Copy link
Contributor Author

This just got a huge boost with latest release (0.2.9) now we have tests passing on the browser, in CI! :D

@jbenet
Copy link
Contributor

jbenet commented Nov 20, 2015

\o/

On Thu, Nov 19, 2015 at 4:20 AM David Dias notifications@github.com wrote:

This just got a huge boost with latest release (0.2.9) now we have tests
passing on the browser, in CI! :D


Reply to this email directly or view it on GitHub
#57 (comment).

@dignifiedquire
Copy link
Contributor

Closing in favor of #170 which is all that's left to do :)

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

Successfully merging a pull request may close this issue.

5 participants