Added tests for update

Author: Morten Turn Pedersen

NetCoreEntityFramework.Tests/EntityRepositoryTests.cs

@@ -1,3 +1,4 @@
+using AutoFixture.NUnit3;
 using Microsoft.EntityFrameworkCore;
 using Nodes.NetCore.EntityFramework.Tests.Mocks;
 using NUnit.Framework;
@@ -54,6 +55,7 @@ public void Setup()
             _repository = new TestEntityRepository(_context);
         }
 
+        #region Add
         [Test]
         public async Task AddAddsEntityAndSetsAttributes()
         {
@@ -90,6 +92,14 @@ public async Task AddEntityWithIdKeepsId()
             Assert.AreEqual(id, entity.Id);
         }
 
+        [Test]
+        public void AddThrowsExceptionIfEntityIsNull()
+        {
+            Assert.ThrowsAsync<ArgumentNullException>(() => _repository.Add(null));
+        }
+        #endregion
+
+        #region Get
         [Test]
         public async Task GetValidEntityReturnsEntity()
         {
@@ -113,5 +123,34 @@ public async Task GetDeletedEntityWithFlag()
 
             Assert.AreSame(_deletedEntity, entity);
         }
+        #endregion
+
+        #region Update
+        [Test]
+        [AutoData]
+        public async Task UpdateUpdatesUpdated(string propertyValue)
+        {
+            DateTime oldUpdated = _entity.Updated;
+            DateTime oldCreated = _entity.Created;
+            _entity.Property = propertyValue;
+
+            using(_repository)
+            {
+                await _repository.Update(_entity);
+            }
+
+            var entity = await _repository.Get((Guid)_entity.Id);
+
+            Assert.AreEqual(propertyValue, entity.Property);
+            Assert.AreNotEqual(oldUpdated, entity.Updated);
+            Assert.AreEqual(oldCreated, entity.Created);
+        }
+
+        [Test]
+        public void UpdateThrowsExceptionIfNull()
+        {
+            Assert.ThrowsAsync<ArgumentNullException>(() => _repository.Update(null));
+        }
+        #endregion
     }
 }

NetCoreEntityFramework.Tests/NetCoreEntityFramework.Tests.csproj

@@ -11,6 +11,7 @@
   </PropertyGroup>
 
   <ItemGroup>
+    <PackageReference Include="AutoFixture.NUnit3" Version="4.11.0" />
     <PackageReference Include="Microsoft.EntityFrameworkCore.InMemory" Version="3.1.3" />
     <PackageReference Include="nunit" Version="3.12.0" />
     <PackageReference Include="NUnit3TestAdapter" Version="3.16.1">

NetCoreEntityFramework/Repositories/EntityRepository.cs

@@ -113,7 +113,7 @@ public Task Add(TEntity entity)
         /// </summary>
         /// <param name="entity">The entity to update.</param>
         /// <exception cref="ArgumentNullException"></exception>
-        public Task<TEntity> Update(TEntity entity)
+        public Task Update(TEntity entity)
         {
             if (entity == null)
                 throw new ArgumentNullException(nameof(entity));
@@ -122,7 +122,7 @@ public Task<TEntity> Update(TEntity entity)
 
             Context.Update(entity);
 
-            return Task.FromResult(entity);
+            return Task.CompletedTask;
         }
 
         /// <summary>