Fix move project request (#22)

* Fix move project request
olsh · Dec 27, 2022 · 2dc4a37 · 2dc4a37
1 parent ccf8c89
commit 2dc4a37
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 4 deletions.
diff --git a/src/Todoist.Net.Tests/Services/ProjectsServiceTests.cs b/src/Todoist.Net.Tests/Services/ProjectsServiceTests.cs
@@ -59,10 +59,14 @@ public void CreateUpdateOrderMoveAndDelete_Success()
 
             client.Projects.ReorderAsync(new ReorderEntry(project.Id, 1)).Wait();
 
-            // TODO: Fix the request
-            //client.Projects.MoveAsync(new MoveArgument(project.Id, null)).Wait();
+            var parentProjectName = Guid.NewGuid().ToString();
+            var parentProject = new Project(parentProjectName);
+            client.Projects.AddAsync(parentProject).Wait();
+
+            client.Projects.MoveAsync(new MoveArgument(project.Id, parentProject.Id)).Wait();
 
             client.Projects.DeleteAsync(project.Id).Wait();
+            client.Projects.DeleteAsync(parentProject.Id).Wait();
         }
 
         [Fact]

diff --git a/src/Todoist.Net/Models/MoveArgument.cs b/src/Todoist.Net/Models/MoveArgument.cs
@@ -15,14 +15,19 @@ public class MoveArgument : BaseEntity
         /// <param name="id">The identifier of moved entity.</param>
         /// <param name="parentId">The parent entity identifier.</param>
         /// <exception cref="T:System.ArgumentException">Entity ID is required for the operation</exception>
-        public MoveArgument(ComplexId id, ComplexId? parentId)
+        public MoveArgument(ComplexId id, ComplexId parentId)
             : base(id)
         {
             if (id.IsEmpty)
             {
                 throw new ArgumentException("Entity ID is required for the move operation", nameof(id));
             }
 
+            if (parentId.IsEmpty)
+            {
+                throw new ArgumentException("Parent ID is required for the move operation", nameof(parentId));
+            }
+
             ParentId = parentId;
         }
 
@@ -33,6 +38,6 @@ internal MoveArgument()
         /// <summary>Gets the parent entity identifier.</summary>
         /// <value>The parent entity identifier.</value>
         [JsonProperty("parent_id")]
-        public ComplexId? ParentId { get; internal set; }
+        public ComplexId ParentId { get; internal set; }
     }
 }