From 8e986191b5e9e01f6d0bfa60abef737215118352 Mon Sep 17 00:00:00 2001
From: Padhraic <pcoogan@gmail.com>
Date: Wed, 18 Nov 2015 11:25:01 +0900
Subject: [PATCH 1/4] Add code first unit test

---
 .../EntityFramework.Extended.Test/App.config  |  13 +-
 .../CodeFirst/EFExtendedCodeFirstTest.cs      | 163 ++++++++++++++++++
 ...EntityFramework.Extended.Test.net45.csproj |  13 +-
 3 files changed, 187 insertions(+), 2 deletions(-)
 create mode 100644 Source/EntityFramework.Extended.Test/CodeFirst/EFExtendedCodeFirstTest.cs

diff --git a/Source/EntityFramework.Extended.Test/App.config b/Source/EntityFramework.Extended.Test/App.config
index daf2d39..cb46948 100644
--- a/Source/EntityFramework.Extended.Test/App.config
+++ b/Source/EntityFramework.Extended.Test/App.config
@@ -2,8 +2,11 @@
 <configuration>
   <configSections>
     <section name="entityFramework" type="System.Data.Entity.Internal.ConfigFile.EntityFrameworkSection, EntityFramework, Version=6.0.0.0, Culture=neutral, PublicKeyToken=b77a5c561934e089" requirePermission="false" />
-    
+
   <!-- For more information on Entity Framework configuration, visit http://go.microsoft.com/fwlink/?LinkID=237468 --></configSections>
+  <connectionStrings>
+    <add name="EFExtendedCodeFirstTest" connectionString="Data Source=(LocalDb)\ProjectsV12;Initial Catalog=EFExtendedCodeFirstTest;Integrated Security=SSPI;" providerName="System.Data.SqlClient" />
+  </connectionStrings>
   <entityFramework>
     <defaultConnectionFactory type="System.Data.Entity.Infrastructure.LocalDbConnectionFactory, EntityFramework">
       <parameters>
@@ -14,4 +17,12 @@
       <provider invariantName="System.Data.SqlClient" type="System.Data.Entity.SqlServer.SqlProviderServices, EntityFramework.SqlServer" />
     </providers>
   </entityFramework>
+  <runtime>
+    <assemblyBinding>
+      <dependentAssembly  xmlns="urn:schemas-microsoft-com:asm.v1">
+        <assemblyIdentity name="Utile.Money" publicKeyToken="null" culture="neutral" />
+        <bindingRedirect oldVersion="0.0.0.0-1.0.1.1" newVersion="1.0.1.1" />
+      </dependentAssembly>
+    </assemblyBinding>
+  </runtime>
 </configuration>
