{"action":"created","issue":{"url":"https://api.github.com/repos/WahlNetwork/Vester/issues/38","repository_url":"https://api.github.com/repos/WahlNetwork/Vester","labels_url":"https://api.github.com/repos/WahlNetwork/Vester/issues/38/labels{/name}","comments_url":"https://api.github.com/repos/WahlNetwork/Vester/issues/38/comments","events_url":"https://api.github.com/repos/WahlNetwork/Vester/issues/38/events","html_url":"https://github.com/WahlNetwork/Vester/issues/38","id":174562767,"number":38,"title":"Complex Testing and Sub-Tests Discussion","user":{"login":"chriswahl","id":4888402,"avatar_url":"https://avatars.githubusercontent.com/u/4888402?v=3","gravatar_id":"","url":"https://api.github.com/users/chriswahl","html_url":"https://github.com/chriswahl","followers_url":"https://api.github.com/users/chriswahl/followers","following_url":"https://api.github.com/users/chriswahl/following{/other_user}","gists_url":"https://api.github.com/users/chriswahl/gists{/gist_id}","starred_url":"https://api.github.com/users/chriswahl/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chriswahl/subscriptions","organizations_url":"https://api.github.com/users/chriswahl/orgs","repos_url":"https://api.github.com/users/chriswahl/repos","events_url":"https://api.github.com/users/chriswahl/events{/privacy}","received_events_url":"https://api.github.com/users/chriswahl/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/WahlNetwork/Vester/labels/question","name":"question","color":"cc317c"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":7,"created_at":"2016-09-01T16:02:55Z","updated_at":"2016-09-03T13:09:15Z","closed_at":null,"body":"As revealed with the HA tests, there are certain types of tests that end up being dependencies of themselves. As an example:\r\n\r\n* Issue1:  HA should be on - as such, other settings should be configured such as Admission Control and Heartbeat Datastores\r\n* Issue2: HA should be off - as such, other settings should not be tested because they are not applicable such as Admission Control and Heartbeat Datastores\r\n* Issue3: HA should be on but we don't care about the other settings such as Admission Control and Heartbeat Datastores\r\n\r\nIssue1 is relatively easy and is how Vester works today.\r\n\r\nIssue2 could be solved in a few different ways. Probably the easiest is to simply wrap up the sub-tests (that check Admission Control and whatnot) with a boolean test to see if the user wants HA on in the first place. If HA enable $true, we run all the tests. If $false, we only check the state of HA.\r\n\r\nIssue3 is a bit more interesting. There's a few ways we could handle this. One is to allow people to comment out configurations that aren't desired or perhaps set the value to $null. The Config.Tests file would need to allow for this. The .Tests file would need to then handle configs that are $null or missing and be told to skip those tests.\r\n\r\nWhat say you @cars @equelin @brianbunke ?"},"comment":{"url":"https://api.github.com/repos/WahlNetwork/Vester/issues/comments/244545446","html_url":"https://github.com/WahlNetwork/Vester/issues/38#issuecomment-244545446","issue_url":"https://api.github.com/repos/WahlNetwork/Vester/issues/38","id":244545446,"user":{"login":"equelin","id":9823778,"avatar_url":"https://avatars.githubusercontent.com/u/9823778?v=3","gravatar_id":"","url":"https://api.github.com/users/equelin","html_url":"https://github.com/equelin","followers_url":"https://api.github.com/users/equelin/followers","following_url":"https://api.github.com/users/equelin/following{/other_user}","gists_url":"https://api.github.com/users/equelin/gists{/gist_id}","starred_url":"https://api.github.com/users/equelin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/equelin/subscriptions","organizations_url":"https://api.github.com/users/equelin/orgs","repos_url":"https://api.github.com/users/equelin/repos","events_url":"https://api.github.com/users/equelin/events{/privacy}","received_events_url":"https://api.github.com/users/equelin/received_events","type":"User","site_admin":false},"created_at":"2016-09-03T13:09:15Z","updated_at":"2016-09-03T13:09:15Z","body":"The problem of mixing `-Tag` `-ExcludeTag` or `-TestNames` is that it can overcomplicated the usage of Vester...\r\n\r\nI was thinking at another solution, what if the config file is used to select wich test to run? if the user put some data in it, Vester run the tests related to this data. No data, no tests.\r\nWhat do you guys think about that?"}}
{"actor":{"display_login":"equelin"}}