Fix lint errors found by eslint.

Adds some missing assertions that eslint found.

Author: realityking

test/integration/cli-test.js

@@ -1,9 +1,3 @@
-/* eslint-disable
-    no-return-assign,
-    no-unused-vars,
-*/
-// TODO: This file was created by bulk-decaffeinate.
-// Fix any style issues and re-enable lint.
 const fs = require('fs');
 const { assert } = require('chai');
 const { exec } = require('child_process');
@@ -29,12 +23,12 @@ describe('Command line', () => {
         done();
       });
 
-      cli.on('exit', code => exitStatus = code);
+      cli.on('exit', (code) => { exitStatus = code; });
     });
 
     it('should not return nothing to stderr', () => assert.equal(stderr, ''));
 
-    it('should return with exit code 0', () => {});
+    it('should return with exit code 0', () => assert.equal(exitStatus, 0));
 
     it('should return parsed body to standard output', (done) => {
       const expectedOutputPath = './test/fixtures/get/expected-output';
@@ -63,12 +57,12 @@ describe('Command line', () => {
         done();
       });
 
-      cli.on('exit', code => exitStatus = code);
+      cli.on('exit', (code) => { exitStatus = code; });
     });
 
     it('should not return nothing to stderr', () => assert.equal(stderr, ''));
 
-    it('should return with exit code 0', () => {});
+    it('should return with exit code 0', () => assert.equal(exitStatus, 0));
 
     it('should return parsed body in API Blueprint format to standard output', (done) => {
       const expectedOutputPath = './test/fixtures/post/expected-output.md';
@@ -80,14 +74,12 @@ describe('Command line', () => {
   });
 
   describe('no input on stdin and no options', () => {
-    let stdout = '';
     let stderr = '';
     let exitStatus = '';
 
     before((done) => {
       const cmd = './bin/curl-trace-parser';
       const cli = exec(cmdPrefix + cmd, (error, out, err) => {
-        stdout = out;
         stderr = err;
         if (error) {
           exitStatus = error.code;

test/integration/js-api-test.js

@@ -1,14 +1,6 @@
-/* eslint-disable
-    no-shadow,
-    no-unused-vars,
-*/
-// TODO: This file was created by bulk-decaffeinate.
-// Fix any style issues and re-enable lint.
-
 const fs = require('fs');
 
 const { assert } = require('chai');
-const { exec } = require('child_process');
 const parser = require('../../lib/parser');
 
 describe('Javascript API', () =>
@@ -20,7 +12,7 @@ describe('Javascript API', () =>
 
       fs.readFile(traceFilePath, 'utf8', (err, trace) => {
         const parsed = parser.parseToString(trace);
-        fs.readFile(expectedOutputPath, 'utf8', (err, expected) => {
+        fs.readFile(expectedOutputPath, 'utf8', (error, expected) => {
           assert.equal(parsed, expected);
           done();
         });

test/unit/parser-test.js

@@ -1,10 +1,3 @@
-/* eslint-disable
-    no-return-assign,
-    prefer-destructuring,
-*/
-// TODO: This file was created by bulk-decaffeinate.
-// Fix any style issues and re-enable lint.
-
 const fs = require('fs');
 const { assert } = require('chai');
 const parser = require('../../lib/parser');
@@ -23,7 +16,7 @@ describe('parser module', () => {
   it('has parse() defined', () => assert.isFunction(parser.parse));
 
   describe('parse(string) return', () => {
-    before(() => output = parser.parse(trace));
+    before(() => { output = parser.parse(trace); });
 
     it('is obejct', () => assert.isObject(output));
 
@@ -34,7 +27,7 @@ describe('parser module', () => {
 
       describe('parsed request', () => {
         let request = '';
-        before(() => request = output.request);
+        before(() => { ({ request } = output); });
 
         it('contains multiple lines delimited by CRLF', () => {
           const lines = request.split('\r\n');
@@ -46,7 +39,6 @@ describe('parser module', () => {
           assert.include(request, agentString);
         });
 
-
         it('does not contain double LF at the end', () => {
           const outputChars = request.split('');
 
@@ -59,7 +51,7 @@ describe('parser module', () => {
 
       describe('parsed response', () => {
         let response = '';
-        before(() => response = output.response);
+        before(() => { ({ response } = output); });
 
         it('containt application/json', () => {
           const string = 'application/json';
@@ -77,7 +69,7 @@ describe('parser module', () => {
   it('has parseToString() defined', () => assert.isFunction(parser.parseToString));
 
   describe('parseToString(traceString) return', () => {
-    before(() => output = parser.parseToString(trace));
+    before(() => { output = parser.parseToString(trace); });
 
     it('should have trailing LF as last character to do not brake terminal', () => {
       const outputChars = output.split('');
@@ -99,7 +91,7 @@ describe('parser module', () => {
     });
 
     it("should have all parsed Request lines leaded by '> '", () => {
-      const request = parser.parse(trace).request;
+      const { request } = parser.parse(trace);
 
       let counter = 0;
       for (const line of request.split('\r\n')) {
@@ -110,7 +102,7 @@ describe('parser module', () => {
     });
 
     it("should have all parsed Response lines leaded by '< '", () => {
-      const response = parser.parse(trace).response;
+      const { response } = parser.parse(trace);
 
       let counter = 0;
       for (const line of response.split('\r\n')) {
@@ -142,7 +134,7 @@ describe('parser module', () => {
 
       describe('parsed request', () => {
         let request = '';
-        before(() => request = output.request);
+        before(() => { ({ request } = output); });
 
         it('is a string', () => assert.isString(request));
 
@@ -151,7 +143,7 @@ describe('parser module', () => {
 
       describe('parsed response', () => {
         let response = '';
-        before(() => response = output.response);
+        before(() => { ({ response } = output); });
 
         it('is a string', () => assert.isString(response));