Fix importing minio_iam_group

[pjsier]

Changes

• Updates the read operation for minio_iam_group to use d.Id() instead of referencing the group config so that importing works
• Updates the associated test to check whether or not import works by adding a step with ImportState

While updating this I noticed that the behavior of using a hidden group_name field in addition to name as well as only reading group_name in minioReadGroup seemed a bit odd. For now I just ignored the fields that aren't read to match the pattern from other test files. Would it make sense to refactor that in the future or am I missing something in the implementation?

Closing issues

• Closes: terraform import minio_iam_group.test doesn't work #375

[felladrin]

Thanks for fixing it!
I've tested it, and it's working as described, solving the issue mentioned above.

Would it make sense to refactor that in the future or am I missing something in the implementation?

Hmm, I'm unable to answer it. But I'd vote for refactoring it.