Skip to content

feat(acc): Add provider_acceptance_tests input for automated acceptance test setup#97

Merged
diofeher merged 5 commits intoopentofu:mainfrom
hoivikaj:feat/acceptance
Apr 16, 2026
Merged

feat(acc): Add provider_acceptance_tests input for automated acceptance test setup#97
diofeher merged 5 commits intoopentofu:mainfrom
hoivikaj:feat/acceptance

Conversation

@hoivikaj
Copy link
Copy Markdown
Contributor

@hoivikaj hoivikaj commented Mar 19, 2026

resolves #36

Used https://github.com/orgs/opentofu/discussions/975 as a framework for decisions.

Adds a new provider_acceptance_tests input that, when set to true, automatically exports the environment variables needed to run OpenTofu provider acceptance tests. This removes boilerplate from CI workflows that run provider acceptance tests.

Signed-off-by: hoivikaj <github@hoivik.co>
@hoivikaj hoivikaj marked this pull request as ready for review March 19, 2026 00:35
@hoivikaj hoivikaj requested a review from a team as a code owner March 19, 2026 00:35
Copy link
Copy Markdown

@lbordowitz lbordowitz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On the face of it, this looks functional and ergonomic. Nice job!

I'm going to reach out to another maintainer who's more familiar with the setup-opentofu repo for a second opinion, since this is my first time reviewing a PR on this repo.

@hoivikaj
Copy link
Copy Markdown
Contributor Author

hoivikaj commented Apr 6, 2026

I noticed this got a thumbs up reaction from @diofeher -- is this still awaiting an additional review?

hoivikaj and others added 2 commits April 6, 2026 13:17
@hoivikaj hoivikaj requested a review from diofeher as a code owner April 6, 2026 17:21
Copy link
Copy Markdown
Member

@diofeher diofeher left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good, thanks for doing that work @hoivikaj!

A really small nit: provider_acceptance_test -> provider_acceptance_tests would be a better fit.

A rebase must be made, though.

P.S. Sorry for taking so long to get back to you, I have a few reviews related to some work you've done on my to-do list :)

hoivikaj and others added 2 commits April 15, 2026 20:18
Signed-off-by: Anthony Hoivik <38190936+hoivikaj@users.noreply.github.com>
…e_tests

Signed-off-by: hoivikaj <github@hoivik.co>
@hoivikaj hoivikaj changed the title feat(acc): Add provider_acceptance_test input for automated acceptance test setup feat(acc): Add provider_acceptance_tests input for automated acceptance test setup Apr 16, 2026
@diofeher diofeher merged commit 34bf4e3 into opentofu:main Apr 16, 2026
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Improvement for CICD acceptance tests

3 participants