Patched unit test to account for validation bug

In #2bf635e, a bug was missed by the `Map_ComplexObject_ReturnsFlattenedTopic` unit test since the `Contact` content type had an `Email` attribute. As such, when the validator was supposed to be validating e.g. `AlternateEmail`, it was _actually_ validating `Email`, and passing. In a real-world scenario, however, this won't usually be the case, and defeats the benefit of validating the model against the content type.

To account for this, the generic `Email` attribute was removed, and a new `Name` attribute was introduced. This clutters up the unit test and the model a bit by requiring two separate objects (`ContactTopicBindingModel` and `EmailTopicBindingModel`) but this effectively tests the same conditions while additionally validating that the `AttributePrefix` context is being properly accounted for as part of that validation.

Author: JeremyCaney

OnTopic.TestDoubles/StubTopicRepository.cs

@@ -222,7 +222,7 @@ private Topic CreateFakeData() {
 
       var contactContentType = TopicFactory.Create("Contact", "ContentTypeDescriptor", contentTypes);
 
-      addAttribute(contactContentType, "Email");
+      addAttribute(contactContentType, "Name");
       addAttribute(contactContentType, "AlternateEmail");
       addAttribute(contactContentType, "BillingContactEmail");
 

OnTopic.Tests/BindingModels/ContactTopicBindingModel.cs

@@ -18,7 +18,7 @@ namespace OnTopic.Tests.BindingModels {
   /// </remarks>
   public class ContactTopicBindingModel {
 
-    public string? Email { get; set; }
+    public string? Name { get; set; }
 
   } //Class
 } //Namespace

OnTopic.Tests/BindingModels/EmailTopicBindingModel.cs

@@ -0,0 +1,24 @@
+/*==============================================================================================================================
+| Author        Ignia, LLC
+| Client        Ignia, LLC
+| Project       Topics Library
+\=============================================================================================================================*/
+
+namespace OnTopic.Tests.BindingModels {
+
+  /*============================================================================================================================
+  | BINDING MODEL: EMAIL
+  \---------------------------------------------------------------------------------------------------------------------------*/
+  /// <summary>
+  ///   Provides a minimal implementation of a custom topic binding model with a single property intended to be called from
+  ///   <see cref="MapToParentTopicBindingModel"/>.
+  /// </summary>
+  /// <remarks>
+  ///   This is a sample class intended for test purposes only; it is not designed for use in a production environment.
+  /// </remarks>
+  public class EmailTopicBindingModel {
+
+    public string? Email { get; set; }
+
+  } //Class
+} //Namespace

OnTopic.Tests/BindingModels/MapToParentTopicBindingModel.cs

@@ -22,10 +22,10 @@ public class MapToParentTopicBindingModel : BasicTopicBindingModel {
     public ContactTopicBindingModel PrimaryContact { get; set; } = new ContactTopicBindingModel();
 
     [MapToParent(AttributePrefix="Alternate")]
-    public ContactTopicBindingModel AlternateContact { get; set; } = new ContactTopicBindingModel();
+    public EmailTopicBindingModel AlternateContact { get; set; } = new EmailTopicBindingModel();
 
     [MapToParent]
-    public ContactTopicBindingModel BillingContact { get; set; } = new ContactTopicBindingModel();
+    public EmailTopicBindingModel BillingContact { get; set; } = new EmailTopicBindingModel();
 
   } //Class
 } //Namespace

OnTopic.Tests/ReverseTopicMappingServiceTest.cs

@@ -139,7 +139,7 @@ public async Task Map_Existing_ReturnsUpdatedTopic() {
       target.Title              = "Original Attribute";
       target.DefaultValue       = "Hello";
       target.IsRequired         = true;
-      target.IsExtendedAttribute        = false;
+      target.IsExtendedAttribute= false;
       target.Description        = "Original Description";
 
       target                    = (TextAttribute?)await mappingService.MapAsync(bindingModel, target).ConfigureAwait(false);
@@ -170,14 +170,14 @@ public async Task Map_ComplexObject_ReturnsFlattenedTopic() {
         ContentType             = "Contact"
       };
 
-      bindingModel.PrimaryContact.Email = "PrimaryContact@Ignia.com";
-      bindingModel.AlternateContact.Email = "AlternateContact@Ignia.com";
-      bindingModel.BillingContact.Email = "BillingContact@Ignia.com";
-
+      bindingModel.PrimaryContact.Name                          = "Jeremy";
+      bindingModel.AlternateContact.Email                       = "AlternateContact@Ignia.com";
+      bindingModel.BillingContact.Email                         = "BillingContact@Ignia.com";
+ 
       var target                = (Topic?)await mappingService.MapAsync(bindingModel).ConfigureAwait(false);
 
       Assert.IsNotNull(target);
-      Assert.AreEqual<string>("PrimaryContact@Ignia.com", target.Attributes.GetValue("Email"));
+      Assert.AreEqual<string>("Jeremy", target.Attributes.GetValue("Name"));
       Assert.AreEqual<string>("AlternateContact@Ignia.com", target.Attributes.GetValue("AlternateEmail"));
       Assert.AreEqual<string>("BillingContact@Ignia.com", target.Attributes.GetValue("BillingContactEmail"));