Fix 'for' param, add some tests vknet#1610

xiadosw · Mar 15, 2024 · 68797a3 · 68797a3
1 parent 7bf8a7b
commit 68797a3
Show file tree

Hide file tree

Showing 5 changed files with 57 additions and 3 deletions.
diff --git a/VkNet.Tests/Categories/Messages/MessagesCategoryTest.cs b/VkNet.Tests/Categories/Messages/MessagesCategoryTest.cs
@@ -1122,10 +1122,10 @@ public void SetMemberRole()
 	}
 
 	[Fact]
-	public void ChangeConversationMemberRestrictions()
+	public void ChangeConversationMemberRestrictions_Mute()
 	{
 		Url = "https://api.vk.com/method/messages.changeConversationMemberRestrictions";
-		ReadCategoryJsonPath(nameof(ChangeConversationMemberRestrictions));
+		ReadCategoryJsonPath(nameof(ChangeConversationMemberRestrictions_Mute));
 
 		var response = Api.Messages.ChangeConversationMemberRestrictions(new()
 		{
@@ -1142,4 +1142,46 @@ public void ChangeConversationMemberRestrictions()
 			.BeEquivalentTo(new List<long> { 814412 });
 
 	}
+
+	[Fact]
+	public void ChangeConversationMemberRestrictions_MuteForever()
+	{
+		Url = "https://api.vk.com/method/messages.changeConversationMemberRestrictions";
+		ReadCategoryJsonPath(nameof(ChangeConversationMemberRestrictions_MuteForever));
+
+		var response = Api.Messages.ChangeConversationMemberRestrictions(new()
+		{
+			PeerId = 1,
+			MemberIds = new List<long>
+			{
+				814412, 4245645
+			},
+			Action = ConversationMemberRestrictionsActionType.Ro
+		});
+
+		response.FailedMemberIds.Should()
+			.BeEquivalentTo(new List<long> { 814412 });
+
+	}
+
+	[Fact]
+	public void ChangeConversationMemberRestrictions_Unmute()
+	{
+		Url = "https://api.vk.com/method/messages.changeConversationMemberRestrictions";
+		ReadCategoryJsonPath(nameof(ChangeConversationMemberRestrictions_Unmute));
+
+		var response = Api.Messages.ChangeConversationMemberRestrictions(new()
+		{
+			PeerId = 1,
+			MemberIds = new List<long>
+			{
+				4245645
+			},
+			Action = ConversationMemberRestrictionsActionType.Rw
+		});
+
+		response.FailedMemberIds.Should()
+			.BeEmpty();
+
+	}
 }
diff --git a/...ChangeConversationMemberRestrictions.json → ...eConversationMemberRestrictions_Mute.json b/...ChangeConversationMemberRestrictions.json → ...eConversationMemberRestrictions_Mute.json
diff --git a/....Tests/TestData/Categories/Messages/ChangeConversationMemberRestrictions_MuteForever.json b/....Tests/TestData/Categories/Messages/ChangeConversationMemberRestrictions_MuteForever.json
@@ -0,0 +1,7 @@
+{
+	"response": {
+		"failed_member_ids": [
+			814412
+		]
+	}
+}
diff --git a/VkNet.Tests/TestData/Categories/Messages/ChangeConversationMemberRestrictions_Unmute.json b/VkNet.Tests/TestData/Categories/Messages/ChangeConversationMemberRestrictions_Unmute.json
@@ -0,0 +1,5 @@
+{
+	"response": {
+		"failed_member_ids": []
+	}
+}
diff --git a/VkNet/Model/RequestParams/Messages/MessagesChangeConversationMemberRestrictionsParams.cs b/VkNet/Model/RequestParams/Messages/MessagesChangeConversationMemberRestrictionsParams.cs
@@ -27,7 +27,7 @@ public class MessagesChangeConversationMemberRestrictionsParams
 	/// Время в секундах. Если нужно замутить навсегда, то указывать не нужно.
 	/// </summary>
 	[JsonProperty("for", NullValueHandling = NullValueHandling.Ignore)]
-	public long For { get; set; }
+	public long? For { get; set; }
 
 	/// <summary>
 	/// Разрешенные действия.