Branch: refs/heads/master Home: https://github.com/jenkinsci/google-play-android-publisher-plugin Commit: 66f4bd5e956f9827da436ea3caf1a2bbfb273f2d https://github.com/jenkinsci/google-play-android-publisher-plugin/commit/66f4bd5e956f9827da436ea3caf1a2bbfb273f2d Author: Joe Hansche <jhans...@themeetgroup.com> Date: 2019-07-22 (Mon, 22 Jul 2019)
Changed paths: M src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/Util.java A src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/JenkinsUtil.java A src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/JenkinsUtilImpl.java Log Message: ----------- Changes for basic testability Commit: 6161ad9e2da1947d0517bec86f0d169580b0c7d7 https://github.com/jenkinsci/google-play-android-publisher-plugin/commit/6161ad9e2da1947d0517bec86f0d169580b0c7d7 Author: Joe Hansche <jhans...@themeetgroup.com> Date: 2019-07-22 (Mon, 22 Jul 2019) Changed paths: M pom.xml A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/UtilsTest.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestUtil.java Log Message: ----------- Adding basic tests for Utils Commit: be72e837faad178ad7bc4b8371a42cf8655e2e0b https://github.com/jenkinsci/google-play-android-publisher-plugin/commit/be72e837faad178ad7bc4b8371a42cf8655e2e0b Author: Joe Hansche <jhans...@themeetgroup.com> Date: 2019-07-22 (Mon, 22 Jul 2019) Changed paths: M src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/PublisherApiException.java M src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/Util.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/UtilsTest.java Log Message: ----------- Fix some bugs in PublisherApiException These bugs were surfaced while writing unit tests for Util.getPublisherErrorMessage. Commit: ee407c8f3030e7e47ca1080f0fd121ecd3c3ed8e https://github.com/jenkinsci/google-play-android-publisher-plugin/commit/ee407c8f3030e7e47ca1080f0fd121ecd3c3ed8e Author: Joe Hansche <jhans...@themeetgroup.com> Date: 2019-07-29 (Mon, 29 Jul 2019) Changed paths: M src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/Util.java A src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/AndroidUtil.java A src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/CheckedFunction.java M src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/JenkinsUtil.java R src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/JenkinsUtilImpl.java A src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/UtilsImpl.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/UtilsTest.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestUtil.java Log Message: ----------- Abstract more utils into a separate AndroidUtils This splits implementations of JenkinsUtils vs AndroidUtils. For some of the Android (APK) utility methods, that means we likely have to invoke a Jenkins util that executes remote code from the Android utils; for example, extracting package name and versionCode from a remote APK file. Commit: 32de0cc3dc228c0603af99e94d7dade9d7c6ef97 https://github.com/jenkinsci/google-play-android-publisher-plugin/commit/32de0cc3dc228c0603af99e94d7dade9d7c6ef97 Author: Joe Hansche <jhans...@themeetgroup.com> Date: 2019-07-29 (Mon, 29 Jul 2019) Changed paths: A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestHttpTransport.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakeAssignTrackResponse.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakeCommitResponse.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakeHttpResponse.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakeListApksResponse.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakePostEditsResponse.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakePutApkResponse.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakeUploadApkResponse.java Log Message: ----------- Create some basic fake HttpResponse handlers Using the built-in MockHttpTransport, we're able to create a LowLevelHttpRequest which can return a LowLevelHttpResponse. That allows us to set up a kind of test director where we can specify what response we want to see for a specific request URL. The basic full end-to-end test requires 7 HTTP requests to be mocked in order to simulate a successful upload. Commit: 9f6462285ecaa35fe7a4c98eb0b3adff9edba0de https://github.com/jenkinsci/google-play-android-publisher-plugin/commit/9f6462285ecaa35fe7a4c98eb0b3adff9edba0de Author: Joe Hansche <jhans...@themeetgroup.com> Date: 2019-07-29 (Mon, 29 Jul 2019) Changed paths: M src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/ApkPublisher.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/ApkPublisherTest.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestHttpTransport.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestUtil.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakeHttpResponse.java A src/test/resources/foo.apk Log Message: ----------- Add initial passing integration tests Commit: 4d5e742996123da3dbd74b15babf318e33630d83 https://github.com/jenkinsci/google-play-android-publisher-plugin/commit/4d5e742996123da3dbd74b15babf318e33630d83 Author: Joe Hansche <jhans...@themeetgroup.com> Date: 2019-07-29 (Mon, 29 Jul 2019) Changed paths: M src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/ReleaseTrackAssignmentBuilder.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/ApkPublisherTest.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/ReleaseTrackAssignmentTest.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/UtilsTest.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestHttpTransport.java R src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestUtil.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestUtilImpl.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestsHelper.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakeHttpResponse.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakeListApksResponse.java Log Message: ----------- Add tests for ReleaseTrackAssignment Commit: cc8cf50b88ac66ce53a1f24a1f368939c0ee61cd https://github.com/jenkinsci/google-play-android-publisher-plugin/commit/cc8cf50b88ac66ce53a1f24a1f368939c0ee61cd Author: Joe Hansche <jhans...@themeetgroup.com> Date: 2019-07-29 (Mon, 29 Jul 2019) Changed paths: M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestHttpTransport.java Log Message: ----------- Expand the mocked requests/responses to include more data Commit: 3f1ab042c34adb4371e01bc2854392ece4d49cdb https://github.com/jenkinsci/google-play-android-publisher-plugin/commit/3f1ab042c34adb4371e01bc2854392ece4d49cdb Author: Joe Hansche <jhans...@themeetgroup.com> Date: 2019-08-02 (Fri, 02 Aug 2019) Changed paths: M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/ApkPublisherTest.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestUtilImpl.java Log Message: ----------- Use TestUtil instead of mocks Commit: 4d374fea5ba70295ace39b0d9c8e38e699bf5fd4 https://github.com/jenkinsci/google-play-android-publisher-plugin/commit/4d374fea5ba70295ace39b0d9c8e38e699bf5fd4 Author: Joe Hansche <jhans...@themeetgroup.com> Date: 2019-08-02 (Fri, 02 Aug 2019) Changed paths: M pom.xml Log Message: ----------- Update Mockito Commit: 0fef6d91ce299e0ea697c031e4fbc16c06cc6b1b https://github.com/jenkinsci/google-play-android-publisher-plugin/commit/0fef6d91ce299e0ea697c031e4fbc16c06cc6b1b Author: Joe Hansche <jhans...@themeetgroup.com> Date: 2019-08-02 (Fri, 02 Aug 2019) Changed paths: M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/ApkPublisherTest.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/ReleaseTrackAssignmentTest.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestHttpTransport.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestsHelper.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/oauth/TestCredentials.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/oauth/TestCredentialsModule.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakeHttpResponse.java Log Message: ----------- Work on getting the test to run in a fake slave environment. Several objects used during the test are not serializable, which is required in order to test on a fake Slave. Commit: af950e2a625d82a00f09f955542bbb681aad9ced https://github.com/jenkinsci/google-play-android-publisher-plugin/commit/af950e2a625d82a00f09f955542bbb681aad9ced Author: Joe Hansche <jhans...@themeetgroup.com> Date: 2019-08-02 (Fri, 02 Aug 2019) Changed paths: M src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/Util.java M src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/JenkinsUtil.java M src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/UtilsImpl.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/ApkPublisherTest.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestUtilImpl.java Log Message: ----------- Remove the Jenkins.actOnPath() override It seems this wasn't actually necessary after all. Commit: 929f64b9928599ef439e762495f26f6f3e57043b https://github.com/jenkinsci/google-play-android-publisher-plugin/commit/929f64b9928599ef439e762495f26f6f3e57043b Author: Joe Hansche <jhans...@themeetgroup.com> Date: 2019-08-02 (Fri, 02 Aug 2019) Changed paths: M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/ApkPublisherTest.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestHttpTransport.java M src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakeHttpResponse.java Log Message: ----------- Serialize and recreate the HttpResponse on the slaves Commit: d23a3d38760eafb81dd3a952d4799bdf1d457071 https://github.com/jenkinsci/google-play-android-publisher-plugin/commit/d23a3d38760eafb81dd3a952d4799bdf1d457071 Author: Christopher Orr <ch...@orr.me.uk> Date: 2019-08-05 (Mon, 05 Aug 2019) Changed paths: M pom.xml M src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/ApkPublisher.java M src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/PublisherApiException.java M src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/ReleaseTrackAssignmentBuilder.java M src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/Util.java A src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/AndroidUtil.java A src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/CheckedFunction.java A src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/JenkinsUtil.java A src/main/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/UtilsImpl.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/ApkPublisherTest.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/ReleaseTrackAssignmentTest.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/UtilsTest.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestHttpTransport.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestUtilImpl.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/TestsHelper.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/oauth/TestCredentials.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/oauth/TestCredentialsModule.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakeAssignTrackResponse.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakeCommitResponse.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakeHttpResponse.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakeListApksResponse.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakePostEditsResponse.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakePutApkResponse.java A src/test/java/org/jenkinsci/plugins/googleplayandroidpublisher/internal/responses/FakeUploadApkResponse.java A src/test/resources/foo.apk Log Message: ----------- Merge pull request #14 from MeetMe/pr-testing Add unit tests for basic functionality Compare: https://github.com/jenkinsci/google-play-android-publisher-plugin/compare/b96fbdee4bbe...d23a3d38760e -- You received this message because you are subscribed to the Google Groups "Jenkins Commits" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-commits+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-commits/jenkinsci/google-play-android-publisher-plugin/push/refs/heads/master/b96fbd-d23a3d%40github.com.