Github user vegaed commented on the issue:
    @jvwing Thanks for all the great suggestions and code. 
    I moved the try up in order to catch those types of parsing exceptions so 
now they will route to failure.
    Good catch on the integration tests
    Yes I agree unit testing is good. I was modeling off of S3 and SQS and 
didn't see any there, but I see now that dynamo and kinesis packages are being 
both unit and integration tested.

If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at or file a JIRA ticket
with INFRA.

Reply via email to