\ No newline at end of file
diff --git a/Source/EntityFramework.Extended.Test/CodeFirst/EFExtendedCodeFirstTest.cs b/Source/EntityFramework.Extended.Test/CodeFirst/EFExtendedCodeFirstTest.cs
new file mode 100644
index 0000000..2d94650
--- /dev/null
+++ b/Source/EntityFramework.Extended.Test/CodeFirst/EFExtendedCodeFirstTest.cs
@@ -0,0 +1,163 @@
+using System.ComponentModel.DataAnnotations;
+using System.Data.Entity;
+using System.Linq;
+using EntityFramework.Extensions;
+using Microsoft.VisualStudio.TestTools.UnitTesting;
+using System.ComponentModel.DataAnnotations.Schema;
+using EntityFramework.Audit;
+using System.Data;
+using System.Data.Entity.Core;
+
+//using Utile.Money;
+
+namespace EntityFramework.Test.CodeFirst
+{
+    [ComplexType]
+    public class Money
+    {
+        public Money()
+        {
+            CurrencyCode = "USD";
+            Amount = 0;
+        }
+        public Money(decimal amount,string currencyCode)
+        {
+            CurrencyCode = currencyCode;
+            Amount = amount;
+        }
+        public string CurrencyCode { get; set; }
+        public decimal Amount { get; set; }
+
+    }
+    public class Transaction
+    {
+        [Key]
+        [DatabaseGenerated(DatabaseGeneratedOption.Identity)]
+        public int Id { get; set; }
+        public Money Money { get; set; }
+        public string Detail { get; set; }
+    }
+
+    public class EFExtendedEntities : DbContext
+    {
+        public EFExtendedEntities() : base("EFExtendedCodeFirstTest")
+        {
+
+        }
+        public DbSet<Transaction> Transactions { get; set; }
+    }
+
+    public class TestsInitializer : DropCreateDatabaseAlways<EFExtendedEntities>
+    {
+        protected override void Seed(EFExtendedEntities ctx)
+        {
+            //add a transaction
+            var trx = new Transaction
+            {
+                Money = new Money( 123456789012.3456m,"USD"),
+                Detail = "First Transaction"
+            };
+            ctx.Transactions.Add(trx);
+            ctx.SaveChanges();
+            base.Seed(ctx);
+        }
+    }
+
+    [TestClass]
+    public class EFExtendedCodeFirstTest
+    {
+        private EFExtendedEntities _ctx;
+
+        [TestInitialize]
+        public void Init()
+        {
+            var auditConfiguration = AuditConfiguration.Default;
+
+            auditConfiguration.IncludeRelationships = true;
+            auditConfiguration.LoadRelationships = true;
+            auditConfiguration.DefaultAuditable = true;
+
+            Database.SetInitializer(new TestsInitializer());
+            _ctx = new EFExtendedEntities();
+        }
+        
+
+        [TestMethod]
+        public void EFExtendedCodeFirst_toXml()
+        {
+            // Arrange
+            var trx = new Transaction
+            {
+                Money = new Money(123456789012.3456m, "USD"),// complex type
+                Detail = "Another Transaction"
+            };
+            var audit = _ctx.BeginAudit();
+            _ctx.Transactions.Add(trx);
+            _ctx.SaveChanges();
+            var log = audit.LastLog;
+            //Act
+            var xml = log.ToXml();//Exception thrown here
+
+            // Assert
+            Assert.IsTrue(!string.IsNullOrEmpty(xml), "xml is not null or blank");
+        }
+
+        [TestMethod]
+        public void EFExtendedCodeFirst_Edit_Entities_not_empty_after_Complex_type_edit()
+        {
+            // Arrange
+            var trx = new Transaction
+            {
+                Money = new Money(123456789012.3456m, "USD"),
+                Detail = "Another Transaction"
+            };
+            _ctx.Transactions.Add(trx);
+            _ctx.SaveChanges();
+
+            //Act
+            var audit = _ctx.BeginAudit();
+            trx.Money.Amount = 10;
+            var t = _ctx.Set<Transaction>().FirstOrDefault(x => x.Id == trx.Id);
+            _ctx.Entry(t).CurrentValues.SetValues(trx);
+            _ctx.Entry(t).State = EntityState.Modified;
+            _ctx.SaveChanges();
+            var log = audit.LastLog;
+            
+            // Assert
+            Assert.AreEqual(1, log.Entities.Count, "Change to Money was reconised by aduit");
+
+        }
+
+        [TestMethod]
+        public void EFExtendedCodeFirst_Edit_Properties_contains_ComplexType_class()
+        {
+            // Arrange
+            var trx = new Transaction
+            {
+                Money = new Money(123456789012.3456m, "USD"),
+                Detail = "Another Transaction"
+            };
+            _ctx.Transactions.Add(trx);
+            _ctx.SaveChanges();
+
+            //Act
+            var audit = _ctx.BeginAudit();
+            //trx.Money.Amount = 10;
+            trx.Detail = "updated detail";
+            var t = _ctx.Set<Transaction>().FirstOrDefault(x => x.Id == trx.Id);
+            _ctx.Entry(t).CurrentValues.SetValues(trx);
+            _ctx.Entry(t).State = EntityState.Modified;
+            _ctx.SaveChanges();
+            var log = audit.LastLog;
+
+            // Assert
+
+            var obj = log.Entities[0].Properties;
+            Assert.AreEqual(3, obj.Count,"count equals number of properties in Transaction");
+
+            var tran = (IExtendedDataRecord) obj[2].Current;
+
+        }
+
+    }
+}
diff --git a/Source/EntityFramework.Extended.Test/EntityFramework.Extended.Test.net45.csproj b/Source/EntityFramework.Extended.Test/EntityFramework.Extended.Test.net45.csproj
index 25227f2..81368f9 100644
--- a/Source/EntityFramework.Extended.Test/EntityFramework.Extended.Test.net45.csproj
+++ b/Source/EntityFramework.Extended.Test/EntityFramework.Extended.Test.net45.csproj
@@ -60,6 +60,7 @@
       <Private>True</Private>
     </Reference>
     <Reference Include="Microsoft.CSharp" />
+    <Reference Include="Microsoft.VisualStudio.QualityTools.UnitTestFramework, Version=10.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a, processorArchitecture=MSIL" />
     <Reference Include="System" />
     <Reference Include="System.ComponentModel.DataAnnotations" />
     <Reference Include="System.Core">
@@ -72,6 +73,10 @@
     <Reference Include="System.Security" />
     <Reference Include="System.Xml" />
     <Reference Include="System.Xml.Linq" />
+    <Reference Include="Utile.Money, Version=1.0.1.1, Culture=neutral, processorArchitecture=MSIL">
+      <SpecificVersion>False</SpecificVersion>
+      <HintPath>..\..\..\Utile.Money\Utile.Money\bin\Debug\Utile.Money.dll</HintPath>
+    </Reference>
     <Reference Include="xunit">
       <HintPath>..\packages\xunit.1.9.2\lib\net20\xunit.dll</HintPath>
     </Reference>
@@ -86,6 +91,7 @@
     <Compile Include="Caching\CacheManagerTest.cs" />
     <Compile Include="Caching\CachePolicyTest.cs" />
     <Compile Include="Caching\CacheTagTest.cs" />
+    <Compile Include="CodeFirst\EFExtendedCodeFirstTest.cs" />
     <Compile Include="ContainerTest.cs" />
     <Compile Include="ExceptionAssert.cs" />
     <Compile Include="IoCTest.cs" />
@@ -109,9 +115,14 @@
       <DependentUpon>App.config</DependentUpon>
       <CopyToOutputDirectory>Always</CopyToOutputDirectory>
     </None>
-    <None Include="App.config" />
+    <None Include="App.config">
+      <SubType>Designer</SubType>
+    </None>
     <None Include="packages.config" />
   </ItemGroup>
+  <ItemGroup>
+    <Service Include="{82A7F48D-3B50-4B1E-B82E-3ADA8210C358}" />
+  </ItemGroup>
   <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
   <!-- To modify your build process, add your task inside one of the targets below and uncomment it. 
        Other similar extension points exist, see Microsoft.Common.targets.

From 813291c4ccc9ad8990fbb31e494a8bc3b3d784a3 Mon Sep 17 00:00:00 2001
From: Padhraic <pcoogan@gmail.com>
Date: Wed, 18 Nov 2015 11:39:14 +0900
Subject: [PATCH 2/4] Remove unnessary assembly reference

---
 Source/EntityFramework.Extended.Test/App.config | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/Source/EntityFramework.Extended.Test/App.config b/Source/EntityFramework.Extended.Test/App.config
index cb46948..e3239cb 100644
--- a/Source/EntityFramework.Extended.Test/App.config
+++ b/Source/EntityFramework.Extended.Test/App.config
@@ -17,12 +17,4 @@
       <provider invariantName="System.Data.SqlClient" type="System.Data.Entity.SqlServer.SqlProviderServices, EntityFramework.SqlServer" />
     </providers>
   </entityFramework>
-  <runtime>
-    <assemblyBinding>
-      <dependentAssembly  xmlns="urn:schemas-microsoft-com:asm.v1">
-        <assemblyIdentity name="Utile.Money" publicKeyToken="null" culture="neutral" />
-        <bindingRedirect oldVersion="0.0.0.0-1.0.1.1" newVersion="1.0.1.1" />
-      </dependentAssembly>
-    </assemblyBinding>
-  </runtime>
 </configuration>
