fix: domain update and expand E2E

[dimitarproynov]

Summary of Pull Request

The domain update logic failed to do a refresh (read) just after the update operation has completed. This resulted in failing acceptance test, the fix to which was simple: execute a read at the end of domain update method. Some function parameter order was messed up.

Additionally, the Domain E2E test was expanded with:

Add a second cluster, Add a forth host in the additional cluster, Remove the forth host, Remove the additional cluster

This way all update operations, currently supported for domain are tested.

Also optimise domain E2E test resource usage.

Type of Pull Request

• [X] This is a bug fix.
• [ ] This is an enhancement or feature.
• [ ] This is a code style/formatting update.
• [ ] This is a documentation update.
• [X] This is a refactoring update.
• [ ] This is a chore update
• [ ] This is something else. Please describe:

Related to Existing Issues

Issue Number: N/A

Test and Documentation Coverage

Testing done: make lint make build make test

=== RUN TestAccResourceVcfDomain === PAUSE TestAccResourceVcfDomain === CONT TestAccResourceVcfDomain --- PASS: TestAccResourceVcfDomain (16852.67s) PASS

For bug fixes or features:

• [X] Tests have been completed.
• [ ] Documentation has been added/updated.

Breaking Changes?

• [ ] Yes, there are breaking changes.
• [X] No, there are no breaking changes.

[github-actions[bot]]

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

If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.