-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat(acc_test): Adds acceptance testing primitives and test cases #158
Conversation
8215e7f
to
d98f8ed
Compare
README.md
Outdated
# if we have self-signed certificate for a testing instance | ||
export PANOS_SKIP_VERIFY_CERTIFICATE=true | ||
|
||
# We use either API key or username, password pair for authentication |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like use of usernames and passwords has an undesired effect of causing tests to fail when executed in parallel? Should we even document that both can be used in that case? Perhaps we need a check in the check function to verify that credentials are not used when parallel > 1 ?
nameSuffix := acctest.RandStringFromCharSet(6, acctest.CharSetAlphaNum) | ||
compareValuesDiffer := statecheck.CompareValue(compare.ValuesDiffer()) | ||
|
||
resource.Test(t, resource.TestCase{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All tests seem to be missing CheckDestroy used to verify that resources have actually been removed from the server.
resource.Test(t, resource.TestCase{ | ||
PreCheck: func() { testAccPreCheck(t) }, | ||
ProtoV6ProviderFactories: testAccProviders, | ||
Steps: []resource.TestStep{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets either uncomment this code and fix it, or just drop it from the PR
Adds pango client to the test suite and implements check destroys
d98f8ed
to
59d9f46
Compare
Description
Adds acceptance testing primitives for Terraform Panos Provider resources and data sources
Motivation and Context
To be sure of the behaviour and predictability of the resources, data sources and other elements enabled by the Panos Terraform provider, we need to verify them through acceptance testing.
How Has This Been Tested?
Screenshots (if appropriate)
Types of changes
The changes add primitives and patterns of acceptance testing framework by terraform. Tests covers the following resources:
template
address_objects
ethernet_interface
address_group
virtual_router
Checklist
address_objects
tests by updating state path traversal.