\ No newline at end of file

From 32150038ef190d54acbe3c77ea34bd9d8a17797c Mon Sep 17 00:00:00 2001
From: Padhraic <pcoogan@gmail.com>
Date: Wed, 18 Nov 2015 14:26:40 +0900
Subject: [PATCH 3/4] Add complex type property update unit test

---
 .../CodeFirst/EFExtendedCodeFirstTest.cs      | 33 ++++++++++++++++++-
 1 file changed, 32 insertions(+), 1 deletion(-)

diff --git a/Source/EntityFramework.Extended.Test/CodeFirst/EFExtendedCodeFirstTest.cs b/Source/EntityFramework.Extended.Test/CodeFirst/EFExtendedCodeFirstTest.cs
index 2d94650..22f9915 100644
--- a/Source/EntityFramework.Extended.Test/CodeFirst/EFExtendedCodeFirstTest.cs
+++ b/Source/EntityFramework.Extended.Test/CodeFirst/EFExtendedCodeFirstTest.cs
@@ -6,6 +6,7 @@
 using System.ComponentModel.DataAnnotations.Schema;
 using EntityFramework.Audit;
 using System.Data;
+using System.Data.Common;
 using System.Data.Entity.Core;
 
 //using Utile.Money;
@@ -154,8 +155,38 @@ public void EFExtendedCodeFirst_Edit_Properties_contains_ComplexType_class()
 
             var obj = log.Entities[0].Properties;
             Assert.AreEqual(3, obj.Count,"count equals number of properties in Transaction");
+            
+
+        }
 
-            var tran = (IExtendedDataRecord) obj[2].Current;
+        [TestMethod]
+        public void EFExtendedCodeFirst_Edit_Complex_type_property_Original_and_Current_not_the_same()
+        {
+            // Arrange
+            var trx = new Transaction
+            {
+                Money = new Money(123456789012.3456m, "USD"),
+                Detail = "Another Transaction"
+            };
+            _ctx.Transactions.Add(trx);
+            _ctx.SaveChanges();
+
+            //Act
+            var audit = _ctx.BeginAudit();
+            trx.Money.Amount = 10;
+            trx.Detail = "updated detail";
+            var t = _ctx.Set<Transaction>().FirstOrDefault(x => x.Id == trx.Id);
+            _ctx.Entry(t).CurrentValues.SetValues(trx);
+            _ctx.Entry(t).State = EntityState.Modified;
+            _ctx.SaveChanges();
+            var log = audit.LastLog;
+
+            // Assert
+            Assert.AreNotEqual(log.Entities[0].Properties[1].Original, log.Entities[0].Properties[1].Current,"Current and origional of string property Transaction.Detail are not the same.");
+            //cast complex type to DbDataRecord to access its property values.
+            var currentMoney = (DbDataRecord)log.Entities[0].Properties[2].Current;
+            var origionalMoney = (DbDataRecord)log.Entities[0].Properties[2].Original;
+            Assert.AreNotEqual(currentMoney.GetDecimal(1), origionalMoney.GetDecimal(1), "Current and origional of complex type property Transaction.Money.Amount are not the same.");
 
         }
 

From 957d89240f3b4797373f9fe6fff62c959e9b9cff Mon Sep 17 00:00:00 2001
From: Padhraic <pcoogan@gmail.com>
Date: Thu, 19 Nov 2015 19:46:59 +0900
Subject: [PATCH 4/4] Add AuditPropertyFormat attribute implemnetation to
 Transaction money property.

---
 .../CodeFirst/EFExtendedCodeFirstTest.cs      | 29 +++++++++++++++----
 ...EntityFramework.Extended.Test.net45.csproj |  2 +-
 2 files changed, 24 insertions(+), 7 deletions(-)

