Remove MinItems from data groups

[jpreese]

Was trying to use the azuread_groups with the optional names parameter, but was getting an error about a value must be returned as they are both set to 1 and return an error.

Looks like a similar issue to: #227

[jpreese]

For context, my use case is that a user can supply an optional value which is a list of Azure AD group names.

I take that variable, and pass it into the azuread_groups data provider, but since its optional it might be empty.

I then have another resource that does a count on the azuread_groups provider to add a resource per group that was returned. If no groups were returned, especially in the case of an empty input, don't recreate any resources.

[jpreese]

As an aside, for a future PR, I think there's also value in changing the Object_IDs to be a set type. This would make it easier to use for_each and the keys should be unique.

[manicminer]

Hi @jpreese, thanks for this change suggestion! We've discussed and believe that in this case, it's OK to support empty lists for search inputs. I've moved this to a new PR #257, preserving your contribution, as I've added a test case and clarified the new behavior in the docs.

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!