search

bvalosek / infusionsoft-api

A promise-driven Node.js wrapper for the XML-RPC Infusionsoft API
MIT License
16 stars 14 forks source link

Tests? #4

Open bitmage opened 10 years ago

bitmage commented 10 years ago

This looks like a very handy library, but I'm hesitant to use it without tests. Do you have plans to test, or have you identified barriers that would prevent it?

bvalosek commented 10 years ago

I don't have any plans to implement tests at the moment, though I'd welcome any pull requests.

The Queryable class could use benefit from testing that would not require any connection to the actual infusionsoft API, but just verifying the semantics of how it is building the queries.

If I were to take the time to implement tests personally, I would actual revisit the way the library is structured in general as I think it is overly-automagical without a whole lot of benefit.