More unit tests and fix

Author: shakilsiraj

src/main/DeserializationHelper.ts

@@ -174,9 +174,6 @@ export const DeserializeComplexType = (
     if (metadata === undefined) {
       metadata = { name: key, required: false, access: AccessType.BOTH };
     }
-    if (config.ignoreNameMetadata === true) {
-      metadata.name = undefined;
-    }
     // tslint:disable-next-line:triple-equals
     if (AccessType.WRITE_ONLY != metadata.access) {
       /**
@@ -193,7 +190,8 @@ export const DeserializeComplexType = (
         );
       }
       // tslint:disable-next-line:triple-equals
-      const jsonKeyName = metadata.name != undefined ? metadata.name : key;
+      const jsonKeyName =
+        config.ignoreNameMetadata === true ? key : metadata.name || key;
       // tslint:disable-next-line:triple-equals
       if (json[jsonKeyName] != undefined) {
         /**

src/test/index.spec.ts

@@ -722,22 +722,73 @@ describe("Testing JsonIgnore decorator", () => {
     expect(testInstance.state).toBe("old");
   });
 
-  it("Testing DeserializationConfig", () => {
-    class Event {
+  describe("DeserializationConfig", () => {
+    it("{ ignoreNameMetadata: true } should use property name", () => {
+      class Event {
+        @JsonProperty()
+        id: number = undefined;
+        @JsonProperty({ name: "geo-location" })
+        location = "old";
+      }
+
+      const json = {
+        id: "1",
+        location: "Canberra",
+      };
+
+      const testInstance: Event = ObjectMapper.deserialize(Event, json, {
+        ignoreNameMetadata: true,
+      });
+      expect(testInstance.location).toBe("Canberra");
+    });
+
+    it("{ ignoreNameMetadata: true } should use name metadata during serialization", () => {
+      class Event {
+        @JsonProperty()
+        id: number = undefined;
+        @JsonProperty({ name: "geo-location" })
+        location = "old";
+      }
+
+      const json = {
+        id: "1",
+        location: "Canberra",
+      };
+
+      const testInstance: Event = ObjectMapper.deserialize(Event, json, {
+        ignoreNameMetadata: true,
+      });
+
+      const serialized = ObjectMapper.serialize(testInstance);
+      expect(serialized).toBe('{"id":"1","geo-location":"Canberra"}');
+    });
+  });
+
+  it("{ ignoreNameMetadata: true } should work with array serialization", () => {
+    class DeserializeComplexTypeArrayTest {
       @JsonProperty()
-      id: number = undefined;
-      @JsonProperty({ name: "geo-location" })
-      location = "old";
+      storeName: string = undefined;
+
+      @JsonProperty({ name: "AVAILABLE_AT" })
+      availableAt: String[] = undefined;
     }
 
     const json = {
-      id: "1",
-      location: "Canberra",
+      storeName: "PizzaHut",
+      availableAt: ["2000", "3000", "4000", "5000"],
     };
 
-    const testInstance: Event = ObjectMapper.deserialize(Event, json, {
-      ignoreNameMetadata: true,
-    });
-    expect(testInstance.location).toBe("Canberra");
+    const testInstance = ObjectMapper.deserialize(
+      DeserializeComplexTypeArrayTest,
+      json,
+      {
+        ignoreNameMetadata: true,
+      }
+    );
+
+    const serialized = ObjectMapper.serialize(testInstance);
+    expect(serialized).toBe(
+      '{"storeName":"PizzaHut","AVAILABLE_AT":["2000","3000","4000","5000"]}'
+    );
   });
 });