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

terraform test: remove planned state from plan #35049

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

liamcervante
Copy link
Member

This is an internal refactoring of the PlannedState field within the Terraform internal plan struct. This was added in the past to help the testing framework perform validations after a plan operation. As of 0f845ad this is no longer required. The PlannedState was only being referenced either in places it wasn't needed, or just within test files. There's no behaviour change here, just removing the no-longer used field to ensure it doesn't cause confusion elsewhere or become a dependency by accident that we'd have to maintain long term.

@apparentlymart
Copy link
Member

Heh, funny coincidence that on Friday I was looking at this and wondering if it was actually still needed, but found references to it that I wasn't familiar with and so put the question aside. I'm glad you had a similar thought!

I'll let someone directly on the core team be the reviewer, but just wanted to say thanks for checking in on this and tidying up my loose end! 😀

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.

None yet

3 participants