Low Unit Test Coverage

[ryanhuff]

I think we're only at 58% right now.

[mattlacey]

Still not managed to get around to looking at this properly, really do need
it though! Will put it on the list for this week.

On 6 January 2014 12:09, ryanhuff notifications@github.com wrote:

I think we're only at 58% right now.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/6
.

[Taskle]

Hi guys - full disclosure that we're a Cirrus Insights customer (Doctor.com). I'm working on deploying this library but am unable to because of the lack of code coverage. At this point I'm forced to write my own unit tests to deploy it (since we need to deploy ASAP, long story). If you guys have already happened to do that please let me know.

Also, FYI, 2 fields missing from the Stripe_Webhook object that were needed to deploy it: Event_ID__c and Error_Message__c (I haven't implemented custom logic for handling these webhooks yet).

Anyhow, if you have unit tests, let me know, as most people can't deploy until this is fixed because of Salesforce's 75% unit test coverage requirement.

Cheers,
Dave

[ryanhuff]

Thanks for submitting this, Dave. If you have time, you can submit your changes as a pull request.

Sent usingCirrus Insight Mobile(http://www2.cirrusinsight.com/mobile-appstore)

On Mar 31, 2015, at 12:40 PM, "David Murray"<notifications@github.com>wrote:

Hi guys - full disclosure that we're a Cirrus Insights customer (Doctor.com). I'm working on deploying this library but am unable to because of the lack of code coverage. At this point I'm forced to write my own unit tests to deploy it (since we need to deploy ASAP, long story). If you guys have already happened to do that please let me know.

Also, FYI, 2 fields missing from the Stripe_Webhook object that were needed to deploy it: Event_ID__c and Error_Message__c (I haven't implemented custom logic for handling these webhooks yet).

Anyhow, if you have unit tests, let me know, as most people can't deploy until this is fixed because of Salesforce's 75% unit test coverage requirement.

Cheers,
Dave

—
Reply to this email directly orview it on GitHub(#6 (comment)).