lint & run the linters as part of the unit testing

mmomtchev · Jun 21, 2024 · b19755f · b19755f
1 parent da71159
commit b19755f
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/package.json b/package.json
@@ -53,7 +53,7 @@
     }
   ],
   "scripts": {
-    "test": "mocha && npm run test:stress 20",
+    "test": "npm run lint && mocha && npm run test:stress 20",
     "bench": "node bench/streams.js",
     "lint:cpp": "clang-format -i src/*.cpp src/*.hpp && clang-format -i src/*/*.cpp src/*/*.hpp",
     "lint:js": "eslint lib test examples",
@@ -113,7 +113,7 @@
     "documentation-hipster": "^1.0.0",
     "documentation-polyglot": "^1.0.1",
     "eslint": "^8.15.0",
-    "eslint-plugin-array-func": "^5.0.1",
+    "eslint-plugin-array-func": "^4.0.0",
     "eslint-plugin-mocha": "^10.1.0",
     "eslint-plugin-prefer-arrow": "^1.1.3",
     "gh-pages": "^6.1.1",

diff --git a/test/api_layer.test.ts b/test/api_layer.test.ts
@@ -280,7 +280,7 @@ describe('gdal.Layer', () => {
           assert.closeTo(actual_envelope.maxY, expected_envelope.maxY, 0.00001)
         })
       })
-      it("should throw error if force flag is false and layer doesn't have extent already computed", function() {
+      it("should throw error if force flag is false and layer doesn't have extent already computed", () => {
         // No longer true in GDAL 3.9
         if (semver.lt(gdal.version, '3.9.0')) {
           const dataset = gdal.open(`${__dirname}/data/park.geo.json`)

diff --git a/test/api_rasterband.test.ts b/test/api_rasterband.test.ts
@@ -654,7 +654,7 @@ describe('gdal.RasterBand', () => {
                 ds.rasterSize.x / 2, ds.rasterSize.y / 2, data)
             }, /Array length must be greater than/)
           })
-          it('w/file over the 4G elements limit', function() {
+          it('w/file over the 4G elements limit', function () {
             if (semver.gte(process.versions.node, '22.0.0')) {
               // It seems that Node.js 22 (V8?) has removed the 32-bit index restrictions
               // on TypedArrays

diff --git a/test/api_wrapVRT.test.ts b/test/api_wrapVRT.test.ts
@@ -100,7 +100,7 @@ ${Object.keys(metadata).map((k) => `    <MDI key="${k}">${metadata[k]}</MDI>`).j
 
     const bands = ds.bands.map((b) => ({ sources: [ b ] }))
 
-    const actual = gdal.wrapVRT({bands})
+    const actual = gdal.wrapVRT({ bands })
 
     assert.strictEqual(actual, expected)
   })