feat(setServiceUrl): use setServiceUrl instead of setServiceEndpoint

[mamoonraja]

BREAKING CHANGE: Use SetServiceUrl to set the service endpoint rather than SetEndpoint

[mediumTaj]

👍 looks good! tagged phil on review as well

[padamstx]

Before the service_url value is used, we'll need to validate it and possibly throw/raise an error if it's not valid. And we should add tests that check this as well :)

[mediumTaj]

For Unity we didn't have a testing platform until very recently. We were only able to do integration tests. We're moving to the new way of testing as part of the unit test generation epic. Even now though we cannot test Unity in CI since we are being blocked from downloading/installing the Unity application in travis. Also no way of finding code coverage...yet :)

[padamstx]

I left a comment which might or might not lead to additional changes, but I'll let you decide :)

[padamstx]

A couple things that you can choose to address or not. Looks good.

[padamstx]

Just realized that we need a GetServiceUrl() method on the BaseService class :)

[padamstx]

Sorry, but just realized we need a GetServiceUrl() method on BaseService :)

[padamstx]

Looks good now.