chore: modify MaaAgentClient.ctor() accessibility

Author: moomiji

src/MaaFramework.Binding.Native/MaaAgentClient.cs

@@ -29,7 +29,7 @@ public class MaaAgentClient : MaaDisposableHandle<MaaAgentClientHandle>, IMaaAge
     /// <remarks>
     ///     Wrapper of <see cref="MaaAgentClientCreate"/>.
     /// </remarks>
-    public MaaAgentClient(string identifier = "")
+    protected MaaAgentClient(string identifier = "")
         : base(invalidHandleValue: MaaAgentClientHandle.Zero)
     {
         var handle = MaaAgentClientCreate();
@@ -40,35 +40,18 @@ public MaaAgentClient(string identifier = "")
             _ = Id.ThrowIfNotEquals(identifier);
     }
 
-    /// <param name="identifier">The unique identifier used to communicate with the agent server.</param>
-    /// <param name="resource">The resource.</param>
-    /// <inheritdoc cref="MaaAgentClient(string)"/>
-    [SetsRequiredMembers]
-    public MaaAgentClient(string identifier, MaaResource resource)
-        : this(identifier)
-    {
-        Resource = resource;
-    }
-
-    /// <inheritdoc cref="MaaAgentClient(string, MaaResource)"/>
-    [SetsRequiredMembers]
-    public MaaAgentClient(MaaResource resource)
-        : this("", resource)
-    {
-    }
-
     /// <summary>
     ///     Creates a <see cref="MaaAgentClient"/> instance.
     /// </summary>
     /// <param name="identifier">The unique identifier used to communicate with the agent server.</param>
     /// <param name="resource">The resource.</param>
     /// <returns>The <see cref="MaaAgentClient"/> instance.</returns>
     public static MaaAgentClient Create(string identifier, MaaResource resource)
-        => new(identifier, resource);
+        => new(identifier) { Resource = resource, };
 
     /// <inheritdoc cref="Create(string, MaaResource)"/>
     public static MaaAgentClient Create(MaaResource resource)
-        => new(resource);
+        => new() { Resource = resource, };
 
     /// <inheritdoc/>
     public string Id { get; }

src/MaaFramework.Binding.UnitTests/Test_IMaaAgentClient.cs

@@ -13,7 +13,7 @@ public class Test_IMaaAgentClient
     public static Dictionary<MaaTypes, object> NewData => new()
     {
 #if MAA_NATIVE
-        { MaaTypes.Native, new MaaAgentClient(new MaaResource()) },
+        { MaaTypes.Native, MaaAgentClient.Create(new MaaResource()) },
 #endif
     };
     public static Dictionary<MaaTypes, object> Data { get; private set; } = default!;
@@ -39,11 +39,9 @@ public void CreateInstances()
     {
 #if MAA_NATIVE
         var newId = Guid.NewGuid().ToString();
-        using var native1 = new MaaAgentClient(/*identifier = string.Empty*/) { Resource = new MaaResource() };
-        using var native2 = new MaaAgentClient(/*identifier = string.Empty*/ new MaaResource());
-        //    var native3 = new MaaAgentClient(newId, new MaaResource());
+        using var native1 = MaaAgentClient.Create(new MaaResource());
+        using var native2 = MaaAgentClient.Create(new MaaResource());
         using var native3 = MaaAgentClient.Create(newId, new MaaResource());
-        using var native4 = MaaAgentClient.Create(new MaaResource());
 
         Assert.AreNotEqual(native1.Id, native2.Id);
         Assert.AreEqual(newId, native3.Id);