Made user id required for methods inside RoomMembershipApi.

jpfigueiras · jpfigueiras · commit 388086ae7402 · 2016-06-21T09:13:44.000+01:00
diff --git a/src/SymphonyOSS.RestApiClient/Api/PodApi/RoomMembershipApi.cs b/src/SymphonyOSS.RestApiClient/Api/PodApi/RoomMembershipApi.cs
@@ -55,7 +55,7 @@ public RoomMembershipApi(IAuthTokens authTokens, Configuration configuration, IA
         /// <param name="roomId">The id of the room.</param>
         /// <param name="userId">The id of the user to add to the room.</param>
         /// <returns></returns>
-        public SuccessResponse AddMemberToRoom(string roomId, long? userId)
+        public SuccessResponse AddMemberToRoom(string roomId, long userId)
         {
             return _apiExecutor.Execute(_roomMembershipApi.V1RoomIdMembershipAddPost, roomId, new UserId(userId), _authTokens.SessionToken);
         }
@@ -66,7 +66,7 @@ public SuccessResponse AddMemberToRoom(string roomId, long? userId)
         /// <param name="roomId">The id of the room.</param>
         /// <param name="userId">The id of the user to add to the room.</param>
         /// <returns></returns>
-        public SuccessResponse RemoveMemberFromRoom(string roomId, long? userId)
+        public SuccessResponse RemoveMemberFromRoom(string roomId, long userId)
         {
             return _apiExecutor.Execute(_roomMembershipApi.V1RoomIdMembershipRemovePost, roomId, new UserId(userId), _authTokens.SessionToken);
         }
@@ -77,7 +77,7 @@ public SuccessResponse RemoveMemberFromRoom(string roomId, long? userId)
         /// <param name="roomId">The id of the room.</param>
         /// <param name="userId">The id of the user to add to the room.</param>
         /// <returns></returns>
-        public SuccessResponse PromoteUserToRoomOwner(string roomId, long? userId)
+        public SuccessResponse PromoteUserToRoomOwner(string roomId, long userId)
         {
             return _apiExecutor.Execute(_roomMembershipApi.V1RoomIdMembershipPromoteOwnerPost, roomId, new UserId(userId), _authTokens.SessionToken);
         }
@@ -88,7 +88,7 @@ public SuccessResponse PromoteUserToRoomOwner(string roomId, long? userId)
         /// <param name="roomId">The id of the room.</param>
         /// <param name="userId">The id of the user to add to the room.</param>
         /// <returns></returns>
-        public SuccessResponse DemoteRoomOwner(string roomId, long? userId)
+        public SuccessResponse DemoteRoomOwner(string roomId, long userId)
         {
             return _apiExecutor.Execute(_roomMembershipApi.V1RoomIdMembershipDemoteOwnerPost, roomId, new UserId(userId), _authTokens.SessionToken);
         }
diff --git a/test/SymphonyOSS.RestApiClient.Tests/RoomMembershipApiTest.cs b/test/SymphonyOSS.RestApiClient.Tests/RoomMembershipApiTest.cs
@@ -31,7 +31,7 @@ public RoomMembershipApiTest()
         public void EnsureAddMemberToRoom_uses_retry_strategy()
         {
             var roomId = "some_room";
-            long? userId = 123456789;
+            long userId = 123456789;
             _roomMembershipApi.AddMemberToRoom(roomId, userId);
             _apiExecutorMock.Verify(obj => obj.Execute(It.IsAny<Func<string, long?, string, SuccessResponse>>(), roomId, userId, "sessionToken"));
         }
@@ -40,7 +40,7 @@ public void EnsureAddMemberToRoom_uses_retry_strategy()
         public void EnsureRemoveMemberFromRoom_uses_retry_strategy()
         {
             var roomId = "some_room";
-            long? userId = 123456789;
+            long userId = 123456789;
             _roomMembershipApi.RemoveMemberFromRoom(roomId, userId);
             _apiExecutorMock.Verify(obj => obj.Execute(It.IsAny<Func<string, long?, string, SuccessResponse>>(), roomId, userId, "sessionToken"));
         }
@@ -49,7 +49,7 @@ public void EnsureRemoveMemberFromRoom_uses_retry_strategy()
         public void EnsurePromoteUserToRoomOwner_uses_retry_strategy()
         {
             var roomId = "some_room";
-            long? userId = 123456789;
+            long userId = 123456789;
             _roomMembershipApi.PromoteUserToRoomOwner(roomId, userId);
             _apiExecutorMock.Verify(obj => obj.Execute(It.IsAny<Func<string, long?, string, SuccessResponse>>(), roomId, userId, "sessionToken"));
         }
@@ -58,7 +58,7 @@ public void EnsurePromoteUserToRoomOwner_uses_retry_strategy()
         public void EnsureDemoteRoomOwner_uses_retry_strategy()
         {
             var roomId = "some_room";
-            long? userId = 123456789;
+            long userId = 123456789;
             _roomMembershipApi.DemoteRoomOwner(roomId, userId);
             _apiExecutorMock.Verify(obj => obj.Execute(It.IsAny<Func<string, long?, string, SuccessResponse>>(), roomId, userId, "sessionToken"));
         }

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,7 @@ public RoomMembershipApi(IAuthTokens authTokens, Configuration configuration, IA`
`55`	`55`	`/// <param name="roomId">The id of the room.</param>`
`56`	`56`	`/// <param name="userId">The id of the user to add to the room.</param>`
`57`	`57`	`/// <returns></returns>`
`58`		`- public SuccessResponse AddMemberToRoom(string roomId, long? userId)`
	`58`	`+ public SuccessResponse AddMemberToRoom(string roomId, long userId)`
`59`	`59`	`{`
`60`	`60`	`return _apiExecutor.Execute(_roomMembershipApi.V1RoomIdMembershipAddPost, roomId, new UserId(userId), _authTokens.SessionToken);`
`61`	`61`	`}`
`@@ -66,7 +66,7 @@ public SuccessResponse AddMemberToRoom(string roomId, long? userId)`
`66`	`66`	`/// <param name="roomId">The id of the room.</param>`
`67`	`67`	`/// <param name="userId">The id of the user to add to the room.</param>`
`68`	`68`	`/// <returns></returns>`
`69`		`- public SuccessResponse RemoveMemberFromRoom(string roomId, long? userId)`
	`69`	`+ public SuccessResponse RemoveMemberFromRoom(string roomId, long userId)`
`70`	`70`	`{`
`71`	`71`	`return _apiExecutor.Execute(_roomMembershipApi.V1RoomIdMembershipRemovePost, roomId, new UserId(userId), _authTokens.SessionToken);`
`72`	`72`	`}`
`@@ -77,7 +77,7 @@ public SuccessResponse RemoveMemberFromRoom(string roomId, long? userId)`
`77`	`77`	`/// <param name="roomId">The id of the room.</param>`
`78`	`78`	`/// <param name="userId">The id of the user to add to the room.</param>`
`79`	`79`	`/// <returns></returns>`
`80`		`- public SuccessResponse PromoteUserToRoomOwner(string roomId, long? userId)`
	`80`	`+ public SuccessResponse PromoteUserToRoomOwner(string roomId, long userId)`
`81`	`81`	`{`
`82`	`82`	`return _apiExecutor.Execute(_roomMembershipApi.V1RoomIdMembershipPromoteOwnerPost, roomId, new UserId(userId), _authTokens.SessionToken);`
`83`	`83`	`}`
`@@ -88,7 +88,7 @@ public SuccessResponse PromoteUserToRoomOwner(string roomId, long? userId)`
`88`	`88`	`/// <param name="roomId">The id of the room.</param>`
`89`	`89`	`/// <param name="userId">The id of the user to add to the room.</param>`
`90`	`90`	`/// <returns></returns>`
`91`		`- public SuccessResponse DemoteRoomOwner(string roomId, long? userId)`
	`91`	`+ public SuccessResponse DemoteRoomOwner(string roomId, long userId)`
`92`	`92`	`{`
`93`	`93`	`return _apiExecutor.Execute(_roomMembershipApi.V1RoomIdMembershipDemoteOwnerPost, roomId, new UserId(userId), _authTokens.SessionToken);`
`94`	`94`	`}`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ public RoomMembershipApiTest()`
`31`	`31`	`public void EnsureAddMemberToRoom_uses_retry_strategy()`
`32`	`32`	`{`
`33`	`33`	`var roomId = "some_room";`
`34`		`- long? userId = 123456789;`
	`34`	`+ long userId = 123456789;`
`35`	`35`	`_roomMembershipApi.AddMemberToRoom(roomId, userId);`
`36`	`36`	`_apiExecutorMock.Verify(obj => obj.Execute(It.IsAny<Func<string, long?, string, SuccessResponse>>(), roomId, userId, "sessionToken"));`
`37`	`37`	`}`
`@@ -40,7 +40,7 @@ public void EnsureAddMemberToRoom_uses_retry_strategy()`
`40`	`40`	`public void EnsureRemoveMemberFromRoom_uses_retry_strategy()`
`41`	`41`	`{`
`42`	`42`	`var roomId = "some_room";`
`43`		`- long? userId = 123456789;`
	`43`	`+ long userId = 123456789;`
`44`	`44`	`_roomMembershipApi.RemoveMemberFromRoom(roomId, userId);`
`45`	`45`	`_apiExecutorMock.Verify(obj => obj.Execute(It.IsAny<Func<string, long?, string, SuccessResponse>>(), roomId, userId, "sessionToken"));`
`46`	`46`	`}`
`@@ -49,7 +49,7 @@ public void EnsureRemoveMemberFromRoom_uses_retry_strategy()`
`49`	`49`	`public void EnsurePromoteUserToRoomOwner_uses_retry_strategy()`
`50`	`50`	`{`
`51`	`51`	`var roomId = "some_room";`
`52`		`- long? userId = 123456789;`
	`52`	`+ long userId = 123456789;`
`53`	`53`	`_roomMembershipApi.PromoteUserToRoomOwner(roomId, userId);`
`54`	`54`	`_apiExecutorMock.Verify(obj => obj.Execute(It.IsAny<Func<string, long?, string, SuccessResponse>>(), roomId, userId, "sessionToken"));`
`55`	`55`	`}`
`@@ -58,7 +58,7 @@ public void EnsurePromoteUserToRoomOwner_uses_retry_strategy()`
`58`	`58`	`public void EnsureDemoteRoomOwner_uses_retry_strategy()`
`59`	`59`	`{`
`60`	`60`	`var roomId = "some_room";`
`61`		`- long? userId = 123456789;`
	`61`	`+ long userId = 123456789;`
`62`	`62`	`_roomMembershipApi.DemoteRoomOwner(roomId, userId);`
`63`	`63`	`_apiExecutorMock.Verify(obj => obj.Execute(It.IsAny<Func<string, long?, string, SuccessResponse>>(), roomId, userId, "sessionToken"));`
`64`	`64`	`}`