Fix shutdown pi_instance name change behavior and refactor refresh functions

[Alexander-Kita]

Community Note

• Please vote on this pull request by adding a 👍 reaction to the original pull request comment to help the community and maintainers prioritize this request
• Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for pull request followers and do not help prioritize the request

Relates OR Closes #0000

Output from acceptance testing:

pi_instance (r)

=== RUN   TestAccIBMPIInstanceDeploymentGRS
--- PASS: TestAccIBMPIInstanceDeploymentGRS (910.37s)
PASS

=== RUN   TestAccIBMPIInstanceUserTags
--- PASS: TestAccIBMPIInstanceUserTags (980.91s)
PASS

=== RUN   TestAccIBMPIInstanceBasic
--- PASS: TestAccIBMPIInstanceBasic (667.58s)
PASS

=== RUN   TestAccIBMPIInstanceDeploymentType
--- PASS: TestAccIBMPIInstanceDeploymentType (1177.23s)
PASS

=== RUN   TestAccIBMPIInstanceUpdateActiveState
--- PASS: TestAccIBMPIInstanceUpdateActiveState (1458.95s)
PASS

=== RUN   TestAccIBMPIInstanceUpdateStoppedState
--- PASS: TestAccIBMPIInstanceUpdateStoppedState (1486.56s)
PASS

=== RUN   TestAccIBMPIInstanceNetwork
--- PASS: TestAccIBMPIInstanceNetwork (1062.86s)
PASS

=== RUN   TestAccIBMPISAPInstance
--- PASS: TestAccIBMPISAPInstance (1412.23s)
PASS

=== RUN   TestAccIBMPIInstanceDeploymentTypeNoStorage
--- PASS: TestAccIBMPIInstanceDeploymentTypeNoStorage (505.96s)
PASS

NOTE

Do not merge until tested with VSN items, as it, uses some of these functions.

Also fixes bug when updating virtual optical device

[Alexander-Kita]

Opening #188 instead