Skip to content

Conversation

vepatel
Copy link
Contributor

@vepatel vepatel commented Nov 22, 2022

Proposed changes

  • Add pytest command line option to skip resource teardown (only "yes" and "no" values are accepted, anything else will result in failure at setup with message)

Usage:

>>>pytest -s -v -m vs --skip-fixture-teardown="yes"

<TEST_START>
Tests will use this kubeconfig: /Users/<username>/.kube/config
Tests will use a current context: <CONTEXT_NAME>
Tests will use the image: <link_to_image_repo>
Tests will run with the image-pull-policy: Always
Tests will use the IC deployment of type: deployment
Tests will run against the IC of type: nginx-ingress
Number of pods spun up will be : 1
Tests will use Service of this type: loadbalancer
All test fixtures be available for debugging: yes, /// ONLY USE THIS OPTION FOR INDIVIDUAL TEST DEBUGGING ///
------------------------- Create IC Prerequisites  -----------------------------------
.
.
.
.
<TEST_RUN>
.
.
.
.
===================== WARNING =====================
Make sure to remove resources from this test run manually using kubectl utility

PASSED
<TEST_END>

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto main
  • I will ensure my PR is targeting the main branch and pulling from my branch from my own fork

@vepatel vepatel requested a review from a team as a code owner November 22, 2022 12:50
@vepatel vepatel requested a review from lucacome November 22, 2022 12:50
@github-actions github-actions bot added documentation Pull requests/issues for documentation tests Pull requests that update tests labels Nov 22, 2022
@vepatel vepatel merged commit 3988291 into main Nov 22, 2022
@vepatel vepatel deleted the tests/optional-skip-teardowns branch November 22, 2022 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Pull requests/issues for documentation tests Pull requests that update tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants