alibazlamit commented on this pull request.
> + public void testGet() {
+ User result = userApi().get(currentUser.id());
+
+ assertEquals(result.id(), currentUser.id());
+ }
+
+ @Test(dependsOnMethods = "testGet")
+ public void testUpdate() throws InterruptedException {
+ //this test will fail too since its not allowed to update other users
from the API
+// String updatedDescription = "updatejclouds";
+//
+// User updateResult = userApi().update(currentUser.id(),
User.UpdateUser.builder()
+// .description(updatedDescription)
+// .build());
+//
+// assertEquals(updateResult.description(), updatedDescription);
@nacx
I think that this PR and the Users API does not make a lot of sense to jClouds,
most of the operations are only allowed to the current user, and changing the
API key or deleting the users might break any flow, if you agree i think this
feature could be skipped for jClouds, WDYT?
--
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/336
