Skip to content

Conversation

@remyleone
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings December 12, 2025 06:46
@remyleone remyleone marked this pull request as draft December 12, 2025 06:46
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new unit test to validate that all data sources and resources in the SDK provider have non-empty descriptions. This ensures documentation completeness and helps maintain quality standards for the provider's schema definitions.

Key Changes

  • Added TestProviderSDKDescriptionsAreNotEmpty function to verify all resources and data sources have descriptions

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

})
}

func TestProviderSDKDescriptionsAreNotEmpty(t *testing.T) {
Copy link

Copilot AI Dec 12, 2025

Choose a reason for hiding this comment

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

The test function name should follow the naming convention used by other acceptance tests in this file. Since this is a unit test rather than an acceptance test, consider using "TestSDKProvider_DescriptionsNotEmpty" or a similar pattern to be consistent with the "TestAccProvider_*" naming convention for acceptance tests in this file. Alternatively, if it should be an acceptance test, prefix it with "TestAcc".

Suggested change
func TestProviderSDKDescriptionsAreNotEmpty(t *testing.T) {
func TestSDKProvider_DescriptionsNotEmpty(t *testing.T) {

Copilot uses AI. Check for mistakes.
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.

1 participant