`azuredevops_check_approval`,`azuredevops_check_exclusive_lock`, `azuredevops_check_branch_control`, `azuredevops_check_business_hours`, `azuredevops_check_required_template` - Optimize flatten and setId

[xuzhang3]

All Submissions:

• [x] Have you added an explanation of what your changes do and why you'd like us to include them?
• [x] I have updated the documentation accordingly.
• [x] I have added tests to cover my changes.
• [x] All new and existing tests passed.
• [x] My code follows the code style of this project.
• [x] I ran lint checks locally prior to submission.
• [x] Have you checked to ensure there aren't other open PRs for the same update/change?

What about the current behavior has changed?

Issue Number:

--- PASS: TestAccCheckApproval_basic (107.97s)
--- PASS: TestAccCheckApproval_complete (108.19s)
--- PASS: TestAccCheckApproval_update (147.13s)
PASS
ok      github.com/microsoft/terraform-provider-azuredevops/azuredevops/internal/acceptancetests        147.733s

=== RUN   TestAccCheckBranchControl_basic
=== PAUSE TestAccCheckBranchControl_basic
=== RUN   TestAccCheckBranchControl_complete
=== PAUSE TestAccCheckBranchControl_complete
=== RUN   TestAccCheckBranchControl_update
=== PAUSE TestAccCheckBranchControl_update
=== CONT  TestAccCheckBranchControl_basic
=== CONT  TestAccCheckBranchControl_update
=== CONT  TestAccCheckBranchControl_complete
--- PASS: TestAccCheckBranchControl_basic (79.40s)
--- PASS: TestAccCheckBranchControl_complete (86.49s)
--- PASS: TestAccCheckBranchControl_update (114.05s)
PASS
ok      github.com/microsoft/terraform-provider-azuredevops/azuredevops/internal/acceptancetests        114.459s

=== RUN   TestAccCheckBusinessHours_basic
=== PAUSE TestAccCheckBusinessHours_basic
=== RUN   TestAccCheckBusinessHours_complete
=== PAUSE TestAccCheckBusinessHours_complete
=== RUN   TestAccCheckBusinessHours_update
=== PAUSE TestAccCheckBusinessHours_update
=== CONT  TestAccCheckBusinessHours_basic
=== CONT  TestAccCheckBusinessHours_update
=== CONT  TestAccCheckBusinessHours_complete
--- PASS: TestAccCheckBusinessHours_basic (99.70s)
--- PASS: TestAccCheckBusinessHours_complete (99.94s)
--- PASS: TestAccCheckBusinessHours_update (126.88s)
PASS
ok      github.com/microsoft/terraform-provider-azuredevops/azuredevops/internal/acceptancetests        127.451s

=== RUN   TestAccCheckExclusiveLock_basic
=== PAUSE TestAccCheckExclusiveLock_basic
=== CONT  TestAccCheckExclusiveLock_basic
--- PASS: TestAccCheckExclusiveLock_basic (103.05s)
PASS
ok      github.com/microsoft/terraform-provider-azuredevops/azuredevops/internal/acceptancetests        103.613s

=== RUN   TestAccCheckRequiredTemplate_basic
=== PAUSE TestAccCheckRequiredTemplate_basic
=== RUN   TestAccCheckRequiredTemplate_complete
=== PAUSE TestAccCheckRequiredTemplate_complete
=== RUN   TestAccCheckRequiredTemplate_update
=== PAUSE TestAccCheckRequiredTemplate_update
=== CONT  TestAccCheckRequiredTemplate_basic
=== CONT  TestAccCheckRequiredTemplate_update
=== CONT  TestAccCheckRequiredTemplate_complete
--- PASS: TestAccCheckRequiredTemplate_complete (85.25s)
--- PASS: TestAccCheckRequiredTemplate_basic (85.33s)
--- PASS: TestAccCheckRequiredTemplate_update (116.01s)
PASS
ok      github.com/microsoft/terraform-provider-azuredevops/azuredevops/internal/acceptancetests        116.497s

Does this introduce a change to go.mod, go.sum or vendor/?

• [ ] Yes
• [x] No

Does this introduce a breaking change?

• [ ] Yes
• [x] No

Any relevant logs, error output, etc?

(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)

Other information