{"action":"opened","issue":{"url":"https://api.github.com/repos/hcoles/pitest/issues/205","labels_url":"https://api.github.com/repos/hcoles/pitest/issues/205/labels{/name}","comments_url":"https://api.github.com/repos/hcoles/pitest/issues/205/comments","events_url":"https://api.github.com/repos/hcoles/pitest/issues/205/events","html_url":"https://github.com/hcoles/pitest/issues/205","id":77585261,"number":205,"title":"NON_VOID_METHOD_CALLS Mutator is buggy","user":{"login":"lkwg82","id":1891171,"avatar_url":"https://avatars.githubusercontent.com/u/1891171?v=3","gravatar_id":"","url":"https://api.github.com/users/lkwg82","html_url":"https://github.com/lkwg82","followers_url":"https://api.github.com/users/lkwg82/followers","following_url":"https://api.github.com/users/lkwg82/following{/other_user}","gists_url":"https://api.github.com/users/lkwg82/gists{/gist_id}","starred_url":"https://api.github.com/users/lkwg82/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lkwg82/subscriptions","organizations_url":"https://api.github.com/users/lkwg82/orgs","repos_url":"https://api.github.com/users/lkwg82/repos","events_url":"https://api.github.com/users/lkwg82/events{/privacy}","received_events_url":"https://api.github.com/users/lkwg82/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-05-18T09:42:52Z","updated_at":"2015-05-18T09:42:52Z","closed_at":null,"body":"I think the `NON_VOID_METHOD_CALLS`-mutator is kind of buggy.\r\n\r\nFor this class \r\n\r\n```java\r\npackage de.lgohlke.pitxest;\r\n\r\nimport java.util.concurrent.ExecutorService;\r\nimport java.util.concurrent.Executors;\r\n\r\npublic class RemovedCallBug {\r\n    private final ExecutorService executorService = Executors.newFixedThreadPool(1);\r\n\r\n    boolean doSomething() throws InterruptedException {\r\n        return executorService != null;\r\n    }\r\n}\r\n```\r\n\r\nI expect the mutation:\r\n\r\n```java\r\npackage de.lgohlke.pitxest;\r\n\r\nimport java.util.concurrent.ExecutorService;\r\nimport java.util.concurrent.Executors;\r\n\r\npublic class RemovedCallBug {\r\n    private final ExecutorService executorService = null;\r\n\r\n    boolean doSomething() throws InterruptedException {\r\n        return executorService != null;\r\n    }\r\n}\r\n```\r\n\r\nwith this test\r\n\r\n```java\r\npackage de.lgohlke.pitxest;\r\n\r\nimport org.junit.Test;\r\n\r\nimport static org.assertj.core.api.Assertions.assertThat;\r\n\r\npublic class RemovedCallBugTest {\r\n\r\n    @Test\r\n    public void test() throws InterruptedException {\r\n        assertThat(new RemovedCallBug().doSomething()).isTrue();\r\n    }\r\n}\r\n```\r\n\r\nthe reports looks this way\r\n\r\n1. removed call to java/util/concurrent/Executors::newFixedThreadPool → NO_COVERAGE\r\n\r\nan I dont agree ;)\r\n\r\nDid I oversee someting?"}}