{"action":"created","issue":{"url":"https://api.github.com/repos/machinekit/machinekit-cnc/issues/16","repository_url":"https://api.github.com/repos/machinekit/machinekit-cnc","labels_url":"https://api.github.com/repos/machinekit/machinekit-cnc/issues/16/labels{/name}","comments_url":"https://api.github.com/repos/machinekit/machinekit-cnc/issues/16/comments","events_url":"https://api.github.com/repos/machinekit/machinekit-cnc/issues/16/events","html_url":"https://github.com/machinekit/machinekit-cnc/issues/16","id":346982771,"node_id":"MDU6SXNzdWUzNDY5ODI3NzE=","number":16,"title":"we need a regression test framework which actually works. runtests has to go. ","user":{"login":"ArcEye","id":770392,"node_id":"MDQ6VXNlcjc3MDM5Mg==","avatar_url":"https://avatars2.githubusercontent.com/u/770392?v=4","gravatar_id":"","url":"https://api.github.com/users/ArcEye","html_url":"https://github.com/ArcEye","followers_url":"https://api.github.com/users/ArcEye/followers","following_url":"https://api.github.com/users/ArcEye/following{/other_user}","gists_url":"https://api.github.com/users/ArcEye/gists{/gist_id}","starred_url":"https://api.github.com/users/ArcEye/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ArcEye/subscriptions","organizations_url":"https://api.github.com/users/ArcEye/orgs","repos_url":"https://api.github.com/users/ArcEye/repos","events_url":"https://api.github.com/users/ArcEye/events{/privacy}","received_events_url":"https://api.github.com/users/ArcEye/received_events","type":"User","site_admin":false},"labels":[{"id":820329075,"node_id":"MDU6TGFiZWw4MjAzMjkwNzU=","url":"https://api.github.com/repos/machinekit/machinekit-cnc/labels/bug","name":"bug","color":"d73a4a","default":true},{"id":1012492162,"node_id":"MDU6TGFiZWwxMDEyNDkyMTYy","url":"https://api.github.com/repos/machinekit/machinekit-cnc/labels/problem","name":"problem","color":"fbca04","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":14,"created_at":"2018-08-02T12:06:48Z","updated_at":"2018-08-02T12:06:58Z","closed_at":null,"author_association":"COLLABORATOR","body":"<a href=\"https://github.com/mhaberler\"><img src=\"https://avatars1.githubusercontent.com/u/901479?v=4\" align=\"left\" width=\"96\" height=\"96\" hspace=\"10\"></img></a> **Issue by [mhaberler](https://github.com/mhaberler)**\n_Sun Jun 15 06:57:00 2014_\n_Originally opened as https://github.com/machinekit/machinekit/issues/220_\n\n----\n\nI'm loosing patience with this utterly naive approach to 'testing' which we inherited - this is the singlemost biggest time waster. This has to go.\n\nThe current suite of 'tests' is so full of race conditions and using naive programs and scripts that in many cases the failure of a runtest is basically meaningless - it usuallly just shows the author has not considered timing of parallel activities and how they can be properly interlocked.\n\nMilestone proposal: deprecate runtests.\n\nFor each 'failure' found, rewrite the test with the new cython bindings once we have them, make sure the test actually says something, and disable the old one.\n\n"},"comment":{"url":"https://api.github.com/repos/machinekit/machinekit-cnc/issues/comments/409903701","html_url":"https://github.com/machinekit/machinekit-cnc/issues/16#issuecomment-409903701","issue_url":"https://api.github.com/repos/machinekit/machinekit-cnc/issues/16","id":409903701,"node_id":"MDEyOklzc3VlQ29tbWVudDQwOTkwMzcwMQ==","user":{"login":"ArcEye","id":770392,"node_id":"MDQ6VXNlcjc3MDM5Mg==","avatar_url":"https://avatars2.githubusercontent.com/u/770392?v=4","gravatar_id":"","url":"https://api.github.com/users/ArcEye","html_url":"https://github.com/ArcEye","followers_url":"https://api.github.com/users/ArcEye/followers","following_url":"https://api.github.com/users/ArcEye/following{/other_user}","gists_url":"https://api.github.com/users/ArcEye/gists{/gist_id}","starred_url":"https://api.github.com/users/ArcEye/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ArcEye/subscriptions","organizations_url":"https://api.github.com/users/ArcEye/orgs","repos_url":"https://api.github.com/users/ArcEye/repos","events_url":"https://api.github.com/users/ArcEye/events{/privacy}","received_events_url":"https://api.github.com/users/ArcEye/received_events","type":"User","site_admin":false},"created_at":"2018-08-02T12:06:58Z","updated_at":"2018-08-02T12:06:58Z","author_association":"COLLABORATOR","body":"<a href=\"https://github.com/mhaberler\"><img src=\"https://avatars1.githubusercontent.com/u/901479?v=4\" align=\"left\" width=\"48\" height=\"48\" hspace=\"10\"></img></a> **Comment by [mhaberler](https://github.com/mhaberler)**\n_Thu Oct  1 11:27:30 2015_\n\n----\n\nplease do not base any work on this branch, and do not assume this will be merged as is - it is not ready, so there's no point in going for a build, and I do not want that branch to start creeping out early and setting any expectations\n\nplease see the docs at http://check.sourceforge.net/\n\njust take regressions/check_tests as an example to build your own, and take the suite files as examples to read and learn from \n\ntake regressions/check_tests/check_tests.c and the Submakefile, plus the change in src/Makefile, and create your own test program from it; delete all the *-suite.h includes and the srunner_add_suite() calls which refer to the tests in the *suite includes  \n\n"}}
{"actor":{"display_login":"ArcEye"}}