Thanks for the review @jmspring! According to your comments I've slightly changed the PR to:
* Support both formats of the OAuth endpoint (tests added). * Extracted the service principal call into a GraphRBACApi class, so we can add additional methods there as needed. * Modified the structure of the HTTPApiModule and the base mock test class to allow overriding the custom endpoints used in the graph API, and possibly others (as they differ from the endpoint configured when creating the context, and overriding is not trivial). -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/jclouds/jclouds-labs/pull/424#issuecomment-348901851
