Upgrade legacy API from v5 to v7 - azuredevops_check_approval, azuredevops_check_branch_control, azuredevops_check_business_hours, azuredevops_variable_group, azuredevops_variable_group(data source), azuredevops_group

[xuzhang3]

All Submissions:

• Have you added an explanation of what your changes do and why you'd like us to include them?
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.
• My code follows the code style of this project.
• I ran lint checks locally prior to submission.
• 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:
Update legacy API from v5 to v7 - Resources: azuredevops_check_approval, azuredevops_check_branch_control, azuredevops_check_business_hours, azuredevops_variable_group, azuredevops_variable_group(data source), azuredevops_group

Test log:

=== RUN   TestAccCheckApproval_basic
=== PAUSE TestAccCheckApproval_basic
=== RUN   TestAccCheckApproval_complete
=== PAUSE TestAccCheckApproval_complete
=== RUN   TestAccCheckApproval_update
=== PAUSE TestAccCheckApproval_update
=== CONT  TestAccCheckApproval_basic
=== CONT  TestAccCheckApproval_update
=== CONT  TestAccCheckApproval_complete
--- PASS: TestAccCheckApproval_complete (64.51s)
--- PASS: TestAccCheckApproval_basic (74.47s)
--- PASS: TestAccCheckApproval_update (103.29s)
PASS
ok      github.com/microsoft/terraform-provider-azuredevops/azuredevops/internal/acceptancetests        108.503s

=== 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 (50.75s)
--- PASS: TestAccCheckBranchControl_complete (60.17s)
--- PASS: TestAccCheckBranchControl_update (70.82s)
PASS
ok      github.com/microsoft/terraform-provider-azuredevops/azuredevops/internal/acceptancetests        74.950s

=== 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_complete (50.54s)
--- PASS: TestAccCheckBusinessHours_update (59.82s)
--- PASS: TestAccCheckBusinessHours_basic (60.57s)
PASS
ok      github.com/microsoft/terraform-provider-azuredevops/azuredevops/internal/acceptancetests        61.263s

=== RUN   TestAccVariableGroup_DataSource
=== PAUSE TestAccVariableGroup_DataSource
=== RUN   TestAccVariableGroup_DataSourceKeyVault
=== PAUSE TestAccVariableGroup_DataSourceKeyVault
=== RUN   TestAccVariableGroup_CreateAndUpdate
=== PAUSE TestAccVariableGroup_CreateAndUpdate
=== CONT  TestAccVariableGroup_DataSource
=== CONT  TestAccVariableGroup_CreateAndUpdate
=== CONT  TestAccVariableGroup_DataSourceKeyVault
--- PASS: TestAccVariableGroup_DataSource (61.38s)
--- PASS: TestAccVariableGroup_CreateAndUpdate (78.83s)
--- PASS: TestAccVariableGroup_DataSourceKeyVault (81.85s)
PASS
ok      github.com/microsoft/terraform-provider-azuredevops/azuredevops/internal/acceptancetests        88.435s

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

• Yes
• No

Does this introduce a breaking change?

• Yes
• No

Any relevant logs, error output, etc?

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

Other information