{"action":"created","comment":{"url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/pulls/comments/243786581","pull_request_review_id":187645323,"id":243786581,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDI0Mzc4NjU4MQ==","diff_hunk":"@@ -195,4 +200,34 @@ public void testQueryCptCase7() {\n         Assert.assertEquals(ErrorCode.UNKNOW_ERROR.getCode(), response.getErrorCode().intValue());\n         Assert.assertNull(response.getResult());\n     }\n+\n+    /**\n+     * case: mock DataTypetUtils.bytes32DynamicArrayToStringArrayWithoutTrim()\n+     *      for DataTypeCastException.DataTypeCastException()\n+     */\n+    @Test\n+    public void testQueryCptCase8() {\n+\n+        MockUp<DataTypetUtils> mockTest = new MockUp<DataTypetUtils>() {\n+            @Mock\n+            public String[] bytes32DynamicArrayToStringArrayWithoutTrim(\n+                DynamicArray<Bytes32> bytes32DynamicArray)\n+                throws DataTypeCastException {\n+                WeIdBaseException e = new WeIdBaseException(\"test.\");\n+                logger.info(\"testQueryCptCase8:{}\", e.toString());","path":"src/test/java/com/webank/weid/full/cpt/TestQueryCpt.java","position":null,"original_position":44,"commit_id":"84cd2db05381dd128efa573837e8ee0ceb6da09b","original_commit_id":"425ec923cbdfd977d285fe21442102d3e8a7ab81","user":{"login":"darwindu","id":30070842,"node_id":"MDQ6VXNlcjMwMDcwODQy","avatar_url":"https://avatars0.githubusercontent.com/u/30070842?v=4","gravatar_id":"","url":"https://api.github.com/users/darwindu","html_url":"https://github.com/darwindu","followers_url":"https://api.github.com/users/darwindu/followers","following_url":"https://api.github.com/users/darwindu/following{/other_user}","gists_url":"https://api.github.com/users/darwindu/gists{/gist_id}","starred_url":"https://api.github.com/users/darwindu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/darwindu/subscriptions","organizations_url":"https://api.github.com/users/darwindu/orgs","repos_url":"https://api.github.com/users/darwindu/repos","events_url":"https://api.github.com/users/darwindu/events{/privacy}","received_events_url":"https://api.github.com/users/darwindu/received_events","type":"User","site_admin":false},"body":"toString()方法有一个异常类的方法需要覆盖；","created_at":"2018-12-24T04:40:07Z","updated_at":"2018-12-24T04:40:07Z","html_url":"https://github.com/WeBankFinTech/weidentity-java-sdk/pull/27#discussion_r243786581","pull_request_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/pulls/27","author_association":"NONE","_links":{"self":{"href":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/pulls/comments/243786581"},"html":{"href":"https://github.com/WeBankFinTech/weidentity-java-sdk/pull/27#discussion_r243786581"},"pull_request":{"href":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/pulls/27"}},"in_reply_to_id":243784303},"pull_request":{"url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/pulls/27","id":240392722,"node_id":"MDExOlB1bGxSZXF1ZXN0MjQwMzkyNzIy","html_url":"https://github.com/WeBankFinTech/weidentity-java-sdk/pull/27","diff_url":"https://github.com/WeBankFinTech/weidentity-java-sdk/pull/27.diff","patch_url":"https://github.com/WeBankFinTech/weidentity-java-sdk/pull/27.patch","issue_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/issues/27","number":27,"state":"open","locked":false,"title":"Feature/known issues fix","user":{"login":"darwindu","id":30070842,"node_id":"MDQ6VXNlcjMwMDcwODQy","avatar_url":"https://avatars0.githubusercontent.com/u/30070842?v=4","gravatar_id":"","url":"https://api.github.com/users/darwindu","html_url":"https://github.com/darwindu","followers_url":"https://api.github.com/users/darwindu/followers","following_url":"https://api.github.com/users/darwindu/following{/other_user}","gists_url":"https://api.github.com/users/darwindu/gists{/gist_id}","starred_url":"https://api.github.com/users/darwindu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/darwindu/subscriptions","organizations_url":"https://api.github.com/users/darwindu/orgs","repos_url":"https://api.github.com/users/darwindu/repos","events_url":"https://api.github.com/users/darwindu/events{/privacy}","received_events_url":"https://api.github.com/users/darwindu/received_events","type":"User","site_admin":false},"body":"## Summary\r\n- Concisely describe the issue and your solution:\r\n  - fix issuers: add coverage;\r\n- Link to existing issue (type \"#\" to check): no\r\n- Specify the tests done so far:\r\n  - see: https://www.travis-ci.org/WeBankFinTech/weidentity-java-sdk/builds/470885740\r\n## Solutions (Optional)\r\nPlease describe the details of your solution if it might be too complicated for reviewers.\r\n","created_at":"2018-12-21T07:26:43Z","updated_at":"2018-12-24T04:40:07Z","closed_at":null,"merged_at":null,"merge_commit_sha":"65c2cd93d9aebc9df756b1f9c92130b4a8e2cc04","assignee":null,"assignees":[],"requested_reviewers":[{"login":"gmzer","id":3457149,"node_id":"MDQ6VXNlcjM0NTcxNDk=","avatar_url":"https://avatars1.githubusercontent.com/u/3457149?v=4","gravatar_id":"","url":"https://api.github.com/users/gmzer","html_url":"https://github.com/gmzer","followers_url":"https://api.github.com/users/gmzer/followers","following_url":"https://api.github.com/users/gmzer/following{/other_user}","gists_url":"https://api.github.com/users/gmzer/gists{/gist_id}","starred_url":"https://api.github.com/users/gmzer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gmzer/subscriptions","organizations_url":"https://api.github.com/users/gmzer/orgs","repos_url":"https://api.github.com/users/gmzer/repos","events_url":"https://api.github.com/users/gmzer/events{/privacy}","received_events_url":"https://api.github.com/users/gmzer/received_events","type":"User","site_admin":false},{"login":"chaoxinhu","id":44358688,"node_id":"MDQ6VXNlcjQ0MzU4Njg4","avatar_url":"https://avatars2.githubusercontent.com/u/44358688?v=4","gravatar_id":"","url":"https://api.github.com/users/chaoxinhu","html_url":"https://github.com/chaoxinhu","followers_url":"https://api.github.com/users/chaoxinhu/followers","following_url":"https://api.github.com/users/chaoxinhu/following{/other_user}","gists_url":"https://api.github.com/users/chaoxinhu/gists{/gist_id}","starred_url":"https://api.github.com/users/chaoxinhu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chaoxinhu/subscriptions","organizations_url":"https://api.github.com/users/chaoxinhu/orgs","repos_url":"https://api.github.com/users/chaoxinhu/repos","events_url":"https://api.github.com/users/chaoxinhu/events{/privacy}","received_events_url":"https://api.github.com/users/chaoxinhu/received_events","type":"User","site_admin":false}],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/pulls/27/commits","review_comments_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/pulls/27/comments","review_comment_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/pulls/comments{/number}","comments_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/issues/27/comments","statuses_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/statuses/84cd2db05381dd128efa573837e8ee0ceb6da09b","head":{"label":"darwindu:feature/known-issues-fix","ref":"feature/known-issues-fix","sha":"84cd2db05381dd128efa573837e8ee0ceb6da09b","user":{"login":"darwindu","id":30070842,"node_id":"MDQ6VXNlcjMwMDcwODQy","avatar_url":"https://avatars0.githubusercontent.com/u/30070842?v=4","gravatar_id":"","url":"https://api.github.com/users/darwindu","html_url":"https://github.com/darwindu","followers_url":"https://api.github.com/users/darwindu/followers","following_url":"https://api.github.com/users/darwindu/following{/other_user}","gists_url":"https://api.github.com/users/darwindu/gists{/gist_id}","starred_url":"https://api.github.com/users/darwindu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/darwindu/subscriptions","organizations_url":"https://api.github.com/users/darwindu/orgs","repos_url":"https://api.github.com/users/darwindu/repos","events_url":"https://api.github.com/users/darwindu/events{/privacy}","received_events_url":"https://api.github.com/users/darwindu/received_events","type":"User","site_admin":false},"repo":{"id":160162045,"node_id":"MDEwOlJlcG9zaXRvcnkxNjAxNjIwNDU=","name":"weidentity-java-sdk","full_name":"darwindu/weidentity-java-sdk","private":false,"owner":{"login":"darwindu","id":30070842,"node_id":"MDQ6VXNlcjMwMDcwODQy","avatar_url":"https://avatars0.githubusercontent.com/u/30070842?v=4","gravatar_id":"","url":"https://api.github.com/users/darwindu","html_url":"https://github.com/darwindu","followers_url":"https://api.github.com/users/darwindu/followers","following_url":"https://api.github.com/users/darwindu/following{/other_user}","gists_url":"https://api.github.com/users/darwindu/gists{/gist_id}","starred_url":"https://api.github.com/users/darwindu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/darwindu/subscriptions","organizations_url":"https://api.github.com/users/darwindu/orgs","repos_url":"https://api.github.com/users/darwindu/repos","events_url":"https://api.github.com/users/darwindu/events{/privacy}","received_events_url":"https://api.github.com/users/darwindu/received_events","type":"User","site_admin":false},"html_url":"https://github.com/darwindu/weidentity-java-sdk","description":null,"fork":true,"url":"https://api.github.com/repos/darwindu/weidentity-java-sdk","forks_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/forks","keys_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/keys{/key_id}","collaborators_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/teams","hooks_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/hooks","issue_events_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/issues/events{/number}","events_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/events","assignees_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/assignees{/user}","branches_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/branches{/branch}","tags_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/tags","blobs_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/git/refs{/sha}","trees_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/git/trees{/sha}","statuses_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/statuses/{sha}","languages_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/languages","stargazers_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/stargazers","contributors_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/contributors","subscribers_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/subscribers","subscription_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/subscription","commits_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/commits{/sha}","git_commits_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/git/commits{/sha}","comments_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/comments{/number}","issue_comment_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/issues/comments{/number}","contents_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/contents/{+path}","compare_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/compare/{base}...{head}","merges_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/merges","archive_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/downloads","issues_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/issues{/number}","pulls_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/pulls{/number}","milestones_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/milestones{/number}","notifications_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/labels{/name}","releases_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/releases{/id}","deployments_url":"https://api.github.com/repos/darwindu/weidentity-java-sdk/deployments","created_at":"2018-12-03T09:03:31Z","updated_at":"2018-12-03T09:03:33Z","pushed_at":"2018-12-24T03:47:37Z","git_url":"git://github.com/darwindu/weidentity-java-sdk.git","ssh_url":"git@github.com:darwindu/weidentity-java-sdk.git","clone_url":"https://github.com/darwindu/weidentity-java-sdk.git","svn_url":"https://github.com/darwindu/weidentity-java-sdk","homepage":null,"size":1502,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":0,"license":{"key":"lgpl-3.0","name":"GNU Lesser General Public License v3.0","spdx_id":"LGPL-3.0","url":"https://api.github.com/licenses/lgpl-3.0","node_id":"MDc6TGljZW5zZTEy"},"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"WeBankFinTech:develop","ref":"develop","sha":"b32c44a967e78391397f0bbf4cc2e2b31ba7f1b2","user":{"login":"WeBankFinTech","id":44182399,"node_id":"MDEyOk9yZ2FuaXphdGlvbjQ0MTgyMzk5","avatar_url":"https://avatars2.githubusercontent.com/u/44182399?v=4","gravatar_id":"","url":"https://api.github.com/users/WeBankFinTech","html_url":"https://github.com/WeBankFinTech","followers_url":"https://api.github.com/users/WeBankFinTech/followers","following_url":"https://api.github.com/users/WeBankFinTech/following{/other_user}","gists_url":"https://api.github.com/users/WeBankFinTech/gists{/gist_id}","starred_url":"https://api.github.com/users/WeBankFinTech/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/WeBankFinTech/subscriptions","organizations_url":"https://api.github.com/users/WeBankFinTech/orgs","repos_url":"https://api.github.com/users/WeBankFinTech/repos","events_url":"https://api.github.com/users/WeBankFinTech/events{/privacy}","received_events_url":"https://api.github.com/users/WeBankFinTech/received_events","type":"Organization","site_admin":false},"repo":{"id":155411577,"node_id":"MDEwOlJlcG9zaXRvcnkxNTU0MTE1Nzc=","name":"weidentity-java-sdk","full_name":"WeBankFinTech/weidentity-java-sdk","private":false,"owner":{"login":"WeBankFinTech","id":44182399,"node_id":"MDEyOk9yZ2FuaXphdGlvbjQ0MTgyMzk5","avatar_url":"https://avatars2.githubusercontent.com/u/44182399?v=4","gravatar_id":"","url":"https://api.github.com/users/WeBankFinTech","html_url":"https://github.com/WeBankFinTech","followers_url":"https://api.github.com/users/WeBankFinTech/followers","following_url":"https://api.github.com/users/WeBankFinTech/following{/other_user}","gists_url":"https://api.github.com/users/WeBankFinTech/gists{/gist_id}","starred_url":"https://api.github.com/users/WeBankFinTech/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/WeBankFinTech/subscriptions","organizations_url":"https://api.github.com/users/WeBankFinTech/orgs","repos_url":"https://api.github.com/users/WeBankFinTech/repos","events_url":"https://api.github.com/users/WeBankFinTech/events{/privacy}","received_events_url":"https://api.github.com/users/WeBankFinTech/received_events","type":"Organization","site_admin":false},"html_url":"https://github.com/WeBankFinTech/weidentity-java-sdk","description":null,"fork":false,"url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk","forks_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/forks","keys_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/keys{/key_id}","collaborators_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/teams","hooks_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/hooks","issue_events_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/issues/events{/number}","events_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/events","assignees_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/assignees{/user}","branches_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/branches{/branch}","tags_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/tags","blobs_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/git/refs{/sha}","trees_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/git/trees{/sha}","statuses_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/statuses/{sha}","languages_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/languages","stargazers_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/stargazers","contributors_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/contributors","subscribers_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/subscribers","subscription_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/subscription","commits_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/commits{/sha}","git_commits_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/git/commits{/sha}","comments_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/comments{/number}","issue_comment_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/issues/comments{/number}","contents_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/contents/{+path}","compare_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/compare/{base}...{head}","merges_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/merges","archive_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/downloads","issues_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/issues{/number}","pulls_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/pulls{/number}","milestones_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/milestones{/number}","notifications_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/labels{/name}","releases_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/releases{/id}","deployments_url":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/deployments","created_at":"2018-10-30T15:41:53Z","updated_at":"2018-12-20T14:21:56Z","pushed_at":"2018-12-24T03:47:38Z","git_url":"git://github.com/WeBankFinTech/weidentity-java-sdk.git","ssh_url":"git@github.com:WeBankFinTech/weidentity-java-sdk.git","clone_url":"https://github.com/WeBankFinTech/weidentity-java-sdk.git","svn_url":"https://github.com/WeBankFinTech/weidentity-java-sdk","homepage":null,"size":1477,"stargazers_count":10,"watchers_count":10,"language":"Java","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":7,"mirror_url":null,"archived":false,"open_issues_count":5,"license":{"key":"lgpl-3.0","name":"GNU Lesser General Public License v3.0","spdx_id":"LGPL-3.0","url":"https://api.github.com/licenses/lgpl-3.0","node_id":"MDc6TGljZW5zZTEy"},"forks":7,"open_issues":5,"watchers":10,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/pulls/27"},"html":{"href":"https://github.com/WeBankFinTech/weidentity-java-sdk/pull/27"},"issue":{"href":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/issues/27"},"comments":{"href":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/issues/27/comments"},"review_comments":{"href":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/pulls/27/comments"},"review_comment":{"href":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/pulls/27/commits"},"statuses":{"href":"https://api.github.com/repos/WeBankFinTech/weidentity-java-sdk/statuses/84cd2db05381dd128efa573837e8ee0ceb6da09b"}},"author_association":"NONE"}}
{"actor":{"display_login":"darwindu"}}