ci: fix lint

X-Profiler · Sep 20, 2024 · 57fa1ca · 57fa1ca
1 parent d590002
commit 57fa1ca
Show file tree

Hide file tree

Showing 7 changed files with 14 additions and 14 deletions.
diff --git a/bin/xtransit.js b/bin/xtransit.js
@@ -1,7 +1,6 @@
 #!/usr/bin/env node
 'use strict';
 
-const os = require('os');
 const fs = require('fs');
 const path = require('path');
 const moment = require('moment');

diff --git a/commands/xprofctl.js b/commands/xprofctl.js
@@ -1,7 +1,6 @@
 'use strict';
 
 const fs = require('fs');
-const os = require('os');
 const path = require('path');
 const { promisify } = require('util');
 const exists = promisify(fs.exists);

diff --git a/common/error.js b/common/error.js
@@ -83,7 +83,7 @@ class Parser {
         reject(err);
       };
 
-      cleanup = function () {
+      cleanup = function() {
         readable.removeListener('data', onData);
         readable.removeListener('end', onEnd);
         readable.removeListener('error', onError);

diff --git a/common/utils.js b/common/utils.js
@@ -88,17 +88,17 @@ exports.getNodeProcessInfo = function(proc, platform) {
 
 exports.sleep = promisify(setTimeout);
 
-exports.getXprofilerPath = function () {
+exports.getXprofilerPath = function() {
   const prefix = process.env.XPROFILER_PREFIX || process.env.XPROFILER_UNIT_TEST_TMP_HOMEDIR || os.homedir();
   return path.join(prefix, '.xprofiler');
-}
+};
 
-exports.getXtransitPath = function () {
+exports.getXtransitPath = function() {
   const prefix = process.env.XTRANSIT_PREFIX || process.env.XPROFILER_UNIT_TEST_TMP_HOMEDIR || os.homedir();
   return path.join(prefix, '.xtransit');
-}
+};
 
-exports.getXtransitLogPath = function () {
+exports.getXtransitLogPath = function() {
   const prefix = process.env.XTRANSIT_PREFIX || process.env.XPROFILER_UNIT_TEST_TMP_HOMEDIR || os.homedir();
   return path.join(prefix, '.xtransit.log');
-}
+};
diff --git a/orders/xprofiler_log.js b/orders/xprofiler_log.js
@@ -130,7 +130,7 @@ exports = module.exports = async function() {
     lines: process.env.UNIT_TEST && logs[0] ? logs[0].split('\n').length : 0,
   };
 
-  if(typeof this.customXprofilerLogs === 'function') {
+  if (typeof this.customXprofilerLogs === 'function') {
     const logs = message.data.logs;
     logs.push(...await this.customXprofilerLogs(logs));
   }

diff --git a/test/config.test.js b/test/config.test.js
@@ -20,6 +20,8 @@ function getError(config) {
 }
 
 describe('get config', function() {
+  afterEach(mm.restore);
+
   it('should throw error', function() {
     expect(getError({})).to.be('config.server must be passed in!');
     expect(getError({ server: 'mock' })).to.be('config.server must be passed in!');

diff --git a/test/utils.test.js b/test/utils.test.js
@@ -5,7 +5,7 @@ const assert = require('assert');
 const mm = require('mm');
 const os = require('os');
 
-describe('utils.test.js', function () {
+describe('utils.test.js', function() {
   afterEach(mm.restore);
 
   describe('get xprofiler path', () => {
@@ -17,7 +17,7 @@ describe('utils.test.js', function () {
 
     it('should work default is home', () => {
       mm(os, 'homedir', () => {
-        return '/home/xxx'
+        return '/home/xxx';
       });
       const p = utils.getXprofilerPath();
       assert.equal(p, '/home/xxx/.xprofiler');
@@ -33,7 +33,7 @@ describe('utils.test.js', function () {
 
     it('should work default is home', () => {
       mm(os, 'homedir', () => {
-        return '/home/xxx'
+        return '/home/xxx';
       });
       const p = utils.getXtransitPath();
       assert.equal(p, '/home/xxx/.xtransit');
@@ -49,7 +49,7 @@ describe('utils.test.js', function () {
 
     it('should work default is home', () => {
       mm(os, 'homedir', () => {
-        return '/home/xxx'
+        return '/home/xxx';
       });
       const p = utils.getXtransitLogPath();
       assert.equal(p, '/home/xxx/.xtransit.log');