Use stat() instead of access(). Remove support for node<=0.12

Author: imyller

.travis.yml

@@ -1,13 +1,8 @@
 language: node_js 
 node_js:
-  - "0.8"
-  - "0.10"
   - "0.12"
   - "iojs-1"
   - "iojs-2"
-os:
-  - linux
-  - osx
 notifications:
   email:
     - [email protected]

README.md

@@ -12,7 +12,7 @@ Detects if `resolv.conf` is created, deleted, appended, modified or renamed.
 Node Compatibility
 ---------------------
 
-Supports Node.js v0.8.0+ and io.js v1.0.0+
+Supports Node.js v0.12.0+ and io.js v1.0.0+
 
 Please let me know if you have problems running it on a later version of Node or
 have platform-specific problems.

lib/resolvmon.js

@@ -14,13 +14,13 @@ function watchFile(filepath, callback) {
 		fdir = path.dirname(fpath),
 		fname = path.basename(fpath);
 
-	fs.accessSync(fdir, fs.F_OK);
+	fs.statSync(fdir);
 
 	return fs.watch(fdir, {persistent: false, recursive: false}, function (event, changed_fname) {
 
 		if (changed_fname === fname) {
 
-			fs.access(fpath, fs.F_OK, function (err) {
+			fs.stat(fpath, function (err, stats) {
 				callback && callback(null, !err, fpath);
 			});
 
@@ -109,7 +109,7 @@ ResolvMon.prototype.update = function (callback) {
 		fdir = path.dirname(fpath);
 
 	try {
-		fs.accessSync(fdir, fs.F_OK);
+		fs.statSync(fdir);
 	} catch (ex) {
 		self._emit('error', ex);
 		if (callback) callback(ex, []); else throw ex;

package.json

@@ -20,13 +20,13 @@
   ],
   "main": "lib/resolvmon.js",
   "engines": {
-    "node": ">= 0.8"
+    "node": ">= 0.12"
   },
   "dependencies": {},
   "devDependencies": {
-    "mocha": "^2.2.4",
-    "chai": "^2.3.0",
-    "temp": "^0.8.1"
+    "mocha": "~2.2.4",
+    "chai": "~2.3.0",
+    "temp": "~0.8.1"
   },
   "scripts": {
     "test": "./node_modules/mocha/bin/mocha test"

test/06_watch.js

@@ -80,27 +80,36 @@ describe('resolvmon watch', function () {
 	it('detect resolv.conf delete', function (done) {
 
 		resolvmon.setPath(temporary_conf);
-		fs.writeFileSync(temporary_conf, "nameserver 127.0.0.11\nnameserver 127.0.0.22\n");
 
-		resolvmon.on('error', function (err) {
+		fs.writeFile(temporary_conf, "nameserver 127.0.0.11\nnameserver 127.0.0.22\n", function (err) {
+
 			expect(err).not.to.be.defined;
-			done(err);
-		});
 
-		resolvmon.on('update', function (nameservers) {
+			if (err) {
+				return done();
+			}
 
-			expect(nameservers).to.be.defined;
-			expect(nameservers).to.be.instanceof(Array);
-			expect(nameservers).to.eql([]);
+			resolvmon.on('error', function (err) {
+				expect(err).not.to.be.defined;
+				done(err);
+			});
 
-			done();
+			resolvmon.on('update', function (nameservers) {
 
-		});
+				expect(nameservers).to.be.defined;
+				expect(nameservers).to.be.instanceof(Array);
+				expect(nameservers).to.eql([]);
 
-		resolvmon.start();
+				done();
+
+			});
+
+			resolvmon.start();
+
+			process.nextTick(function () {
+				fs.unlinkSync(temporary_conf);
+			});
 
-		process.nextTick(function () {
-			fs.unlink(temporary_conf);
 		});
 
 	});
@@ -109,55 +118,71 @@ describe('resolvmon watch', function () {
 
 		resolvmon.setPath(temporary_conf);
 
-		fs.writeFileSync(temporary_conf, "nameserver 127.0.0.12\nnameserver 127.0.0.23");
+		fs.writeFile(temporary_conf, "nameserver 127.0.0.12\nnameserver 127.0.0.23", function (err) {
 
-		resolvmon.on('error', function (err) {
 			expect(err).not.to.be.defined;
-			done(err);
-		});
 
-		resolvmon.on('update', function (nameservers) {
+			if (err) {
+				return done();
+			}
 
-			expect(nameservers).to.be.defined;
-			expect(nameservers).to.be.instanceof(Array);
-			expect(nameservers).to.eql(['127.0.0.12', '127.0.0.23', '127.0.0.34', '127.0.0.45']);
+			resolvmon.on('error', function (err) {
+				expect(err).not.to.be.defined;
+				done(err);
+			});
 
-			done();
+			resolvmon.on('update', function (nameservers) {
 
-		});
+				expect(nameservers).to.be.defined;
+				expect(nameservers).to.be.instanceof(Array);
+				expect(nameservers).to.eql(['127.0.0.12', '127.0.0.23', '127.0.0.34', '127.0.0.45']);
 
-		resolvmon.start();
+				done();
 
-		process.nextTick(function () {
-			fs.appendFileSync(temporary_conf, "\nnameserver 127.0.0.34\nnameserver 127.0.0.45\n");
-		});
+			});
+
+			resolvmon.start();
 
+			process.nextTick(function () {
+				fs.appendFileSync(temporary_conf, "\nnameserver 127.0.0.34\nnameserver 127.0.0.45\n");
+			});
+
+
+		});
 
 	});
 
 	it('detect resolv.conf modification', function (done) {
 
 		resolvmon.setPath(temporary_conf);
 
-		fs.writeFileSync(temporary_conf, "nameserver 127.0.0.77\nnameserver 127.0.0.88\n");
+		fs.writeFile(temporary_conf, "nameserver 127.0.0.77\nnameserver 127.0.0.88\n", function (err) {
 
-		resolvmon.on('error', function (err) {
 			expect(err).not.to.be.defined;
-			done(err);
-		});
 
-		resolvmon.on('update', function (nameservers) {
+			if (err) {
+				return done();
+			}
 
-			expect(nameservers).to.be.defined;
-			expect(nameservers).to.be.instanceof(Array);
-			expect(nameservers).to.eql(['127.0.0.100', '127.0.0.200']);
-			done();
-		});
+			resolvmon.on('error', function (err) {
+				expect(err).not.to.be.defined;
+				done(err);
+			});
 
-		resolvmon.start();
+			resolvmon.on('update', function (nameservers) {
+
+				expect(nameservers).to.be.defined;
+				expect(nameservers).to.be.instanceof(Array);
+				expect(nameservers).to.eql(['127.0.0.100', '127.0.0.200']);
+				done();
+			});
+
+			resolvmon.start();
+
+			process.nextTick(function () {
+				fs.writeFileSync(temporary_conf, "nameserver 127.0.0.100\nnameserver 127.0.0.200");
+			});
 
-		process.nextTick(function () {
-			fs.writeFileSync(temporary_conf, "nameserver 127.0.0.100\nnameserver 127.0.0.200");
 		});
 
 	});
@@ -190,7 +215,7 @@ describe('resolvmon watch', function () {
 		after(function () {
 
 			try {
-				fs.unlink(temporary_conf + '.renamed');
+				fs.unlinkSync(temporary_conf + '.renamed');
 			} catch (ex) {
 			}
 
@@ -226,7 +251,7 @@ describe('resolvmon watch', function () {
 		after(function () {
 
 			try {
-				fs.unlink(temporary_conf + '.renamed');
+				fs.unlinkSync(temporary_conf + '.renamed');
 			} catch (ex) {
 			}