urbanairship/java-library — pull request #19 by scott-seo (2014-08-25 19:48:14+00:00)
Title: List Schedules, Update Schedule, Delete Schedule

I have added the list schedules, update, delete notification to APIClient. I tried unit testing my new methods in APIClientTest classes but failed without success. It looks like once APIClient.schedule(payload) method is called with payload and returned, immediate subsequent call to APIClient.listSchedules() will not yield any data that I am looking for since the state is only maintained during the initial request execution and response execution. Stubbing out entire response body json doesn't really feel like meaningful testing of API logic. I ended up testing my new functionalities by making the actual call to DEV Urban Airship endpoints.

I realize testing is harder in this case and I am not really familiar the entire web request and response mocking used in your library. I wish you can just accept my change but I understand if you require stringent unit testing.

by the way, I am returning a list of maps from list schedules for three reasons.
#1 I don't know what name is good in this case. Your API doc online suggest Schedule Object. Schedule Object has redundant sub-attributes that are present in Create Schedule response.
#2 I am using your library in grails app. Map is as good as a java class in grails.
#3 I am very new to jackson and your library

Please, advise on what I need to do.