Merge pull request #94 from mapbox/update-dependencies

update dependencies
mapbox · Nov 10, 2023 · d80dee8 · d80dee8
2 parents 0d06fdb + 20d1386
commit d80dee8
Show file tree

Hide file tree

Showing 14 changed files with 6,328 additions and 2,999 deletions.
diff --git a/.gitignore b/.gitignore
@@ -1,3 +1,4 @@
 .nyc_output
 geojsonhint.js
 node_modules/
+.tap/
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -15,8 +15,9 @@
     "lib"
   ],
   "dependencies": {
+    "@gerhobbelt/nomnom": "github:gerhobbelt/nomnom",
     "concat-stream": "^1.6.1",
-    "jsonlint-lines": "1.7.1",
+    "jsonlint-lines": "^1.7.1",
     "minimist": "^1.2.8",
     "vfile": "^4.0.0",
     "vfile-reporter": "^5.1.1"
@@ -27,7 +28,7 @@
     "eslint": "^5.15.3",
     "fuzzer": "~0.2.0",
     "glob": "^7.1.2",
-    "tap": "^12.6.1"
+    "tap": "^18"
   },
   "directories": {
     "test": "test"
@@ -43,7 +44,10 @@
     "url": "git://github.com/mapbox/geojsonhint.git"
   },
   "scripts": {
-    "test": "eslint . && tap test/*.js --coverage",
+    "test": "eslint . && tap test/*.js --coverage-report=text",
     "prepublish": "browserify . --standalone geojsonhint > geojsonhint.js"
+  },
+  "overrides": {
+    "underscore": "^1.13.0"
   }
 }
