Add Tests for UpdateAsyncWithInterface

Fix TestDeleteAsync to await, Sleep was not alway sufficient

Author: jonathanlarouche

Dapper.SimpleCRUD/SimpleCRUDAsync.cs

@@ -331,9 +331,9 @@ public static async Task<int> UpdateAsync<TEntity>(this IDbConnection connection
             if (typeof(TEntity).IsInterface) //FallBack to BaseType Generic Method : https://stackoverflow.com/questions/4101784/calling-a-generic-method-with-a-dynamic-type
             {
                 return await(Task<int>)typeof(SimpleCRUD)
-                   .GetMethods().Where(methodInfo => methodInfo.Name == nameof(UpdateAsync) && methodInfo.GetGenericArguments().Count() == 2).Single()
+                   .GetMethods().Where(methodInfo => methodInfo.Name == nameof(UpdateAsync) && methodInfo.GetGenericArguments().Count() == 1).Single()
                    .MakeGenericMethod(new Type[] { entityToUpdate.GetType() })
-                   .Invoke(null, new object[] { connection, entityToUpdate, transaction, commandTimeout });
+                   .Invoke(null, new object[] { connection, entityToUpdate, transaction, commandTimeout, token });
             }
             var idProps = GetIdProperties(entityToUpdate).ToList();
 

Dapper.SimpleCRUDTests/Tests.cs

@@ -815,14 +815,13 @@ public void TestMultipleKeyGetAsync()
             }
         }
 
-        public void TestDeleteByIdAsync()
+        public async void TestDeleteByIdAsync()
         {
             using (var connection = GetOpenConnection())
             {
                 var id = connection.Insert(new User { Name = "UserAsyncDelete", Age = 10 });
-                connection.DeleteAsync<User>(id);
+                await connection.DeleteAsync<User>(id);
                 //tiny wait to let the delete happen
-                System.Threading.Thread.Sleep(300);
                 connection.Get<User>(id).IsNull();
             }
         }
@@ -1396,6 +1395,41 @@ public void TestUpdateUsingInterface()
             }
         }
 
+        public async void TestUpdateAsyncUsingInterface()
+        {
+            using (var connection = GetOpenConnection())
+            using (var transaction = connection.BeginTransaction())
+            {
+                INameColumn newUser = new UserWithIName
+                {
+                    Age = 40,
+                    Name = "Jonathan Larouche",
+                    ScheduledDayOff = DayOfWeek.Sunday,
+                    CreatedDate = new DateTime(2000, 1, 1)
+                };
+
+                ((UserWithIName)newUser).Id = connection.Insert(newUser, transaction).Value;
+                ((UserWithIName)newUser).Age = 41;
+                await connection.UpdateAsync(newUser, transaction);
+
+                INameColumn newCity = new CityWithIName
+                {
+                    Name = "Montreal",
+                    Population = 5675
+                };
+
+                connection.Insert<string, INameColumn>(newCity, transaction);
+                ((CityWithIName)newCity).Population = 6000;
+                await connection.UpdateAsync(newCity, transaction);
+
+                var user = connection.GetList<UserWithIName>(new { Name = "Jonathan Larouche" }, transaction).FirstOrDefault();
+                user.Age.IsEqualTo(41);
+                var city = connection.GetList<CityWithIName>(new { Name = "Montreal" }, transaction).FirstOrDefault();
+                city.Population.IsEqualTo(6000);
+
+            }
+        }
+
         //ignore attribute tests
         //i cheated here and stuffed all of these in one test
         //didn't implement in postgres or mysql tests yet