Subaccount e-mail can be updated

seatsio · Jul 25, 2018 · f34be9b · f34be9b
1 parent 3285e50
commit f34be9b
Show file tree

Hide file tree

Showing 3 changed files with 41 additions and 5 deletions.
diff --git a/SeatsioDotNet.Test/Subaccounts/UpdateSubaccountTest.cs b/SeatsioDotNet.Test/Subaccounts/UpdateSubaccountTest.cs
@@ -7,12 +7,40 @@ public class UpdateSubaccountTest : SeatsioClientTest
         [Fact]
         public void Test()
         {
+            var email = RandomEmail();
             var subaccount = Client.Subaccounts.Create("joske");
 
+            Client.Subaccounts.Update(subaccount.Id, "jefke", email);
+
+            var retrievedSubaccount = Client.Subaccounts.Retrieve(subaccount.Id);
+            Assert.Equal("jefke", retrievedSubaccount.Name);
+            Assert.Equal(email, retrievedSubaccount.Email);
+        }   
+
+        [Fact]
+        public void EmailIsOptional()
+        {
+            var email = RandomEmail();
+            var subaccount = Client.Subaccounts.CreateWithEmail(email, "joske");
+
             Client.Subaccounts.Update(subaccount.Id, "jefke");
 
             var retrievedSubaccount = Client.Subaccounts.Retrieve(subaccount.Id);
             Assert.Equal("jefke", retrievedSubaccount.Name);
+            Assert.Equal(email, retrievedSubaccount.Email);
+        }  
+
+        [Fact]
+        public void NameIsOptional()
+        {
+            var email = RandomEmail();
+            var subaccount = Client.Subaccounts.Create("joske");
+
+            Client.Subaccounts.Update(subaccount.Id, email: email);
+
+            var retrievedSubaccount = Client.Subaccounts.Retrieve(subaccount.Id);
+            Assert.Equal("joske", retrievedSubaccount.Name);
+            Assert.Equal(email, retrievedSubaccount.Email);
         }
     }
 }
diff --git a/SeatsioDotNet/SeatsioDotNet.csproj b/SeatsioDotNet/SeatsioDotNet.csproj
@@ -2,7 +2,7 @@
   <PropertyGroup>
     <OutputType>Library</OutputType>
     <IsPackable>true</IsPackable>
-    <Version>31</Version>
+    <Version>32</Version>
     <Authors>mroloux;bverbeken</Authors>
     <Title>Official Seats.io .NET API client</Title>
     <Description>Official Seats.io .NET API client</Description>

diff --git a/SeatsioDotNet/Subaccounts/Subaccounts.cs b/SeatsioDotNet/Subaccounts/Subaccounts.cs
@@ -33,12 +33,10 @@ public Subaccount CreateWithEmail(string email, string name = null)
         private Subaccount DoCreate(string email = null, string name = null)
         {
             var requestBody = new Dictionary<string, object>();
-
             if (email != null)
             {
                 requestBody.Add("email", email);
             }
-
             if (name != null)
             {
                 requestBody.Add("name", name);
@@ -56,11 +54,21 @@ public Subaccount Retrieve(long id)
             return AssertOk(_restClient.Execute<Subaccount>(restRequest));
         }
 
-        public void Update(long id, string name)
+        public void Update(long id, string name = null, string email = null)
         {
+            var requestBody = new Dictionary<string, object>();
+            if (email != null)
+            {
+                requestBody.Add("email", email);
+            }
+            if (name != null)
+            {
+                requestBody.Add("name", name);
+            }
+
             var restRequest = new RestRequest("/subaccounts/{id}", Method.POST)
                 .AddUrlSegment("id", id)
-                .AddJsonBody(new {name});
+                .AddJsonBody(requestBody);
             AssertOk(_restClient.Execute<object>(restRequest));
         }