{"action":"created","issue":{"url":"https://api.github.com/repos/catch22/octave-doctest/issues/38","labels_url":"https://api.github.com/repos/catch22/octave-doctest/issues/38/labels{/name}","comments_url":"https://api.github.com/repos/catch22/octave-doctest/issues/38/comments","events_url":"https://api.github.com/repos/catch22/octave-doctest/issues/38/events","html_url":"https://github.com/catch22/octave-doctest/pull/38","id":75199291,"number":38,"title":"Suggestion for the new doctest API","user":{"login":"catch22","id":119655,"avatar_url":"https://avatars.githubusercontent.com/u/119655?v=3","gravatar_id":"","url":"https://api.github.com/users/catch22","html_url":"https://github.com/catch22","followers_url":"https://api.github.com/users/catch22/followers","following_url":"https://api.github.com/users/catch22/following{/other_user}","gists_url":"https://api.github.com/users/catch22/gists{/gist_id}","starred_url":"https://api.github.com/users/catch22/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/catch22/subscriptions","organizations_url":"https://api.github.com/users/catch22/orgs","repos_url":"https://api.github.com/users/catch22/repos","events_url":"https://api.github.com/users/catch22/events{/privacy}","received_events_url":"https://api.github.com/users/catch22/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-05-11T12:06:51Z","updated_at":"2015-05-13T06:49:17Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/catch22/octave-doctest/pulls/38","html_url":"https://github.com/catch22/octave-doctest/pull/38","diff_url":"https://github.com/catch22/octave-doctest/pull/38.diff","patch_url":"https://github.com/catch22/octave-doctest/pull/38.patch"},"body":"See #37 and #31 for related discussion. The new summary format is as follows:\r\n\r\n~~~\r\noctave:5> doctest({'doctest', 'doctest_run', 'doctest_compare', 'doctest_collect', 'doctest_colors'})\r\ndoctest ................................................ PASS    6/6   \r\ndoctest_run ............................................ NO TESTS\r\ndoctest_compare ........................................ PASS    2/2   \r\ndoctest_collect ........................................ NO TESTS\r\ndoctest_colors ......................................... NO TESTS\r\n\r\nSummary:\r\n\r\n   PASS    8/8   \r\n\r\n5/5 targets passed, 3 without tests.\r\n\r\noctave:6> \r\n~~~\r\n\r\n~~~\r\noctave:4> doctest({'doctest', 'doctest_run', 'cat', 'doctest_compare', 'more', 'test.texinfo', 'doctest'})\r\ndoctest ................................................ PASS    6/6   \r\ndoctest_run ............................................ NO TESTS\r\ncat .................................................... FAIL    1/3   \r\n\r\n   >> [A, B]\r\n\r\n      expected: \r\n      got     : ans = 1 1 0 0 1 1 0 0\r\n\r\ndoctest_compare ........................................ PASS    2/2   \r\nmore ................................................... NO TESTS\r\ntest.texinfo ........................................... EXTRACTION ERROR\r\n\r\n    has @example blocks but neither \">>\" nor \"@result{}\"\r\n\r\ndoctest ................................................ PASS    6/6   \r\n\r\nSummary:\r\n\r\n   FAIL    1/17  \r\n\r\n5/7 targets passed, 2 without tests, 1 with extraction errors.\r\n\r\noctave:5> \r\n~~~\r\n\r\nPlease see `doctest.m` for the API documentation. Some points of discussion:\r\n\r\n1. I am now printing test results as we go. However, since the non-interactive fake evalc is still broken (thanks to Octave being broken) maybe I should revert this back to @cbm755's method? On the other hand, it's quite cumbersome when testing larger projects if one has to wait for 10 minutes until any test result is visible. On the third hand (...), this would be somewhat remedied by a future `__run_doctest_suite__`, as one could always `tail -f` the logfile.\r\n2. There is a `fid` parameter as in Octave's `test` function which can be used to redirect all standard output, and detailed test summary is returned as the third output parameter. It should be possible to write a `rundoctests` or `__run_doctest_suite__` wrapper as desired by @oheim.\r\n3. Currently, there is only a `normal` mode since `quiet` did not seem very useful in view of point 2."},"comment":{"url":"https://api.github.com/repos/catch22/octave-doctest/issues/comments/101537675","html_url":"https://github.com/catch22/octave-doctest/pull/38#issuecomment-101537675","issue_url":"https://api.github.com/repos/catch22/octave-doctest/issues/38","id":101537675,"user":{"login":"catch22","id":119655,"avatar_url":"https://avatars.githubusercontent.com/u/119655?v=3","gravatar_id":"","url":"https://api.github.com/users/catch22","html_url":"https://github.com/catch22","followers_url":"https://api.github.com/users/catch22/followers","following_url":"https://api.github.com/users/catch22/following{/other_user}","gists_url":"https://api.github.com/users/catch22/gists{/gist_id}","starred_url":"https://api.github.com/users/catch22/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/catch22/subscriptions","organizations_url":"https://api.github.com/users/catch22/orgs","repos_url":"https://api.github.com/users/catch22/repos","events_url":"https://api.github.com/users/catch22/events{/privacy}","received_events_url":"https://api.github.com/users/catch22/received_events","type":"User","site_admin":false},"created_at":"2015-05-13T06:49:17Z","updated_at":"2015-05-13T06:49:17Z","body":"I'll merge this PR later today unless there are any objections."}}