{"action":"closed","issue":{"url":"https://api.github.com/repos/cpputest/cpputest/issues/476","labels_url":"https://api.github.com/repos/cpputest/cpputest/issues/476/labels{/name}","comments_url":"https://api.github.com/repos/cpputest/cpputest/issues/476/comments","events_url":"https://api.github.com/repos/cpputest/cpputest/issues/476/events","html_url":"https://github.com/cpputest/cpputest/issues/476","id":55727735,"number":476,"title":"Strange ignored test in MockSupporTest","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":"closed","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-01-28T08:33:17Z","updated_at":"2015-01-29T13:23:32Z","closed_at":"2015-01-29T13:23:32Z","body":"I was looking for ignored tests that could be fixed. I don't understand the purpose of this test:\r\n```c++\r\nIGNORE_TEST(MockSupportTest, strictOrderViolatedWithTwoMocks)\r\n{\r\n    //this test and scenario needs a decent failure message.\r\n    mock(\"mock1\").strictOrder();\r\n    mock(\"mock2\").strictOrder();\r\n    mock(\"mock1\").expectOneCall(\"foo1\");\r\n    mock(\"mock2\").expectOneCall(\"foo2\");\r\n\r\n    mock(\"mock2\").actualCall(\"foo2\");\r\n    mock(\"mock1\").actualCall(\"foo1\");\r\n\r\n    mock(\"mock1\").checkExpectations();\r\n    mock(\"mock2\").checkExpectations();\r\n    CHECK_NO_MOCK_FAILURE();\r\n}\r\n```\r\n1. The comment makes no sense to me.\r\n2. I figure that the expected behavior is for strict order to be limited to a mock scope\r\n3. The test fails, because strict order seems to be considered irrespective of scope\r\n\r\nThis would be a bug in CppUTest, which needs fixing? Or is it behaving as intended, and the test case ought to expect a failure?"}}