35 edit command

BusyList/Commands/Commands.cs

@@ -7,4 +7,5 @@ public record NextCommand() : Command;
     public record AddCommand(string Description) : Command;
     public record DeleteCommand(int Id) : Command;
     public record DoneCommand(int Id) : Command;
+    public record EditCommand(int Id, string property, string value) : Command;
 }

BusyList/Handlers/DoneHandler.cs

@@ -16,7 +16,13 @@ public void Run(DoneCommand command)
         {
             TaskItem taskItem = _taskRepository.GetTaskById(command.Id);
 
-            taskItem.TaskStatus = TaskStatus.Done;
+            if(taskItem == null)
+            {
+                Console.WriteLine($"The task with the id {command.Id} does not exist");
+                return;
+            }
+
+            taskItem.Status = TaskStatus.Done;
 
             _taskRepository.UpdateTask(taskItem);
 

BusyList/Handlers/EditHandler.cs

@@ -0,0 +1,45 @@
+using BusyList.Commands;
+using System;
+using System.Reflection;
+
+namespace BusyList.Handlers
+{
+    // There are some problems. 
+    // Only work for string properties e.g description.
+    public class EditHandler : IHandler<EditCommand>
+    {
+        private readonly ITaskRepository _taskRepository;
+
+        public EditHandler(ITaskRepository taskRepository)
+        {
+            _taskRepository = taskRepository;
+        }
+
+        public void Run(EditCommand command)
+        {
+            TaskItem taskItem = _taskRepository.GetTaskById(command.Id);
+
+            if (taskItem != null)
+            {
+                PropertyInfo propertyInfo = taskItem.GetType().GetProperty(command.property);
+
+                if (propertyInfo != null)
+                {
+                    propertyInfo.SetValue(taskItem, command.value);
+
+                    _taskRepository.UpdateTask(taskItem);
+
+                    Console.WriteLine("Task updated");
+                }
+                else
+                {
+                    Console.WriteLine($"The property with the name {command.property} does not exist");
+                }
+            }
+            else 
+            {
+                Console.WriteLine($"The task with the id {command.Id} does not exist");
+            }
+        }
+    }
+}

BusyList/Handlers/NextHandler.cs

@@ -25,7 +25,7 @@ public void Run(NextCommand command)
             {
                 Console.WriteLine($"Task id: {item.Id}");
                 Console.WriteLine($"Description: {item.Description}");
-                Console.WriteLine($"Status: {item.TaskStatus}");
+                Console.WriteLine($"Status: {item.Status}");
                 Console.WriteLine(SEPERATOR);
             }
         }

BusyList/Parsing/CommandGrammar.cs

@@ -9,6 +9,9 @@ public static class CommandGrammar
         private static readonly Parser<string> _keywordAdd =
             Parse.IgnoreCase("add").Text();
 
+        private static readonly Parser<string> _keywordEdit =
+            Parse.IgnoreCase("edit").Text();
+
         private static readonly Parser<string> _keywordDelete =
             Parse.IgnoreCase("delete")
             .Or(Parse.IgnoreCase("del"))
@@ -49,10 +52,21 @@ from _ in Parse.WhiteSpace
             from keyword in _keywordDone
             select new DoneCommand(id);
 
+        private static readonly Parser<Command> _editCommand =
+            from id in _number
+            from _ in Parse.WhiteSpace
+            from keyword in _keywordEdit
+            from __ in Parse.WhiteSpace
+            from property in Parse.LetterOrDigit.AtLeastOnce().Text()
+            from ___ in Parse.WhiteSpace
+            from value in Parse.LetterOrDigit.AtLeastOnce().Text()
+            select new EditCommand(id, property, value);
+
         public static readonly Parser<Command> Source =
             _deleteCommand
             .Or(_nextCommand)
             .Or(_doneCommand)
+            .Or(_editCommand)
             .Or(_readCommand)
             .Or(_addCommand)
             .End();

BusyList/Program.cs

@@ -65,6 +65,7 @@ private static IServiceCollection ConfigureServices()
             services.AddTransient<IHandler<DoneCommand>, DoneHandler>();
             services.AddTransient<IHandler<NextCommand>, NextHandler>();
             services.AddTransient<IHandler<ReadCommand>, ReadHandler>();
+            services.AddTransient<IHandler<EditCommand>, EditHandler>();
 
             return services;
         }
@@ -88,6 +89,9 @@ private static void HandleCommand(ServiceProvider provider, Command command)
                 case ReadCommand read:
                     provider.GetRequiredService<IHandler<ReadCommand>>().Run(read);
                     break;
+                case EditCommand edit:
+                    provider.GetRequiredService<IHandler<EditCommand>>().Run(edit);
+                    break;
                 default:
                     throw new Exception($"Unknown command type {command.GetType().FullName} sent to HandleCommand!");
             }

BusyList/TaskItem.cs

@@ -8,21 +8,21 @@ public TaskItem(int id, string description, TaskStatus status = TaskStatus.NotSt
         {
             Id = id;
             Description = description;
-            TaskStatus = status;
+            Status = status;
         }
         public int Id { get; }
 
-        public string Description { get; }
+        public string Description { get; set; }
 
-        public TaskStatus TaskStatus { get; set; }
+        public TaskStatus Status { get; set; }
 
         public string Print()
         {
             var sb = new StringBuilder();
 
             sb.AppendLine($"Id {Id}");
             sb.AppendLine($"Description {Description}");
-            sb.AppendLine($"Status {TaskStatus}");
+            sb.AppendLine($"Status {Status}");
 
             return sb.ToString();
         }

tests/BusyList.Tests/Handlers/DoneHandlerTests.cs

@@ -25,7 +25,7 @@ public void Run_ShouldSetTaskStatusToDone_WhenAValidIdIsPassed()
             _taskRepository.Setup(_ => _.GetTaskById(command.Id)).Returns(currentTask);
 
             _taskRepository.Setup(_ => _.UpdateTask(It.IsAny<TaskItem>())).Callback<TaskItem>(task =>
-                task.TaskStatus.Should().Be(TaskStatus.Done)
+                task.Status.Should().Be(TaskStatus.Done)
             );
 
             _subject.Run(command);