microsoft / terraform-provider-power-platform

Power Platform Terraform Provider
https://registry.terraform.io/providers/microsoft/power-platform/latest/docs
MIT License
35 stars 13 forks source link

powerplatform_data_loss_prevention_policy added additional validation for default_action_rule_behavior and action_rules when both are empty #524

Closed mawasile closed 3 days ago

mawasile commented 1 week ago

This pull request includes several updates and improvements to the Data Loss Prevention Policy functionality, primarily focusing on schema enhancements and validation logic. The most important changes include adding additional validation for default_action_rule_behavior and action_rules, updating the Go module version, and refining the schema attributes by removing redundant descriptions.

Validation Enhancements:

Schema Attribute Refinements:

Module and Dependency Updates:

New Helper Functions:

Configuration Validation:

codecov-commenter commented 3 days ago

Codecov Report

Attention: Patch coverage is 67.53247% with 25 lines in your changes missing coverage. Please review.

Project coverage is 73.35%. Comparing base (1f1b233) to head (8952d67).

Files with missing lines Patch % Lines
...nternal/services/dlp_policy/resource_dlp_policy.go 52.50% 14 Missing and 5 partials :warning:
internal/services/dlp_policy/helpers.go 90.00% 2 Missing and 1 partial :warning:
internal/services/environment_settings/models.go 0.00% 2 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #524 +/- ## ========================================== - Coverage 73.46% 73.35% -0.12% ========================================== Files 105 105 Lines 14949 14957 +8 ========================================== - Hits 10983 10972 -11 - Misses 3292 3308 +16 - Partials 674 677 +3 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.