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

function: Missing underlying type validation and refactor parameter name validation #991

Merged
merged 6 commits into from
Apr 24, 2024

Conversation

bflad
Copy link
Contributor

@bflad bflad commented Apr 23, 2024

Closes #965
Closes #967

The framework will now raise implementation errors if a function parameter or return definition is missing underlying type information (e.g. collection element type or object attribute type).

…ame validation

Reference: #965
Reference: #967

The framework will now raise implementation errors if a function parameter or return definition is missing underlying type information (e.g. collection element type or object attribute type).
@bflad bflad added the enhancement New feature or request label Apr 23, 2024
@bflad bflad added this to the v1.9.0 milestone Apr 23, 2024
@bflad bflad requested a review from a team as a code owner April 23, 2024 20:52
Copy link
Member

@austinvalle austinvalle left a comment

Choose a reason for hiding this comment

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

Great PR! Just one nit 🚀

"github.com/hashicorp/terraform-plugin-framework/types"
)

func TestContainsMissingUnderlyingType(t *testing.T) {
Copy link
Member

Choose a reason for hiding this comment

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

nit: Do we want to add a test here with a custom type in a collection/object/etc?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is 584f83d what you are looking for?

Copy link
Member

Choose a reason for hiding this comment

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

Yeah that looks great!

@bflad bflad merged commit a150eef into main Apr 24, 2024
28 checks passed
@bflad bflad deleted the bflad/parameter-validation-refactoring branch April 24, 2024 19:37
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
2 participants