diff --git a/Source/EntityFramework.Extended.Test/CodeFirst/EFExtendedCodeFirstTest.cs b/Source/EntityFramework.Extended.Test/CodeFirst/EFExtendedCodeFirstTest.cs
index 22f9915..8ccc6a5 100644
--- a/Source/EntityFramework.Extended.Test/CodeFirst/EFExtendedCodeFirstTest.cs
+++ b/Source/EntityFramework.Extended.Test/CodeFirst/EFExtendedCodeFirstTest.cs
@@ -1,4 +1,5 @@
-using System.ComponentModel.DataAnnotations;
+using System;
+using System.ComponentModel.DataAnnotations;
 using System.Data.Entity;
 using System.Linq;
 using EntityFramework.Extensions;
@@ -8,7 +9,6 @@
 using System.Data;
 using System.Data.Common;
 using System.Data.Entity.Core;
-
 //using Utile.Money;
 
 namespace EntityFramework.Test.CodeFirst
@@ -28,17 +28,36 @@ public Money(decimal amount,string currencyCode)
         }
         public string CurrencyCode { get; set; }
         public decimal Amount { get; set; }
-
     }
     public class Transaction
     {
         [Key]
         [DatabaseGenerated(DatabaseGeneratedOption.Identity)]
         public int Id { get; set; }
+        [AuditPropertyFormat(typeof(TransactionFormat), "FormatMoney")]
         public Money Money { get; set; }
         public string Detail { get; set; }
     }
 
+    public class TransactionFormat
+    {
+        public static object FormatMoney(AuditPropertyContext auditProperty)
+         {
+             DbDataRecord d = auditProperty.Value as DbDataRecord;
+             if (d == null)
+                 return null;
+            try
+            {
+                Money m = new Money(d.GetDecimal(1),d.GetString(0));
+                return m;
+            }
+            catch (InvalidCastException)
+            {
+                return null;
+            }
+         }
+    }
+
     public class EFExtendedEntities : DbContext
     {
         public EFExtendedEntities() : base("EFExtendedCodeFirstTest")
@@ -184,9 +203,7 @@ public void EFExtendedCodeFirst_Edit_Complex_type_property_Original_and_Current_
             // Assert
             Assert.AreNotEqual(log.Entities[0].Properties[1].Original, log.Entities[0].Properties[1].Current,"Current and origional of string property Transaction.Detail are not the same.");
             //cast complex type to DbDataRecord to access its property values.
-            var currentMoney = (DbDataRecord)log.Entities[0].Properties[2].Current;
-            var origionalMoney = (DbDataRecord)log.Entities[0].Properties[2].Original;
-            Assert.AreNotEqual(currentMoney.GetDecimal(1), origionalMoney.GetDecimal(1), "Current and origional of complex type property Transaction.Money.Amount are not the same.");
+            Assert.AreNotEqual(log.Entities[0].Properties[2].Current, log.Entities[0].Properties[2].Original, "Current and origional of complex type property Transaction.Money.Amount are not the same.");
 
         }
 
diff --git a/Source/EntityFramework.Extended.Test/EntityFramework.Extended.Test.net45.csproj b/Source/EntityFramework.Extended.Test/EntityFramework.Extended.Test.net45.csproj
index 81368f9..a5335f4 100644
--- a/Source/EntityFramework.Extended.Test/EntityFramework.Extended.Test.net45.csproj
+++ b/Source/EntityFramework.Extended.Test/EntityFramework.Extended.Test.net45.csproj
@@ -75,7 +75,7 @@
     <Reference Include="System.Xml.Linq" />
     <Reference Include="Utile.Money, Version=1.0.1.1, Culture=neutral, processorArchitecture=MSIL">
       <SpecificVersion>False</SpecificVersion>
-      <HintPath>..\..\..\Utile.Money\Utile.Money\bin\Debug\Utile.Money.dll</HintPath>
+      <HintPath>..\..\..\Utile.Money\Utile.Money\bin\Release\Utile.Money.dll</HintPath>
     </Reference>
     <Reference Include="xunit">
       <HintPath>..\packages\xunit.1.9.2\lib\net20\xunit.dll</HintPath>