diff --git a/test/data/bad/bad-duplicate-properties.cli-output-pretty b/test/data/bad/bad-duplicate-properties.cli-output-pretty
@@ -1,6 +1,6 @@
-  1:1  error  "geometry" member required
-  1:1  error  "properties" member required
-  1:1  error  An object contained duplicate members, making parsing ambigous: type
-  1:1  error  Feature object cannot contain a "features" member
+  1:1  [31merror[39m  "geometry" member required
+  1:1  [31merror[39m  "properties" member required
+  1:1  [31merror[39m  An object contained duplicate members, making parsing ambigous: type
+  1:1  [31merror[39m  Feature object cannot contain a "features" member
 
-✖ 4 errors
+[31m✖[39m 4 errors
diff --git a/test/data/bad/bad-featurelist.cli-output-pretty b/test/data/bad/bad-featurelist.cli-output-pretty
@@ -1,3 +1,3 @@
-  3:1  error  a number was found where a coordinate array should have been found: this needs to be nested more deeply
+  3:1  [31merror[39m  a number was found where a coordinate array should have been found: this needs to be nested more deeply
 
-✖ 1 error
+[31m✖[39m 1 error
diff --git a/test/data/bad/bad-json.cli-output-pretty b/test/data/bad/bad-json.cli-output-pretty
@@ -1,6 +1,6 @@
-  1:1  error  Parse error on line 2:
+  1:1  [31merror[39m  Parse error on line 2:
 ...": "MultiPoint"    "coordinates": [["fo
 ----------------------^
 Expecting 'EOF', '}', ':', ',', ']', got 'STRING'
 
-✖ 1 error
+[31m✖[39m 1 error
diff --git a/test/data/bad/bad-object-type.cli-output-pretty b/test/data/bad/bad-object-type.cli-output-pretty
@@ -1,3 +1,3 @@
-  1:1  error  "type" member should be string, but is an object instead
+  1:1  [31merror[39m  "type" member should be string, but is an object instead
 
-✖ 1 error
+[31m✖[39m 1 error
diff --git a/test/data/bad/bad-polygon-interiorring-rhr.cli-output-pretty b/test/data/bad/bad-polygon-interiorring-rhr.cli-output-pretty
@@ -1,3 +1,3 @@
-  1:1  warning  Polygons and MultiPolygons should follow the right-hand rule
+  1:1  [33mwarning[39m  Polygons and MultiPolygons should follow the right-hand rule
 
-⚠ 1 warning
+[33m⚠[39m 1 warning
diff --git a/test/data/bad/bad-polygonloop.cli-output-pretty b/test/data/bad/bad-polygonloop.cli-output-pretty
diff --git a/test/data/bad/bad-properties-as-array.cli-output-pretty b/test/data/bad/bad-properties-as-array.cli-output-pretty
@@ -1,3 +1,3 @@
-  1:1  error  "properties" member should be object, but is an Array instead
+  1:1  [31merror[39m  "properties" member should be object, but is an Array instead
 
-✖ 1 error
+[31m✖[39m 1 error
diff --git a/test/data/bad/badcoord.cli-output-pretty b/test/data/bad/badcoord.cli-output-pretty
@@ -1,3 +1,3 @@
-  3:1  error  each element in a position must be a number
+  3:1  [31merror[39m  each element in a position must be a number
 
-✖ 1 error
+[31m✖[39m 1 error
diff --git a/test/data/bad/badcoordtype.cli-output-pretty b/test/data/bad/badcoordtype.cli-output-pretty
@@ -1,3 +1,3 @@
-  9:1  error  a number was found where a coordinate array should have been found: this needs to be nested more deeply
+  9:1  [31merror[39m  a number was found where a coordinate array should have been found: this needs to be nested more deeply
 
-✖ 1 error
+[31m✖[39m 1 error
diff --git a/test/data/bad/badfeature.cli-output-pretty b/test/data/bad/badfeature.cli-output-pretty
@@ -1,3 +1,3 @@
-  1:1  error  Every feature must be an object
+  1:1  [31merror[39m  Every feature must be an object
 
-✖ 1 error
+[31m✖[39m 1 error
diff --git a/test/hint.test.js b/test/hint.test.js
@@ -17,13 +17,13 @@ function filejs(x) {
 test('geojsonhint', function(t) {
     glob.sync('test/data/good/*.geojson').forEach(function(f) {
         var gj = file(f);
-        t.deepEqual(geojsonhint.hint(gj), [], f);
+        t.same(geojsonhint.hint(gj), [], f);
     });
-    t.deepEqual(geojsonhint.hint(undefined), [{
+    t.same(geojsonhint.hint(undefined), [{
         message: 'Expected string or object as input',
         line: 0
     }], 'expected string input');
-    t.deepEqual(geojsonhint.hint('{}'), [{
+    t.same(geojsonhint.hint('{}'), [{
         message: '"type" member required',
         line: 1
     }], 'just an object');
@@ -33,7 +33,7 @@ test('geojsonhint', function(t) {
             if (process.env.UPDATE) {
                 fs.writeFileSync(f.replace('geojson', 'result'), JSON.stringify(geojsonhint.hint(gj), null, 2));
             }
-            t.deepEqual(geojsonhint.hint(gj), filejs(f.replace('geojson', 'result')), f);
+            t.same(geojsonhint.hint(gj), filejs(f.replace('geojson', 'result')), f);
         });
         t.end();
     });
@@ -94,18 +94,18 @@ test('geojsonhint', function(t) {
             if (process.env.UPDATE) {
                 fs.writeFileSync(f.replace('geojson', 'result-object'), JSON.stringify(geojsonhint.hint(gj), null, 2));
             }
-            t.deepEqual(geojsonhint.hint(gj), filejs(f.replace('geojson', 'result-object')), f);
+            t.same(geojsonhint.hint(gj), filejs(f.replace('geojson', 'result-object')), f);
         });
         t.end();
     });
     test('noDuplicateMembers option=false', function(t) {
-        t.deepEqual(geojsonhint.hint('{"type":"invalid","type":"Feature","properties":{},"geometry":null}', {
+        t.same(geojsonhint.hint('{"type":"invalid","type":"Feature","properties":{},"geometry":null}', {
             noDuplicateMembers: false
         }), [], 'sketchy object permitted');
         t.end();
     });
     test('noDuplicateMembers option=true', function(t) {
-        t.deepEqual(geojsonhint.hint('{"type":"invalid","type":"Feature","properties":{},"geometry":null}', {
+        t.same(geojsonhint.hint('{"type":"invalid","type":"Feature","properties":{},"geometry":null}', {
             noDuplicateMembers: true
         }), [{
           line: 1,
@@ -114,7 +114,7 @@ test('geojsonhint', function(t) {
         t.end();
     });
     test('noDuplicateMembers option=true', function(t) {
-        t.deepEqual(geojsonhint.hint('{ "type": "Point", "coordinates": [100.0000000001, 5.0000000001] }', {
+        t.same(geojsonhint.hint('{ "type": "Point", "coordinates": [100.0000000001, 5.0000000001] }', {
             precisionWarning: true
         }), [{
           line: 1,
@@ -148,7 +148,7 @@ test('geojsonhint', function(t) {
             precisionWarning: true
         })
         t.equal(truncated.length, 11);
-        t.deepEqual(truncated[10], {
+        t.same(truncated[10], {
           line: 63,
           level: 'message',
           message: 'truncated warnings: we\'ve encountered coordinate precision warning 10 times, '
@@ -190,15 +190,15 @@ test('geojsonhint', function(t) {
         t.end();
     });
     test('invalid roots', function(t) {
-        t.deepEqual(geojsonhint.hint('null'), [{
+        t.same(geojsonhint.hint('null'), [{
             message: 'The root of a GeoJSON object must be an object.',
             line: 0
         }], 'non-object root');
-        t.deepEqual(geojsonhint.hint('1'), [{
+        t.same(geojsonhint.hint('1'), [{
             message: 'The root of a GeoJSON object must be an object.',
             line: 0
         }], 'number root');
-        t.deepEqual(geojsonhint.hint('"string"'), [{
+        t.same(geojsonhint.hint('"string"'), [{
             message: 'The root of a GeoJSON object must be an object.',
             line: 0
         }], 'string root');