Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add automation tests to verify boolean values are set and overridden properly #655

Closed
kim-tsao opened this issue Oct 20, 2021 · 0 comments
Closed
Assignees
Labels
area/test-automation Test automation for devfile specification and library

Comments

@kim-tsao
Copy link
Contributor

kim-tsao commented Oct 20, 2021

Which area/kind this issue is related to?

/area test-automation

Issue Description

Add automation tests that call the Devfile parser (the same way a client would) to validate that:

  • unset booleans are set to their defaults and are never nil
  • booleans are overridden by parent values
  • unset parent boolean values do not override main devfile booleans i.e. parent booleans are not set to default values

We also need to update some of the common API utils. There isn't enough randomness with unset booleans as observed with debugging existing library tests, it's only the parallel and ephemeral that are unset.

We need to add support for RootRequired as well.

@kim-tsao kim-tsao self-assigned this Oct 20, 2021
@kim-tsao kim-tsao added the area/test-automation Test automation for devfile specification and library label Oct 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/test-automation Test automation for devfile specification and library
Projects
None yet
Development

No branches or pull requests

1 participant