Support node 10

Author: sam-lord

lib/fileRotator.js

@@ -47,7 +47,8 @@ class FileRotator extends EventEmitter {
 
     async rotate() {
         if (this._currentHandle) {
-            await this._currentHandle.close();
+            await this._closeFileHandle();
+            this._currentHandle = null;
         }
         if (this._gzip) {
             await this._gzipCurrentFile();
@@ -61,7 +62,8 @@ class FileRotator extends EventEmitter {
 
     async shutdown() {
         if (this._currentHandle) {
-            await this._currentHandle.close();
+            await this._closeFileHandle();
+            this._currentHandle = null;
         }
     }
 
@@ -163,6 +165,14 @@ class FileRotator extends EventEmitter {
             fileNumber -= 1;
         }
     }
+
+    async _closeFileHandle() {
+        if (this._currentHandle) {
+            const closePromise = this._currentHandle.close();
+            this._currentHandle = null;
+            await closePromise;
+        }
+    }
 }
 
 module.exports = FileRotator;

test/filerotate.test.js

@@ -25,7 +25,7 @@ describe('RotatingFileStream', function () {
 
     it('Processes client configuration', async function () {
         const rfs = new RotatingFileStream(testConfig);
-        assert.strictEqual(rfs._path, 'logs/foo.log');
+        assert.strictEqual(rfs._path, testConfig.path);
         await rfs.end();
     });