Merge pull request #7 from alanquillin/AddUserAgentHttpHeaderToAllAPIRequests

[Delivers #43559413 #43555603 #43555605 #43555607] Added User Agent header and new IdentityProvider tests

Author: alanquillin

lib/SimpleRestServices.dll

@@ -0,0 +1,23 @@
+using System.Runtime.Serialization;
+
+namespace net.openstack.Core.Domain
+{
+    [DataContract]
+    public class NewUser
+    {
+        [DataMember(Name = "OS-KSADM:password")]
+        public string Password { get; set; }
+
+        [DataMember(Name = "id", EmitDefaultValue = true)]
+        public string Id { get; set; }
+
+        [DataMember(Name = "username")]
+        public string Username { get; set; }
+
+        [DataMember(Name = "email")]
+        public string Email { get; set; }
+
+        [DataMember(Name = "enabled")]
+        public bool Enabled { get; set; }
+    }
+}

lib/SimpleRestServices.pdb

@@ -6,21 +6,19 @@ namespace net.openstack.Core.Domain
     [DataContract]
     public class User
     {
-
         [DataMember(Name = "RAX-AUTH:defaultRegion")]
         public string DefaultRegion { get; set; }
 
-        [DataMember]
+        [DataMember(Name="id", EmitDefaultValue = true)]
         public string Id { get; set; }
 
-        [DataMember]
+        [DataMember(Name="username")]
         public string Username { get; set; }
 
-        [DataMember]
+        [DataMember(Name="email")]
         public string Email { get; set; }
 
-        [DataMember]
+        [DataMember(Name = "enabled")]
         public bool Enabled { get; set; }
-
     }
 }

src/corelib/Core/Domain/NewUser.cs

@@ -13,7 +13,7 @@ public interface IIdentityProvider
         User[] ListUsers(CloudIdentity identity);
         User GetUserByName(CloudIdentity identity, string name);
         User GetUser(CloudIdentity identity, string id);
-        User AddUser(CloudIdentity identity, User user);
+        NewUser AddUser(CloudIdentity identity, NewUser user);
         User UpdateUser(CloudIdentity identity, User user);
         bool DeleteUser(CloudIdentity identity, string userId);
 

src/corelib/Core/Domain/User.cs

@@ -1,6 +1,7 @@
 using System;
 using System.Collections.Generic;
 using System.Linq;
+using System.Reflection;
 using Newtonsoft.Json;
 using Newtonsoft.Json.Linq;
 using SimpleRestServices.Client;
@@ -215,18 +216,27 @@ public User GetUser(CloudIdentity identity, string userId)
             return response.Data.User;
         }
 
-        public User AddUser(CloudIdentity identity, User user)
+        public NewUser AddUser(CloudIdentity identity, NewUser newUser)
         {
-            var response = ExecuteRESTRequest<UserResponse>(identity, "/v2.0/users", HttpMethod.POST, new AddUserRequest { User = user });
+            newUser.Id = null;
+
+            var response = ExecuteRESTRequest<NewUserResponse>(identity, "/v2.0/users", HttpMethod.POST, new AddUserRequest { User = newUser });
 
             if (response == null || response.Data == null)
                 return null;
 
-            return response.Data.User;
+            // If the user specifies a password, then the password will not be in the response, so we need to fill it in on the return object.
+            if (string.IsNullOrWhiteSpace(response.Data.NewUser.Password))
+                response.Data.NewUser.Password = newUser.Password;
+
+            return response.Data.NewUser;
         }
 
         public User UpdateUser(CloudIdentity identity, User user)
         {
+            if(user == null || string.IsNullOrWhiteSpace(user.Id))
+                throw new ArgumentException("The User or User.Id values cannot be null.");
+
             var urlPath = string.Format("v2.0/users/{0}", user.Id);
 
             var updateUserRequest = new UpdateUserRequest { User = user };
@@ -378,7 +388,7 @@ protected virtual Response ExecuteRESTRequest(CloudIdentity identity, string url
                     bodyStr = JsonConvert.SerializeObject(body, new JsonSerializerSettings{NullValueHandling = NullValueHandling.Ignore});
             }
 
-            var response = _restService.Execute<T>(url, method, bodyStr, headers, queryStringParameter, new JsonRequestSettings() { RetryCount = retryCount, RetryDelayInMS = retryDelay, Non200SuccessCodes = new[] { 401, 409 } });
+            var response = _restService.Execute<T>(url, method, bodyStr, headers, queryStringParameter, new JsonRequestSettings() { RetryCount = retryCount, RetryDelayInMS = retryDelay, Non200SuccessCodes = new[] { 401, 409 }, UserAgent = ProviderBase.GetUserAgentHeaderValue()});
 
             // on errors try again 1 time.
             if (response.StatusCode == 401 && !isRetry && !isTokenRequest)

src/corelib/Core/IIdentityProvider.cs

@@ -67,10 +67,10 @@ public User GetUser(CloudIdentity identity, string userId)
             return provider.GetUser(identity, userId);
         }
 
-        public User AddUser(CloudIdentity identity, User user)
+        public NewUser AddUser(CloudIdentity identity, NewUser newUser)
         {
             var provider = GetProvider(identity);
-            return provider.AddUser(identity, user);
+            return provider.AddUser(identity, newUser);
         }
 
         public User UpdateUser(CloudIdentity identity, User user)
@@ -168,7 +168,7 @@ private IExtendedIdentityProvider GetProvider(CloudIdentity identity)
             var rackspaceCloudIdentity = identity as RackspaceCloudIdentity;
 
             if (rackspaceCloudIdentity == null)
-                throw new InvalidCloudIdentityException(string.Format("Invalid Identity object.  Rackspace Identoty service requires an instance of type: {0}", typeof(RackspaceCloudIdentity)));
+                _factory.Get(CloudInstance.Default);
 
             return _factory.Get(rackspaceCloudIdentity.CloudInstance);
         }

src/corelib/Providers/Rackspace/GeographicalIdentityProvider.cs

@@ -7,6 +7,6 @@ namespace net.openstack.Providers.Rackspace.Objects.Request
     internal class AddUserRequest
     {
         [DataMember(Name = "user")]
-        public User User { get; set; }
+        public NewUser User { get; set; }
     }
 }

src/corelib/Providers/Rackspace/IdentityProvider.cs

@@ -1,13 +1,16 @@
 using System;
 using System.Collections.Generic;
 using System.Linq;
+using System.Runtime.Serialization;
 using System.Text;
 using net.openstack.Core.Domain;
 
 namespace net.openstack.Providers.Rackspace.Objects.Request
 {
+    [DataContract]
     internal class UpdateUserRequest
     {
+        [DataMember(Name = "user")]
         public User User { get; set; }
     }
 }

src/corelib/Providers/Rackspace/Objects/Request/AddUserRequest.cs

@@ -0,0 +1,12 @@
+using System.Runtime.Serialization;
+using net.openstack.Core.Domain;
+
+namespace net.openstack.Providers.Rackspace.Objects.Response
+{
+    [DataContract]
+    internal class NewUserResponse
+    {
+        [DataMember(Name = "user")]
+        public NewUser NewUser { get; set; }
+    }
+}