{"action":"created","issue":{"url":"https://api.github.com/repos/32leaves/ruruku/issues/53","repository_url":"https://api.github.com/repos/32leaves/ruruku","labels_url":"https://api.github.com/repos/32leaves/ruruku/issues/53/labels{/name}","comments_url":"https://api.github.com/repos/32leaves/ruruku/issues/53/comments","events_url":"https://api.github.com/repos/32leaves/ruruku/issues/53/events","html_url":"https://github.com/32leaves/ruruku/issues/53","id":392589034,"node_id":"MDU6SXNzdWUzOTI1ODkwMzQ=","number":53,"title":"validate on session start","user":{"login":"bwl21","id":2022976,"node_id":"MDQ6VXNlcjIwMjI5NzY=","avatar_url":"https://avatars2.githubusercontent.com/u/2022976?v=4","gravatar_id":"","url":"https://api.github.com/users/bwl21","html_url":"https://github.com/bwl21","followers_url":"https://api.github.com/users/bwl21/followers","following_url":"https://api.github.com/users/bwl21/following{/other_user}","gists_url":"https://api.github.com/users/bwl21/gists{/gist_id}","starred_url":"https://api.github.com/users/bwl21/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bwl21/subscriptions","organizations_url":"https://api.github.com/users/bwl21/orgs","repos_url":"https://api.github.com/users/bwl21/repos","events_url":"https://api.github.com/users/bwl21/events{/privacy}","received_events_url":"https://api.github.com/users/bwl21/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-12-19T12:54:35Z","updated_at":"2018-12-21T20:52:17Z","closed_at":null,"author_association":"NONE","body":"1. it appears that testplan.yaml is read only once, and then is persisted in ruruku.db, So I delete ruruku.db before every start\r\n\r\n2. rururku should give messages when validateing testplan.yaml. For example it loads if \"group\" is an empty string. But it does not show testcases\r\n\r\n3. ruruku should not stop if there are duplicate ids. It would be enough to raise a warning and ignore the the newer duplcates\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/32leaves/ruruku/issues/comments/449495482","html_url":"https://github.com/32leaves/ruruku/issues/53#issuecomment-449495482","issue_url":"https://api.github.com/repos/32leaves/ruruku/issues/53","id":449495482,"node_id":"MDEyOklzc3VlQ29tbWVudDQ0OTQ5NTQ4Mg==","user":{"login":"32leaves","id":3210701,"node_id":"MDQ6VXNlcjMyMTA3MDE=","avatar_url":"https://avatars2.githubusercontent.com/u/3210701?v=4","gravatar_id":"","url":"https://api.github.com/users/32leaves","html_url":"https://github.com/32leaves","followers_url":"https://api.github.com/users/32leaves/followers","following_url":"https://api.github.com/users/32leaves/following{/other_user}","gists_url":"https://api.github.com/users/32leaves/gists{/gist_id}","starred_url":"https://api.github.com/users/32leaves/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/32leaves/subscriptions","organizations_url":"https://api.github.com/users/32leaves/orgs","repos_url":"https://api.github.com/users/32leaves/repos","events_url":"https://api.github.com/users/32leaves/events{/privacy}","received_events_url":"https://api.github.com/users/32leaves/received_events","type":"User","site_admin":false},"created_at":"2018-12-21T20:52:17Z","updated_at":"2018-12-21T20:52:17Z","author_association":"OWNER","body":"Indeed, in ruruku the testplan only exists as a file in your source code - it's does not have a direct representation on the backend. That said, a running test session (i.e. the execution/instance of a testplan) contains the same information. Thus, if you want to \"forget\" your test result/test session, deleting the database file is a feasible method.\r\n\r\nRuruku did not do a good job at validating it's input. At least for testplans (`session start`) that's much improved with #55.\r\n\r\nI think that ignoring testcases with a warning due to duplicate ID's would be surprising behavior, and certainly easier to overlook than a complete failure. Most of the time duplicate ID's are unintentional anyways."}}
{"actor":{"display_login":"32leaves"}}