Replace usage of Map for result with Object

It is impractical to iterate over a Map, so let's use an Object for now.

Author: bartveneman

README.md

@@ -14,9 +14,9 @@ const [firstStats, secondStats] = await Promise.all([
 );
 const changes = differ(firstStats, secondStats);
 
-//=> Returns a Map with all changes
-// Map {
-// 'values.colors.unique' => [
+//=> Returns an object with all changes
+// {
+// 'values.colors.unique': [
 // {
 // value: 'brown',
 // removed: true,

diff.js

@@ -4,8 +4,9 @@ const diffLists = require('./src/diff-lists.js')
 function diffStats(statsA, statsB) {
 // This assumes that the statsB is newer than statsA
 return Object.entries(statsB).reduce((diff, [key, value]) => {
-return diff.set(key, diffStat(statsA[key], value))
-}, new Map())
+diff[key] = diffStat(statsA[key], value)
+return diff
+}, {})
 }
 
 function diffStat(statA, statB) {

test/diff.js

@@ -22,15 +22,15 @@ test('An error is thrown if incompatible stat types are given', t => {
 // of the structure is done in other tests
 test('It should return the correct structure', t => {
 const result = diff(defaultFixture, fixtureWithUpdatedValues)
-t.true(result instanceof Map)
-t.deepEqual(['a', 'b'], [...result.keys()])
+t.is('object', typeof result)
+t.deepEqual(['a', 'b'], Object.keys(result))
 
-result.forEach(stat => {
+Object.values(result).forEach(stat => {
 t.is('boolean', typeof stat.changed)
 })
 })
 
 test('It should return the correct structure when new props are added', t => {
 const result = diff(defaultFixture, fixtureWithAddedProp)
-t.deepEqual(['a', 'b', 'c'], [...result.keys()])
+t.deepEqual(['a', 'b', 'c'], Object.keys(result))
 })