{"action":"created","issue":{"url":"https://api.github.com/repos/nunit/nunit-console/issues/24","repository_url":"https://api.github.com/repos/nunit/nunit-console","labels_url":"https://api.github.com/repos/nunit/nunit-console/issues/24/labels{/name}","comments_url":"https://api.github.com/repos/nunit/nunit-console/issues/24/comments","events_url":"https://api.github.com/repos/nunit/nunit-console/issues/24/events","html_url":"https://github.com/nunit/nunit-console/issues/24","id":171274471,"number":24,"title":"Update --labels switch with option to show real-time pass/fail results in console runner","user":{"login":"CharliePoole","id":2678661,"avatar_url":"https://avatars.githubusercontent.com/u/2678661?v=3","gravatar_id":"","url":"https://api.github.com/users/CharliePoole","html_url":"https://github.com/CharliePoole","followers_url":"https://api.github.com/users/CharliePoole/followers","following_url":"https://api.github.com/users/CharliePoole/following{/other_user}","gists_url":"https://api.github.com/users/CharliePoole/gists{/gist_id}","starred_url":"https://api.github.com/users/CharliePoole/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/CharliePoole/subscriptions","organizations_url":"https://api.github.com/users/CharliePoole/orgs","repos_url":"https://api.github.com/users/CharliePoole/repos","events_url":"https://api.github.com/users/CharliePoole/events{/privacy}","received_events_url":"https://api.github.com/users/CharliePoole/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/nunit/nunit-console/labels/easyfix","name":"easyfix","color":"128A0C"},{"url":"https://api.github.com/repos/nunit/nunit-console/labels/is:enhancement","name":"is:enhancement","color":"84b6eb"},{"url":"https://api.github.com/repos/nunit/nunit-console/labels/pri:normal","name":"pri:normal","color":"fbca04"}],"state":"open","locked":false,"assignee":{"login":"constructor-igor","id":1849690,"avatar_url":"https://avatars.githubusercontent.com/u/1849690?v=3","gravatar_id":"","url":"https://api.github.com/users/constructor-igor","html_url":"https://github.com/constructor-igor","followers_url":"https://api.github.com/users/constructor-igor/followers","following_url":"https://api.github.com/users/constructor-igor/following{/other_user}","gists_url":"https://api.github.com/users/constructor-igor/gists{/gist_id}","starred_url":"https://api.github.com/users/constructor-igor/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/constructor-igor/subscriptions","organizations_url":"https://api.github.com/users/constructor-igor/orgs","repos_url":"https://api.github.com/users/constructor-igor/repos","events_url":"https://api.github.com/users/constructor-igor/events{/privacy}","received_events_url":"https://api.github.com/users/constructor-igor/received_events","type":"User","site_admin":false},"assignees":[{"login":"constructor-igor","id":1849690,"avatar_url":"https://avatars.githubusercontent.com/u/1849690?v=3","gravatar_id":"","url":"https://api.github.com/users/constructor-igor","html_url":"https://github.com/constructor-igor","followers_url":"https://api.github.com/users/constructor-igor/followers","following_url":"https://api.github.com/users/constructor-igor/following{/other_user}","gists_url":"https://api.github.com/users/constructor-igor/gists{/gist_id}","starred_url":"https://api.github.com/users/constructor-igor/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/constructor-igor/subscriptions","organizations_url":"https://api.github.com/users/constructor-igor/orgs","repos_url":"https://api.github.com/users/constructor-igor/repos","events_url":"https://api.github.com/users/constructor-igor/events{/privacy}","received_events_url":"https://api.github.com/users/constructor-igor/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/nunit/nunit-console/milestones/1","html_url":"https://github.com/nunit/nunit-console/milestones/3.5","labels_url":"https://api.github.com/repos/nunit/nunit-console/milestones/1/labels","id":1970460,"number":1,"title":"3.5","description":"","creator":{"login":"CharliePoole","id":2678661,"avatar_url":"https://avatars.githubusercontent.com/u/2678661?v=3","gravatar_id":"","url":"https://api.github.com/users/CharliePoole","html_url":"https://github.com/CharliePoole","followers_url":"https://api.github.com/users/CharliePoole/followers","following_url":"https://api.github.com/users/CharliePoole/following{/other_user}","gists_url":"https://api.github.com/users/CharliePoole/gists{/gist_id}","starred_url":"https://api.github.com/users/CharliePoole/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/CharliePoole/subscriptions","organizations_url":"https://api.github.com/users/CharliePoole/orgs","repos_url":"https://api.github.com/users/CharliePoole/repos","events_url":"https://api.github.com/users/CharliePoole/events{/privacy}","received_events_url":"https://api.github.com/users/CharliePoole/received_events","type":"User","site_admin":false},"open_issues":8,"closed_issues":6,"state":"open","created_at":"2016-08-29T19:03:56Z","updated_at":"2016-09-26T18:49:59Z","due_on":"2016-09-30T00:00:00Z","closed_at":null},"comments":11,"created_at":"2016-08-15T22:14:09Z","updated_at":"2016-09-27T04:05:56Z","closed_at":null,"body":"@dybs commented on [Fri Aug 12 2016](https://github.com/nunit/nunit/issues/1738)\n\nWhen running a large number of test cases, it would be nice to see the pass/fail status of each case in the console as it completes.  Using --labels=All, I can see which tests have run, but the results are not available until all cases have completed.  Seeing which cases fail during a long scenario would allow me to investigate the failures while the remaining tests continue to run.  It also has the advantage of knowing which cases passed/failed if the test scenario happens to crash and doesn't get to write the result file.\r\n\r\nPer the discussion in #1735 some ideas are to modify --labels with the following options:\r\n- Results -> same behavior as =All, but includes a Pass/Fail/Error status\r\n- Before -> shows the test case before it executes, and appends the status after the test completes.  Similar to On.\r\n- After -> shows the test case and status together after the test case completes.\n\n---\n\n@CharliePoole commented on [Fri Aug 12 2016](https://github.com/nunit/nunit/issues/1738#issuecomment-239549055)\n\nI like this idea, but I would modify/simplify it by preference:\r\n\r\n* Off No change\r\n* On No change\r\n* Before / All shows test label when the test starts executing.\r\n* After Shows all tests after they have completed, including status.\r\n\r\nNote that All is described as it is supposed to work already. There's currently a bug in that it runs at the end of a test. We should fix that. We should keep All so as not to break existing usage.\r\n\r\nThe introduction of After means that we have to treat it as also meaning On. That is, if a test produces output, there should be a label before that output and another one after it, together with the test status..\r\n\r\nThe change should be made in both the console runner and nunitlite. Once we split the repos, this issue will need to be duplicated.\r\n\r\nThe fix is _relatively_ easy. Depending on who takes it, I can mentor them as needed.\n\n---\n\n@constructor-igor commented on [Sun Aug 14 2016](https://github.com/nunit/nunit/issues/1738#issuecomment-239676310)\n\nHi @CharliePoole,\r\n\r\nProbably, I can start to work on the issue.\r\nI checked all existing options of \"--labels\" parameter. Please, could you help me to understand suggested changes? I added sample with \"--lablels=All\". As I understand, the issue suggests to add \"status\" of each test?\r\n\r\n![image](https://cloud.githubusercontent.com/assets/1849690/17649703/de4f1d40-6243-11e6-93bf-b165b80bdb28.png)\r\n\r\nthank you,\r\nIgor.\n\n---\n\n@CharliePoole commented on [Sun Aug 14 2016](https://github.com/nunit/nunit/issues/1738#issuecomment-239684890)\n\nHi @constructor-igor - I've been meaning to poke you to see what you are up to. :-)\r\n\r\nAs you probably figured out, all the code is in Console's TestEventHandler and in NUnitLite's TestRunner. There are tests for TestEventHandler but not for TestRunner. :-(\r\n\r\nI would first off fix the problem with All, which is supposed to be issued at the start of each test, not at the end. I think I broke this when I was fixing something else.\r\n\r\nOnce All is correct, you've got Before, since it's intended to work the same way.\r\n\r\nAfter is the new thing, which is supposed to show status of the test. For that, I would use a combination of the `result` and `label` attributes to display a set of values like...\r\n * Passed\r\n * Failed\r\n * Error\r\n * Invalid\r\n * Ignored\r\n * Explicit\r\n * Skipped\r\n * Inconclusive\r\nIn NUnitLite, you'll have the ResultState available to use for equivalent logic.\n\n---\n\n@constructor-igor commented on [Mon Aug 15 2016](https://github.com/nunit/nunit/issues/1738#issuecomment-239782084)\n\nI am going to start from \"All\".\r\nBut, right now, I don't understand expected behavior of \"All\".\r\nSo, when I'll investigate code I'll contact with you.\n\n---\n\n@CharliePoole commented on [Mon Aug 15 2016](https://github.com/nunit/nunit/issues/1738#issuecomment-239783383)\n\nYou might try this sequence - a bit different order from my first suggestion:\r\n * Change \"All\" in existing code everywhere to \"After\" because All is now incorrectly running after the test.\r\n * Add status info to After\r\n * Add \"Before\" to run before the test\r\n * Add back \"All\" as a synyonym for Before\n\n---\n\n@constructor-igor commented on [Mon Aug 15 2016](https://github.com/nunit/nunit/issues/1738#issuecomment-239785294)\n\nI try to build nunit by \"build.cmd\":\r\n![image](https://cloud.githubusercontent.com/assets/1849690/17663823/258f1ebc-62f8-11e6-8f20-c85dc33fc293.png)\r\n\r\nVS2013:\r\n![image](https://cloud.githubusercontent.com/assets/1849690/17664078/1d756e14-62fa-11e6-924f-8ecf9bdee198.png)\r\n\n\n---\n\n@CharliePoole commented on [Mon Aug 15 2016](https://github.com/nunit/nunit/issues/1738#issuecomment-239790081)\n\nAre you using the latest master? Can you build in the IDE?\n\n---\n\n@constructor-igor commented on [Mon Aug 15 2016](https://github.com/nunit/nunit/issues/1738#issuecomment-239796767)\n\nI cloned today and tried to build via build.cmd and IDE VS2013: same errors.\n\n---\n\n@CharliePoole commented on [Mon Aug 15 2016](https://github.com/nunit/nunit/issues/1738#issuecomment-239798656)\n\nUsing VS 2015?\n\n---\n\n@rprouse commented on [Mon Aug 15 2016](https://github.com/nunit/nunit/issues/1738#issuecomment-239803785)\n\nI just got latest from master and ran `.\\build.cmd` and it builds fine for me. Maybe do a full clean including deleting your packages folder and rebuild?\n\n---\n\n@constructor-igor commented on [Mon Aug 15 2016](https://github.com/nunit/nunit/issues/1738#issuecomment-239894381)\n\nI could build nunit solution on other computer.\r\nI tried to build on my computer again (downloaded master to new folder) and  see the error.\r\n\r\nI investigate the issue and found, probably, same issues \r\n- https://github.com/nunit/nunit3-vs-adapter/issues/194\r\n- https://github.com/nunit/docs/issues/44\r\n\r\nI continue my investigation.\n\n"},"comment":{"url":"https://api.github.com/repos/nunit/nunit-console/issues/comments/249761228","html_url":"https://github.com/nunit/nunit-console/issues/24#issuecomment-249761228","issue_url":"https://api.github.com/repos/nunit/nunit-console/issues/24","id":249761228,"user":{"login":"CharliePoole","id":2678661,"avatar_url":"https://avatars.githubusercontent.com/u/2678661?v=3","gravatar_id":"","url":"https://api.github.com/users/CharliePoole","html_url":"https://github.com/CharliePoole","followers_url":"https://api.github.com/users/CharliePoole/followers","following_url":"https://api.github.com/users/CharliePoole/following{/other_user}","gists_url":"https://api.github.com/users/CharliePoole/gists{/gist_id}","starred_url":"https://api.github.com/users/CharliePoole/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/CharliePoole/subscriptions","organizations_url":"https://api.github.com/users/CharliePoole/orgs","repos_url":"https://api.github.com/users/CharliePoole/repos","events_url":"https://api.github.com/users/CharliePoole/events{/privacy}","received_events_url":"https://api.github.com/users/CharliePoole/received_events","type":"User","site_admin":false},"created_at":"2016-09-27T04:05:56Z","updated_at":"2016-09-27T04:05:56Z","body":"We're aiming to do it in stages, starting at the end of the week. The framework will be first."}}
{"actor":{"display_login":"CharliePoole"}}