Fix Magento\Sales\Service\V1\ShipmentGetTest::testShipmentGet

[slackerzz]

Description

Fixed Issues (if relevant)

1. Fix Magento\Sales\Service\V1\ShipmentGetTest::testShipmentGet inventory#1336: Fix Magento\Sales\Service\V1\ShipmentGetTest::testShipmentGet
   Related msi pull request: magento-engcom#1350

Manual testing scenarios

1. ...
2. ...

Contribution checklist

• Pull request has a meaningful description of its purpose
• All commits are accompanied by meaningful commit messages
• All new or changed code is covered with unit/integration tests (if applicable)
• All automated tests passed successfully (all builds on Travis CI are green)

[slackerzz]

@vgelani did you run the test after your commit?
Now it fails with:
Class Magento\Sales\Service\V1\Magento\Framework\Api\SimpleDataObjectConverter does not exist

I'm sorry but your commit introduces several errors:

• when you have 2 nested foreach you cannot use the same variables names, it won't work
• SimpleDataObjectConverter::snakeCaseToUpperCamelCase() is always used statically in the core
• using $this->objectManager->get() inside the nested foreach loop is a bad practice

Moreover the pull request was correct before your commit and reviewed by @maghamed in the msi corresponding pull request.
For all these reasons I'll revert your changes.

[gelanivishal]

@slackerzz I have performed these changes which gave errors in travis so I have performed it.

[magento-engcom-team]

Hi @slackerzz. Thank you for your contribution.
Changes from your Pull Request will be available with the upcoming 2.3.0 release.