Require Node.js 10 (#97)

imagemin · May 29, 2020 · 49f230f · 49f230f
1 parent 2b60e63
commit 49f230f
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 24 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -1,9 +1,10 @@
 sudo: false
 language: node_js
 node_js:
+  - '14'
+  - '12'
   - '10'
-  - '8'
-  - '6'
 addons:
   apt:
-    packages: nasm
+    packages:
+      - nasm
diff --git a/lib/install.js b/lib/install.js
@@ -15,7 +15,7 @@ const args = [
 
 bin.run(args).then(() => {
 	log.success('jpegtran pre-build test passed successfully');
-}).catch(error => {
+}).catch(async error => {
 	log.warn(error.message);
 	log.warn('jpegtran pre-build test failed');
 	log.info('compiling from source');
@@ -25,13 +25,15 @@ bin.run(args).then(() => {
 		`--prefix="${bin.dest()}" --bindir="${bin.dest()}"`
 	].join(' ');
 
-	binBuild.file(path.resolve(__dirname, '../vendor/source/libjpeg-turbo-1.5.1.tar.gz'), [
-		'touch configure.ac aclocal.m4 configure Makefile.am Makefile.in',
-		cfg,
-		'make install'
-	]).then(() => {
+	try {
+		await binBuild.file(path.resolve(__dirname, '../vendor/source/libjpeg-turbo-1.5.1.tar.gz'), [
+			'touch configure.ac aclocal.m4 configure Makefile.am Makefile.in',
+			cfg,
+			'make install'
+		]);
+
 		log.success('jpegtran built successfully');
-	}).catch(error => {
+	} catch (error) {
 		log.error(error.stack);
-	});
+	}
 });
diff --git a/package.json b/package.json
@@ -24,11 +24,11 @@
     "jpegtran": "cli.js"
   },
   "engines": {
-    "node": ">=6"
+    "node": ">=10"
   },
   "scripts": {
     "postinstall": "node lib/install.js",
-    "test": "xo && ava"
+    "test": "xo && ava --timeout=120s"
   },
   "files": [
     "index.js",
@@ -53,11 +53,11 @@
     "logalot": "^2.0.0"
   },
   "devDependencies": {
-    "ava": "*",
+    "ava": "^3.8.0",
     "bin-check": "^4.0.1",
     "compare-size": "^3.0.0",
-    "execa": "^1.0.0",
-    "tempy": "^0.2.1",
-    "xo": "*"
+    "execa": "^4.0.0",
+    "tempy": "^0.5.0",
+    "xo": "^0.30.0"
   }
 }
diff --git a/test/test.js b/test/test.js
@@ -10,36 +10,36 @@ const compareSize = require('compare-size');
 const jpegtran = require('..');
 
 test('rebuild the jpegtran binaries', async t => {
-	const tmp = tempy.directory();
+	const temporary = tempy.directory();
 	const cfg = [
 		'./configure --disable-shared',
-		`--prefix="${tmp}" --bindir="${tmp}"`
+		`--prefix="${temporary}" --bindir="${temporary}"`
 	].join(' ');
 
 	await binBuild.url('https://downloads.sourceforge.net/project/libjpeg-turbo/1.5.1/libjpeg-turbo-1.5.1.tar.gz', [
 		cfg,
 		'make install'
 	]);
 
-	t.true(fs.existsSync(path.join(tmp, 'jpegtran')));
+	t.true(fs.existsSync(path.join(temporary, 'jpegtran')));
 });
 
 test('return path to binary and verify that it is working', async t => {
 	t.true(await binCheck(jpegtran, ['-version']));
 });
 
 test('minify a JPG', async t => {
-	const tmp = tempy.directory();
+	const temporary = tempy.directory();
 	const src = path.join(__dirname, 'fixtures/test.jpg');
-	const dest = path.join(tmp, 'test.jpg');
+	const dest = path.join(temporary, 'test.jpg');
 	const args = [
 		'-outfile',
 		dest,
 		src
 	];
 
 	await execa(jpegtran, args);
-	const res = await compareSize(src, dest);
+	const result = await compareSize(src, dest);
 
-	t.true(res[dest] < res[src]);
+	t.true(result[dest] < result[src]);
 });