ipfs-shipyard / dataviz

ipfs data visualizations
MIT License
29 stars 12 forks source link

Automated tests + CI #3

Closed harlantwood closed 9 years ago

harlantwood commented 9 years ago

It occurred to me today that (most of the moving parts of) the tree viz in #1 can be tested on CI pretty readily:

While this does not test D3 or the layout code, it would test all of our logic, and the connection to the IPFS API.

jbenet commented 9 years ago

SGTM! :+1:

harlantwood commented 9 years ago

I'm working on this using mocha + webdriver.io ... @travisperson you mentioned e2e testing with these two, do you have any samples you like to help me get rolling?

travisperson commented 9 years ago

Nothing that I can share unfortunately. All the work I've done has been for my work.

I will give you a run down of some hints and tricks that I learned though!

https://angular.github.io/protractor/#/server-setup

You can of course set it up yourself as well.

If you have any questions let me know.

On Wed, Jun 10, 2015, 1:53 AM Harlan T Wood notifications@github.com wrote:

I'm working on this using mocha + webdriver.io ... @travisperson https://github.com/travisperson you mentioned e2e testing with these two, do you have any samples you like to help me get rolling?

— Reply to this email directly or view it on GitHub https://github.com/ipfs/dataviz/issues/3#issuecomment-110659724.

harlantwood commented 9 years ago

Thanks @travisperson, much appreciated!

harlantwood commented 9 years ago

Progress report: I made a simple webdriverio-mocha-example repo which tests integration of:

After some struggles, it now works both locally and on Circle: https://circleci.com/gh/harlantwood/webdriverio-mocha-example

Next step: actually testing something useful :wink:

jbenet commented 9 years ago

@harlantwood that's great!

harlantwood commented 9 years ago

I've let go of working on this for now. Once I got deep in, it seemed like most of what I was testing was actually that the IPFS API worked as expected, which already has its own tests.

Closing for now, anyone feel free to reopen if it seems like a good pathway for further exploration.