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

Integration tests for the stack resource against a cluster #447

Open
2 of 5 tasks
rquitales opened this issue May 9, 2023 · 1 comment
Open
2 of 5 tasks

Integration tests for the stack resource against a cluster #447

rquitales opened this issue May 9, 2023 · 1 comment
Labels
good-first-issue Start here if you'd like to start contributing to Pulumi impact/reliability Something that feels unreliable or flaky kind/enhancement Improvements or new features

Comments

@rquitales
Copy link
Member

rquitales commented May 9, 2023

Currently, we have tests nested under the test folder. These are integration tests that runs a locally built controller binary against the testdata stacks. We also have some unit tests (pending being fixed in #387), but these are unused currently. We should expand on our unit tests by also running them in CI, and encouraging new feature development to also include unit testing, rather than only having integration testing.

Some potential next steps after landing #387:

  • Enable running unit tests by including a Make target
  • Running both integration and unit tests in CI
  • Most of the tests in stack_controller_test.go should be split up so they test one thing at a time
  • Most of the tests in stack_controller_test.go could be rewritten to not need AWS, with the exception of testing S3 bucket as a backend (which could be skipped locally)
  • Potentially running periodic CI tests against an actual Kubernetes cluster

Update:

CI targets were fixed. stack_controller_test.go is deprecated. This is tracking the remaining work to run some E2E tests against a live cluster.

@rquitales rquitales added kind/enhancement Improvements or new features impact/reliability Something that feels unreliable or flaky good-first-issue Start here if you'd like to start contributing to Pulumi labels May 9, 2023
@cleverguy25
Copy link

Added to epic #586

@mjeffryes mjeffryes changed the title Improve testing by enabling unit and integration tests Unit and integration tests for new operator Aug 2, 2024
@mjeffryes mjeffryes changed the title Unit and integration tests for new operator Integration tests against a cluster Aug 2, 2024
@mjeffryes mjeffryes changed the title Integration tests against a cluster Integration tests for the stack resource against a cluster Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good-first-issue Start here if you'd like to start contributing to Pulumi impact/reliability Something that feels unreliable or flaky kind/enhancement Improvements or new features
Projects
No open projects
Status: No status
Development

No branches or pull requests

2 participants