{"action":"created","issue":{"url":"https://api.github.com/repos/cpputest/cpputest/issues/479","labels_url":"https://api.github.com/repos/cpputest/cpputest/issues/479/labels{/name}","comments_url":"https://api.github.com/repos/cpputest/cpputest/issues/479/comments","events_url":"https://api.github.com/repos/cpputest/cpputest/issues/479/events","html_url":"https://github.com/cpputest/cpputest/issues/479","id":55898832,"number":479,"title":"Cleaning up ignored tests.","user":{"login":"arstrube","id":4686469,"avatar_url":"https://avatars.githubusercontent.com/u/4686469?v=3","gravatar_id":"","url":"https://api.github.com/users/arstrube","html_url":"https://github.com/arstrube","followers_url":"https://api.github.com/users/arstrube/followers","following_url":"https://api.github.com/users/arstrube/following{/other_user}","gists_url":"https://api.github.com/users/arstrube/gists{/gist_id}","starred_url":"https://api.github.com/users/arstrube/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/arstrube/subscriptions","organizations_url":"https://api.github.com/users/arstrube/orgs","repos_url":"https://api.github.com/users/arstrube/repos","events_url":"https://api.github.com/users/arstrube/events{/privacy}","received_events_url":"https://api.github.com/users/arstrube/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":17,"created_at":"2015-01-29T13:30:59Z","updated_at":"2015-01-29T14:11:51Z","closed_at":null,"body":"I noticed there is a certain class of ignored tests that test whether something does or doesn't work inside an ignored test. The following seems as good an example of this as any:\r\n```c++\r\nIGNORE_TEST(UtestShell, IgnoreTestAccessingFixture)\r\n{\r\n    CHECK(&fixture != 0);\r\n}\r\n```\r\nI find it hard to see the significance of such tests, since the test body is never run, so the code will never be executed.... except, if by some happenstance, it would execute unintentionally.\r\n\r\nBut then, we wouldn't need to test all kind of things in ignored tests, but we would only need this one test, something like:\r\n```c++\r\nIGNORE_TEST(UtestShell, IgnoreTestIsNotRun)\r\n{\r\n    FAIL(\"Code inside ignored test was executed.\");\r\n}\r\n```\r\nIf this test fails, something is very wrong indeed. Otherwise, there is no reason to \"test\" code in ignored tests?"},"comment":{"url":"https://api.github.com/repos/cpputest/cpputest/issues/comments/72030485","html_url":"https://github.com/cpputest/cpputest/issues/479#issuecomment-72030485","issue_url":"https://api.github.com/repos/cpputest/cpputest/issues/479","id":72030485,"user":{"login":"Andne","id":3787585,"avatar_url":"https://avatars.githubusercontent.com/u/3787585?v=3","gravatar_id":"","url":"https://api.github.com/users/Andne","html_url":"https://github.com/Andne","followers_url":"https://api.github.com/users/Andne/followers","following_url":"https://api.github.com/users/Andne/following{/other_user}","gists_url":"https://api.github.com/users/Andne/gists{/gist_id}","starred_url":"https://api.github.com/users/Andne/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Andne/subscriptions","organizations_url":"https://api.github.com/users/Andne/orgs","repos_url":"https://api.github.com/users/Andne/repos","events_url":"https://api.github.com/users/Andne/events{/privacy}","received_events_url":"https://api.github.com/users/Andne/received_events","type":"User","site_admin":false},"created_at":"2015-01-29T14:11:51Z","updated_at":"2015-01-29T14:11:51Z","body":"We can test IgnoredUtestShell without creating tests that show up in the main results, and put the CHECK functions that would fail inside them still.  So it seems like all we are testing here is that that IGNORE_TEST macro actually creates an IgnoredUtestShell instead of a normal UtestShell."}}