{"action":"created","comment":{"url":"https://api.github.com/repos/kiegroup/drools/pulls/comments/105955514","pull_request_review_id":26857085,"id":105955514,"diff_hunk":"@@ -103,36 +109,35 @@ private void populateKieSession(KieSession kieSession, List<String> list) {\n     }\n \n     private Person[] prepareData() {\n-        Person[] persons = new Person[4];\n-        persons[0] = new Person(\"Bruno\", 25);\n-        persons[0].setAddress(new InternationalAddress(\"Some Street\", 10, \"Nice City\", \"Safecountry\"));\n+        Person bruno = new Person(\"Bruno\", 25);\n+        bruno.setAddress(new InternationalAddress(\"Some Street\", 10, \"Nice City\", \"Safecountry\"));\n \n-        persons[1] = new Person(\"Robert\", 17);\n-        persons[1].setAddress(new InternationalAddress(\"Some Street\", 12, \"Small City\", \"Riskyland\"));\n+        Person robert = new Person(\"Robert\", 17);\n+        robert.setAddress(new InternationalAddress(\"Some Street\", 12, \"Small City\", \"Riskyland\"));\n \n-        persons[2] = new Person(\"Joe\", 11);\n-        persons[2].setAddress(new InternationalAddress(\"Some Street\", 13, \"Big City\", \"Safecountry\"));\n+        Person joe = new Person(\"Joe\", 11);\n+        joe.setAddress(new InternationalAddress(\"Some Street\", 13, \"Big City\", \"Safecountry\"));\n \n-        persons[3] = new Person(\"Mike\", 25);\n-        persons[3].setAddress(new Address(\"Some Street\", 14, \"Local City\"));\n+        Person mike = new Person(\"Mike\", 25);\n+        mike.setAddress(new Address(\"Some Street\", 14, \"Local City\"));\n \n-        return persons;\n+        return new Person[]{bruno, robert, joe, mike};\n     }\n \n     private void verifyQueryResults(QueryResults results) {\n         assertThat(results).isNotEmpty();\n         final QueryResultsRow resultsRow = results.iterator().next();\n         assertThat(resultsRow.get(\"$cities\")).isInstanceOf(List.class);\n         final List<String> cities = (List<String>) resultsRow.get(\"$cities\");\n-        assertThat(cities).hasSize(2);\n-        assertThat(cities).hasOnlyElementsOfType(String.class);\n-        assertThat(cities).containsAll(Arrays.asList(\"Nice City\", \"Big City\"));\n-        assertThat(cities).doesNotContainAnyElementsOf(Arrays.asList(\"Small City\", \"Local City\"));\n+        assertThat(cities).hasSize(2)\n+                .hasOnlyElementsOfType(String.class)\n+                .containsAll(Arrays.asList(\"Nice City\", \"Big City\"))\n+                .doesNotContain(\"Small City\", \"Local City\");\n     }\n \n     private void verifyRuleFireResults(List<String> list) {\n-        assertThat(list.size()).isEqualTo(2);\n-        assertThat(list).containsOnly(\"SafeDriver\", \"Risky Driver\");\n-        assertThat(list).containsAll(Arrays.asList(\"SafeDriver\", \"Risky Driver\"));\n+        assertThat(list).hasSize(2)\n+                .containsOnly(\"SafeDriver\", \"Risky Driver\")\n+                .containsAll(Arrays.asList(\"SafeDriver\", \"Risky Driver\"));","path":"drools-test-coverage/test-suite/src/test/java/org/drools/testcoverage/functional/oopath/OOPathDtablesTest.java","position":null,"original_position":83,"commit_id":"4a5f18ff3064ae5bcee42328703b5a7425462dce","original_commit_id":"4a5f18ff3064ae5bcee42328703b5a7425462dce","user":{"login":"triceo","id":247403,"avatar_url":"https://avatars3.githubusercontent.com/u/247403?v=3","gravatar_id":"","url":"https://api.github.com/users/triceo","html_url":"https://github.com/triceo","followers_url":"https://api.github.com/users/triceo/followers","following_url":"https://api.github.com/users/triceo/following{/other_user}","gists_url":"https://api.github.com/users/triceo/gists{/gist_id}","starred_url":"https://api.github.com/users/triceo/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/triceo/subscriptions","organizations_url":"https://api.github.com/users/triceo/orgs","repos_url":"https://api.github.com/users/triceo/repos","events_url":"https://api.github.com/users/triceo/events{/privacy}","received_events_url":"https://api.github.com/users/triceo/received_events","type":"User","site_admin":false},"body":"I think the second `contains` is redundant, isn't it?`","created_at":"2017-03-14T16:19:36Z","updated_at":"2017-03-14T16:20:49Z","html_url":"https://github.com/kiegroup/drools/pull/1147#discussion_r105955514","pull_request_url":"https://api.github.com/repos/kiegroup/drools/pulls/1147","_links":{"self":{"href":"https://api.github.com/repos/kiegroup/drools/pulls/comments/105955514"},"html":{"href":"https://github.com/kiegroup/drools/pull/1147#discussion_r105955514"},"pull_request":{"href":"https://api.github.com/repos/kiegroup/drools/pulls/1147"}}},"pull_request":{"url":"https://api.github.com/repos/kiegroup/drools/pulls/1147","id":110659482,"html_url":"https://github.com/kiegroup/drools/pull/1147","diff_url":"https://github.com/kiegroup/drools/pull/1147.diff","patch_url":"https://github.com/kiegroup/drools/pull/1147.patch","issue_url":"https://api.github.com/repos/kiegroup/drools/issues/1147","number":1147,"state":"open","locked":false,"title":"DROOLS-1379 Oopath test dtables","user":{"login":"cimbalek","id":12131154,"avatar_url":"https://avatars3.githubusercontent.com/u/12131154?v=3","gravatar_id":"","url":"https://api.github.com/users/cimbalek","html_url":"https://github.com/cimbalek","followers_url":"https://api.github.com/users/cimbalek/followers","following_url":"https://api.github.com/users/cimbalek/following{/other_user}","gists_url":"https://api.github.com/users/cimbalek/gists{/gist_id}","starred_url":"https://api.github.com/users/cimbalek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cimbalek/subscriptions","organizations_url":"https://api.github.com/users/cimbalek/orgs","repos_url":"https://api.github.com/users/cimbalek/repos","events_url":"https://api.github.com/users/cimbalek/events{/privacy}","received_events_url":"https://api.github.com/users/cimbalek/received_events","type":"User","site_admin":false},"body":"","created_at":"2017-03-14T16:17:35Z","updated_at":"2017-03-14T16:20:49Z","closed_at":null,"merged_at":null,"merge_commit_sha":"152161365204a33b0de198b39f8b461bcd9f7a80","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/kiegroup/drools/pulls/1147/commits","review_comments_url":"https://api.github.com/repos/kiegroup/drools/pulls/1147/comments","review_comment_url":"https://api.github.com/repos/kiegroup/drools/pulls/comments{/number}","comments_url":"https://api.github.com/repos/kiegroup/drools/issues/1147/comments","statuses_url":"https://api.github.com/repos/kiegroup/drools/statuses/4a5f18ff3064ae5bcee42328703b5a7425462dce","head":{"label":"cimbalek:oopath-test-dtables","ref":"oopath-test-dtables","sha":"4a5f18ff3064ae5bcee42328703b5a7425462dce","user":{"login":"cimbalek","id":12131154,"avatar_url":"https://avatars3.githubusercontent.com/u/12131154?v=3","gravatar_id":"","url":"https://api.github.com/users/cimbalek","html_url":"https://github.com/cimbalek","followers_url":"https://api.github.com/users/cimbalek/followers","following_url":"https://api.github.com/users/cimbalek/following{/other_user}","gists_url":"https://api.github.com/users/cimbalek/gists{/gist_id}","starred_url":"https://api.github.com/users/cimbalek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cimbalek/subscriptions","organizations_url":"https://api.github.com/users/cimbalek/orgs","repos_url":"https://api.github.com/users/cimbalek/repos","events_url":"https://api.github.com/users/cimbalek/events{/privacy}","received_events_url":"https://api.github.com/users/cimbalek/received_events","type":"User","site_admin":false},"repo":{"id":65294701,"name":"drools","full_name":"cimbalek/drools","owner":{"login":"cimbalek","id":12131154,"avatar_url":"https://avatars3.githubusercontent.com/u/12131154?v=3","gravatar_id":"","url":"https://api.github.com/users/cimbalek","html_url":"https://github.com/cimbalek","followers_url":"https://api.github.com/users/cimbalek/followers","following_url":"https://api.github.com/users/cimbalek/following{/other_user}","gists_url":"https://api.github.com/users/cimbalek/gists{/gist_id}","starred_url":"https://api.github.com/users/cimbalek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cimbalek/subscriptions","organizations_url":"https://api.github.com/users/cimbalek/orgs","repos_url":"https://api.github.com/users/cimbalek/repos","events_url":"https://api.github.com/users/cimbalek/events{/privacy}","received_events_url":"https://api.github.com/users/cimbalek/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/cimbalek/drools","description":"Drools Expert is the rule engine and Drools Fusion does complex event processing (CEP).","fork":true,"url":"https://api.github.com/repos/cimbalek/drools","forks_url":"https://api.github.com/repos/cimbalek/drools/forks","keys_url":"https://api.github.com/repos/cimbalek/drools/keys{/key_id}","collaborators_url":"https://api.github.com/repos/cimbalek/drools/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/cimbalek/drools/teams","hooks_url":"https://api.github.com/repos/cimbalek/drools/hooks","issue_events_url":"https://api.github.com/repos/cimbalek/drools/issues/events{/number}","events_url":"https://api.github.com/repos/cimbalek/drools/events","assignees_url":"https://api.github.com/repos/cimbalek/drools/assignees{/user}","branches_url":"https://api.github.com/repos/cimbalek/drools/branches{/branch}","tags_url":"https://api.github.com/repos/cimbalek/drools/tags","blobs_url":"https://api.github.com/repos/cimbalek/drools/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/cimbalek/drools/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/cimbalek/drools/git/refs{/sha}","trees_url":"https://api.github.com/repos/cimbalek/drools/git/trees{/sha}","statuses_url":"https://api.github.com/repos/cimbalek/drools/statuses/{sha}","languages_url":"https://api.github.com/repos/cimbalek/drools/languages","stargazers_url":"https://api.github.com/repos/cimbalek/drools/stargazers","contributors_url":"https://api.github.com/repos/cimbalek/drools/contributors","subscribers_url":"https://api.github.com/repos/cimbalek/drools/subscribers","subscription_url":"https://api.github.com/repos/cimbalek/drools/subscription","commits_url":"https://api.github.com/repos/cimbalek/drools/commits{/sha}","git_commits_url":"https://api.github.com/repos/cimbalek/drools/git/commits{/sha}","comments_url":"https://api.github.com/repos/cimbalek/drools/comments{/number}","issue_comment_url":"https://api.github.com/repos/cimbalek/drools/issues/comments{/number}","contents_url":"https://api.github.com/repos/cimbalek/drools/contents/{+path}","compare_url":"https://api.github.com/repos/cimbalek/drools/compare/{base}...{head}","merges_url":"https://api.github.com/repos/cimbalek/drools/merges","archive_url":"https://api.github.com/repos/cimbalek/drools/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/cimbalek/drools/downloads","issues_url":"https://api.github.com/repos/cimbalek/drools/issues{/number}","pulls_url":"https://api.github.com/repos/cimbalek/drools/pulls{/number}","milestones_url":"https://api.github.com/repos/cimbalek/drools/milestones{/number}","notifications_url":"https://api.github.com/repos/cimbalek/drools/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/cimbalek/drools/labels{/name}","releases_url":"https://api.github.com/repos/cimbalek/drools/releases{/id}","deployments_url":"https://api.github.com/repos/cimbalek/drools/deployments","created_at":"2016-08-09T12:53:24Z","updated_at":"2016-08-09T12:53:41Z","pushed_at":"2017-03-14T16:12:43Z","git_url":"git://github.com/cimbalek/drools.git","ssh_url":"git@github.com:cimbalek/drools.git","clone_url":"https://github.com/cimbalek/drools.git","svn_url":"https://github.com/cimbalek/drools","homepage":"http://www.drools.org","size":142579,"stargazers_count":0,"watchers_count":0,"language":"Java","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"kiegroup:master","ref":"master","sha":"7d7087a911f51c85040eb7f10689d87c07126cf6","user":{"login":"kiegroup","id":517980,"avatar_url":"https://avatars0.githubusercontent.com/u/517980?v=3","gravatar_id":"","url":"https://api.github.com/users/kiegroup","html_url":"https://github.com/kiegroup","followers_url":"https://api.github.com/users/kiegroup/followers","following_url":"https://api.github.com/users/kiegroup/following{/other_user}","gists_url":"https://api.github.com/users/kiegroup/gists{/gist_id}","starred_url":"https://api.github.com/users/kiegroup/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kiegroup/subscriptions","organizations_url":"https://api.github.com/users/kiegroup/orgs","repos_url":"https://api.github.com/users/kiegroup/repos","events_url":"https://api.github.com/users/kiegroup/events{/privacy}","received_events_url":"https://api.github.com/users/kiegroup/received_events","type":"Organization","site_admin":false},"repo":{"id":1381673,"name":"drools","full_name":"kiegroup/drools","owner":{"login":"kiegroup","id":517980,"avatar_url":"https://avatars0.githubusercontent.com/u/517980?v=3","gravatar_id":"","url":"https://api.github.com/users/kiegroup","html_url":"https://github.com/kiegroup","followers_url":"https://api.github.com/users/kiegroup/followers","following_url":"https://api.github.com/users/kiegroup/following{/other_user}","gists_url":"https://api.github.com/users/kiegroup/gists{/gist_id}","starred_url":"https://api.github.com/users/kiegroup/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kiegroup/subscriptions","organizations_url":"https://api.github.com/users/kiegroup/orgs","repos_url":"https://api.github.com/users/kiegroup/repos","events_url":"https://api.github.com/users/kiegroup/events{/privacy}","received_events_url":"https://api.github.com/users/kiegroup/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/kiegroup/drools","description":"Drools Expert is the rule engine and Drools Fusion does complex event processing (CEP).","fork":false,"url":"https://api.github.com/repos/kiegroup/drools","forks_url":"https://api.github.com/repos/kiegroup/drools/forks","keys_url":"https://api.github.com/repos/kiegroup/drools/keys{/key_id}","collaborators_url":"https://api.github.com/repos/kiegroup/drools/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/kiegroup/drools/teams","hooks_url":"https://api.github.com/repos/kiegroup/drools/hooks","issue_events_url":"https://api.github.com/repos/kiegroup/drools/issues/events{/number}","events_url":"https://api.github.com/repos/kiegroup/drools/events","assignees_url":"https://api.github.com/repos/kiegroup/drools/assignees{/user}","branches_url":"https://api.github.com/repos/kiegroup/drools/branches{/branch}","tags_url":"https://api.github.com/repos/kiegroup/drools/tags","blobs_url":"https://api.github.com/repos/kiegroup/drools/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/kiegroup/drools/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/kiegroup/drools/git/refs{/sha}","trees_url":"https://api.github.com/repos/kiegroup/drools/git/trees{/sha}","statuses_url":"https://api.github.com/repos/kiegroup/drools/statuses/{sha}","languages_url":"https://api.github.com/repos/kiegroup/drools/languages","stargazers_url":"https://api.github.com/repos/kiegroup/drools/stargazers","contributors_url":"https://api.github.com/repos/kiegroup/drools/contributors","subscribers_url":"https://api.github.com/repos/kiegroup/drools/subscribers","subscription_url":"https://api.github.com/repos/kiegroup/drools/subscription","commits_url":"https://api.github.com/repos/kiegroup/drools/commits{/sha}","git_commits_url":"https://api.github.com/repos/kiegroup/drools/git/commits{/sha}","comments_url":"https://api.github.com/repos/kiegroup/drools/comments{/number}","issue_comment_url":"https://api.github.com/repos/kiegroup/drools/issues/comments{/number}","contents_url":"https://api.github.com/repos/kiegroup/drools/contents/{+path}","compare_url":"https://api.github.com/repos/kiegroup/drools/compare/{base}...{head}","merges_url":"https://api.github.com/repos/kiegroup/drools/merges","archive_url":"https://api.github.com/repos/kiegroup/drools/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/kiegroup/drools/downloads","issues_url":"https://api.github.com/repos/kiegroup/drools/issues{/number}","pulls_url":"https://api.github.com/repos/kiegroup/drools/pulls{/number}","milestones_url":"https://api.github.com/repos/kiegroup/drools/milestones{/number}","notifications_url":"https://api.github.com/repos/kiegroup/drools/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/kiegroup/drools/labels{/name}","releases_url":"https://api.github.com/repos/kiegroup/drools/releases{/id}","deployments_url":"https://api.github.com/repos/kiegroup/drools/deployments","created_at":"2011-02-18T08:37:05Z","updated_at":"2017-03-14T09:08:31Z","pushed_at":"2017-03-14T16:20:11Z","git_url":"git://github.com/kiegroup/drools.git","ssh_url":"git@github.com:kiegroup/drools.git","clone_url":"https://github.com/kiegroup/drools.git","svn_url":"https://github.com/kiegroup/drools","homepage":"http://www.drools.org","size":142759,"stargazers_count":828,"watchers_count":828,"language":"Java","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":1030,"mirror_url":null,"open_issues_count":11,"forks":1030,"open_issues":11,"watchers":828,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/kiegroup/drools/pulls/1147"},"html":{"href":"https://github.com/kiegroup/drools/pull/1147"},"issue":{"href":"https://api.github.com/repos/kiegroup/drools/issues/1147"},"comments":{"href":"https://api.github.com/repos/kiegroup/drools/issues/1147/comments"},"review_comments":{"href":"https://api.github.com/repos/kiegroup/drools/pulls/1147/comments"},"review_comment":{"href":"https://api.github.com/repos/kiegroup/drools/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/kiegroup/drools/pulls/1147/commits"},"statuses":{"href":"https://api.github.com/repos/kiegroup/drools/statuses/4a5f18ff3064ae5bcee42328703b5a7425462dce"}}}}
{"actor":{"display_login":"triceo"}}