feat(dedupe): squash empire and country

Author: missinglink

helper/diffPlaces.js

@@ -95,6 +95,14 @@ function isParentHierarchyDifferent(item1, item2){
     }
   }
 
+  // special case to consider empires and country as the same for deduplication purposes
+  if (
+    (item1.layer === 'empire' && item2.layer === 'country') ||
+    (item1.layer === 'country' && item2.layer === 'empire')
+  ) {
+    return false;
+  }
+
   // special handling of postal codes, which we consider to be strictly
   // unique within a single country/dependency regardless of the rest of
   // the hierarchy (ie. we ignore other parent properties)

test/unit/middleware/dedupe.js

@@ -821,6 +821,90 @@ module.exports.tests.priority = function(test, common) {
     });
   });
 
+  test('real-world test New Zealand: empire vs country', function (t) {
+    var req = {
+      clean: {
+        text: 'New Zealand',
+        size: 100
+      }
+    };
+    var res = {
+      data:  [
+        {
+          'name': {
+            'default': 'New Zealand'
+          },
+          'source': 'whosonfirst',
+          'source_id': '136253053',
+          'layer': 'empire',
+          'parent': {
+            'empire_id': 136253053
+          },
+        },
+        {
+          'name': {
+            'default': [ 'New Zealand', 'Aotearoa' ],
+          },
+          'source': 'whosonfirst',
+          'source_id': '85633345',
+          'layer': 'country',
+          'parent': {
+            'continent_id': 102191583,
+            'country_id': 85633345
+          },
+        }
+      ]
+    };
+
+    dedupe(req, res, function () {
+      t.equal(res.data.length, 1, 'results have fewer items than before');
+      t.equal(res.data[0].layer, 'country', 'empire result removed');
+      t.end();
+    });
+  });
+
+  test('real-world test New Zealand: empire vs country - inverted order', function (t) {
+    var req = {
+      clean: {
+        text: 'New Zealand',
+        size: 100
+      }
+    };
+    var res = {
+      data:  [
+        {
+          'name': {
+            'default': [ 'New Zealand', 'Aotearoa' ],
+          },
+          'source': 'whosonfirst',
+          'source_id': '85633345',
+          'layer': 'country',
+          'parent': {
+            'continent_id': 102191583,
+            'country_id': 85633345
+          },
+        },
+        {
+          'name': {
+            'default': 'New Zealand'
+          },
+          'source': 'whosonfirst',
+          'source_id': '136253053',
+          'layer': 'empire',
+          'parent': {
+            'empire_id': 136253053
+          },
+        }
+      ]
+    };
+
+    dedupe(req, res, function () {
+      t.equal(res.data.length, 1, 'results have fewer items than before');
+      t.equal(res.data[0].layer, 'country', 'empire result removed');
+      t.end();
+    });
+  });
+
   test('A->B B->C dependency graph', function (t) {
     var req = {
       clean: {