Fixed unnecessary warning bug, removed package.json cruft, v1.0.5

Author: cdibbs

package.json

@@ -1,7 +1,7 @@
 {
   "name": "simple-mapper",
   "description": "Angular 2+ object-to-view-model mapper. Intended to facilitate working with JSON from web API endpoints.",
-  "version": "1.0.4",
+  "version": "1.0.5",
   "license": "MIT",
   "repository": "https://github.com/cdibbs/simple-mapper",
   "scripts": {
@@ -12,8 +12,7 @@
     "lint": "ng lint",
     "e2e": "ng e2e",
     "coverage": "ng test --code-coverage --single-run",
-    "postcoverage": "codecov -f coverage/*.json",
-    "coveralls": "cat ./coverage/lcov.info | node node_modules/.bin/coveralls"
+    "postcoverage": "codecov -f coverage/*.json"
   },
   "private": false,
   "dependencies": {

src/app/services/mapper.service.spec.ts

@@ -76,6 +76,20 @@ describe('MapperService', () => {
         expect(result["Two"]).toBeUndefined();
         expect(warned).toBeTruthy();
     }));
+    it('should not warn when not extraneous source properties.',
+    inject([MapperService], (mapper: MapperService) => {
+        var warned = null;
+        mapper["log"].warn = function(yep) { warned = yep; };
+        var json = {
+            Id: 3,
+            One: "something else",
+            Two: 6.53, // ignored
+        };
+        var result = mapper.MapJsonToVM(vm.Mine, json);
+        expect(result.Id).toBe(3);
+        expect(result.One).toBe(json.One);
+        expect(warned).toBeFalsy();
+    }));
     it('should map nested types.', inject([MapperService], (mapper: MapperService) => {
         var json = {
             Id: 3,

src/app/services/mapper.service.ts

@@ -32,8 +32,11 @@ export class MapperService implements IMapperService {
         let tprops = getMappableProperties<T>(vm);
         let keys = Object.keys(json || {});
 
-        if (unmappedWarning && keys.length > Object.keys(tprops).length) {
-            this.log.warn(`Unmapped source properties: ` + keys.filter(k => Object.keys(tprops).indexOf(k) < 0).join(", "));
+        if (unmappedWarning) {
+            let t2props = Object.keys(vm);
+            let unmapped = keys.filter(k => Object.keys(tprops).indexOf(k) < 0 && t2props.indexOf(k) < 0);
+            if (unmapped.length)
+                this.log.warn(`Unmapped source properties: ` + unmapped.join(", "));
         }
 
         for(var prop of keys) {