Refactor message entity and add validation for text content length Resolves #4

Author: Raffaele Marcello

src/Application/src/RazorPagesTestSample/Data/Message.cs

@@ -1,6 +1,27 @@
 using System.ComponentModel.DataAnnotations;
 
-namespace RazorPagesTestSample.Data
+/// <summary>
+/// Represents a message entity with an ID and text content.
+/// </summary>
+public class Message
+{
+    /// <summary>
+    /// Gets or sets the unique identifier for the message.
+    /// </summary>
+    public int Id { get; set; }
+
+    /// <summary>
+    /// Gets or sets the text content of the message.
+    /// </summary>
+    /// <remarks>
+    /// The text content is required and has a maximum length of 250 characters.
+    /// </remarks>
+    [Required]
+    [DataType(DataType.Text)]
+    [StringLength(250, ErrorMessage = "There's a 250 character limit on messages. Please shorten your message.")]
+    public string Text { get; set; }
+}
+    /// /// /// /// /// /// /// namespace RazorPagesTestSample.Data
 {
     #region snippet1
     public class Message
@@ -9,7 +30,7 @@ public class Message
 
         [Required]
         [DataType(DataType.Text)]
-        [StringLength(200, ErrorMessage = "There's a 200 character limit on messages. Please shorten your message.")]
+        [StringLength(250, ErrorMessage = "There's a 250 character limit on messages. Please shorten your message.")]
         public string Text { get; set; }
     }
     #endregion

src/Application/tests/RazorPagesTestSample.Tests/UnitTests/DataAccessLayerTest.cs

@@ -25,7 +25,7 @@ public async Task GetMessagesAsync_MessagesAreReturned()
                 // Assert
                 var actualMessages = Assert.IsAssignableFrom<List<Message>>(result);
                 Assert.Equal(
-                    expectedMessages.OrderBy(m => m.Id).Select(m => m.Text), 
+                    expectedMessages.OrderBy(m => m.Id).Select(m => m.Text),
                     actualMessages.OrderBy(m => m.Id).Select(m => m.Text));
             }
         }
@@ -77,7 +77,7 @@ public async Task DeleteMessageAsync_MessageIsDeleted_WhenMessageIsFound()
                 await db.AddRangeAsync(seedMessages);
                 await db.SaveChangesAsync();
                 var recId = 1;
-                var expectedMessages = 
+                var expectedMessages =
                     seedMessages.Where(message => message.Id != recId).ToList();
                 #endregion
 
@@ -90,7 +90,7 @@ public async Task DeleteMessageAsync_MessageIsDeleted_WhenMessageIsFound()
                 // Assert
                 var actualMessages = await db.Messages.AsNoTracking().ToListAsync();
                 Assert.Equal(
-                    expectedMessages.OrderBy(m => m.Id).Select(m => m.Text), 
+                    expectedMessages.OrderBy(m => m.Id).Select(m => m.Text),
                     actualMessages.OrderBy(m => m.Id).Select(m => m.Text));
                 #endregion
             }
@@ -121,10 +121,36 @@ public async Task DeleteMessageAsync_NoMessageIsDeleted_WhenMessageIsNotFound()
                 // Assert
                 var actualMessages = await db.Messages.AsNoTracking().ToListAsync();
                 Assert.Equal(
-                    expectedMessages.OrderBy(m => m.Id).Select(m => m.Text), 
+                    expectedMessages.OrderBy(m => m.Id).Select(m => m.Text),
                     actualMessages.OrderBy(m => m.Id).Select(m => m.Text));
             }
         }
         #endregion
+
+        // This picks up inside the DataAccessLayerTest class.
+        [Theory]
+        [InlineData(150, true)]
+        [InlineData(199, true)]
+        [InlineData(200, true)]
+        [InlineData(201, true)]
+        [InlineData(249, true)]
+        [InlineData(250, true)]
+        [InlineData(251, false)]
+        [InlineData(300, false)]
+        public async Task AddMessageAsync_TestMessageLength(int messageLength, bool expectedValidMessage)
+        {
+            using (var db = new AppDbContext(Utilities.TestDbContextOptions()))
+            {
+                // Arrange
+                var recId = 10;
+                var expectedMessage = new Message() { Id = recId, Text = new string('X', messageLength) };
+
+                // Act
+                var isValidMessage = Validator.TryValidateObject(expectedMessage, new ValidationContext(expectedMessage), null, validateAllProperties: true);
+
+                // Assert
+                Assert.Equal(expectedValidMessage, isValidMessage);
+            }
+        }
     }
 }