[FIX] Installer allowing invalid config.yaml files

[suvanbanerjee]

uninstall: false
method: xyz
channel: abc
profile: ovos
features:
  skills: true
  gui: true
rapsberry_pi_tuning: true
share_telemetry: true

is an invalid config.yaml file but still is considered valid and installs ovos.

Fix: used conditional statements to validate each options correctness. Closes: #72

[goldyfruit]

@suvanbanerjee Thanks.

• Did you test it?
• What is the output when it breaks?

[suvanbanerjee]

Did you test it?

I tested it locally by running it not usings the BATS tests. and it works by saying seceniro not supported log https://sprunge.us/ZvSMFd

What is the output when it breaks?

this is the output when using method: xyz and channel: abc in the current version

[suvanbanerjee]

@goldyfruit I think the failed bats test in #73 is also due to the invalid config.yaml file. but unable to confirm as i am not aware how to run BATS test locally

[goldyfruit]

@goldyfruit I think the failed bats test in #73 is also due to the invalid config.yaml file. but unable to confirm as i am not aware how to run BATS test locally

Just to make sure, your file is named scenario.yaml and not config.yaml, right?

[suvanbanerjee]

@goldyfruit I think the failed bats test in #73 is also due to the invalid config.yaml file. but unable to confirm as i am not aware how to run BATS test locally

Just to make sure, your file is named scenario.yaml and not config.yaml, right?

my bad it's scenario.yaml only. 😅

https://pastebin.com/wV0AkayA here is the full log for the output when using method: xyz and channel: abc in the current version

[goldyfruit]

I'm gonna test it later today. :+1:

[suvanbanerjee]

I'm gonna test it later today. 👍

Just tested it using github actions and it is still failing the bats test.