test: e2e not workin 7

Author: dbale-altoros

e2e/autofix-test.js

@@ -25,8 +25,6 @@ function retrieveParams(subpath) {
 function compareTextFiles(file1Path, file2Path) {
   const file1Content = fs.readFileSync(file1Path, 'utf-8')
   const file2Content = fs.readFileSync(file2Path, 'utf-8')
-  console.log('file1Content :>> ', file1Content)
-  console.log('file2Content :>> ', file2Content)
   return file1Content === file2Content
 }
 
@@ -52,6 +50,8 @@ function useFixture(dir) {
 
 describe('e2e', function () {
   let result = false
+  let code
+  let stdout
 
   describe('autofix tests', () => {
     if (E2E) {
@@ -166,19 +166,23 @@ describe('e2e', function () {
           expect(result).to.be.true
         })
 
-        it('should compare Foo1 file with template AFTER FIX file and they should match (2)', () => {
-          const { code, stdout } = shell.exec(
+        it('should execute and compare Foo1 with template AFTER FIX and they should match (2)', () => {
+          ({ code, stdout } = shell.exec(
             `${params.command} ${params.param1} -c ${currentConfig} ${currentFile} --fix --disc --noPrompt`
-          )
+          ))
+
+          result = compareTextFiles(currentFile, afterFixFile)
+          expect(result).to.be.true
+        })
 
+        it('should execute and exit with code 1 (2)', () => {
           expect(code).to.equal(1)
+        })
 
+        it('should get the right report (2)', () => {
           const reportLines = stdout.split('\n')
           const finalLine = '5 problems (5 errors, 0 warnings)'
           expect(reportLines[reportLines.length - 3]).to.contain(finalLine)
-
-          result = compareTextFiles(currentFile, afterFixFile)
-          expect(result).to.be.true
         })
       })
 
@@ -208,19 +212,23 @@ describe('e2e', function () {
           expect(result).to.be.true
         })
 
-        it('should compare Foo1 file with template AFTER FIX file and they should match (3)', () => {
-          const { code, stdout } = shell.exec(
+        it('should execute and compare Foo1 with template AFTER FIX and they should match (3)', () => {
+          ({ code, stdout } = shell.exec(
             `${params.command} ${params.param1} -c ${currentConfig} ${currentFile} --fix --disc --noPrompt`
-          )
+          ))
+
+          result = compareTextFiles(currentFile, afterFixFile)
+          expect(result).to.be.true
+        })
 
+        it('should execute and exit with code 1 (3)', () => {
           expect(code).to.equal(1)
+        })
 
+        it('should get the right report (3)', () => {
           const reportLines = stdout.split('\n')
           const finalLine = '9 problems (9 errors, 0 warnings)'
           expect(reportLines[reportLines.length - 3]).to.contain(finalLine)
-
-          result = compareTextFiles(currentFile, afterFixFile)
-          expect(result).to.be.true
         })
       })
 
@@ -250,31 +258,33 @@ describe('e2e', function () {
           expect(result).to.be.true
         })
 
-        it('should compare Foo1 file with template AFTER FIX file and they should match (4)', () => {
-          const { code, stdout } = shell.exec(
+        it('should execute and compare Foo1 with template AFTER FIX and they should match (4)', () => {
+          ({ code, stdout } = shell.exec(
             `${params.command} ${params.param1} -c ${currentConfig} ${currentFile} --fix --disc --noPrompt`
-          )
+          ))
 
+          result = compareTextFiles(currentFile, afterFixFile)
+          expect(result).to.be.true
+        })
+
+        it('should execute and exit with code 1 (4)', () => {
           expect(code).to.equal(1)
+        })
 
+        it('should get the right report (4)', () => {
           const reportLines = stdout.split('\n')
           const finalLine = '19 problems (19 errors, 0 warnings)'
           expect(reportLines[reportLines.length - 3]).to.contain(finalLine)
-
-          result = compareTextFiles(currentFile, afterFixFile)
-          expect(result).to.be.true
         })
       })
+
       it('should check FOO1 does not change after test (4)', () => {
         result = compareTextFiles(currentFile, beforeFixFile)
         expect(result).to.be.true
       })
     })
 
-    describe.only('autofix rule: payable-fallback', () => {
-      let code
-      let stdout
-
+    describe('autofix rule: payable-fallback', () => {
       before(function () {
         params = retrieveParams('payable-fallback/')
         currentConfig = `${params.path}${params.subpath}.solhint.json`
@@ -289,7 +299,7 @@ describe('e2e', function () {
           }
         })
 
-        xit('should compare Foo1 file with template BEFORE FIX file and they should match (5)', () => {
+        it('should compare Foo1 file with template BEFORE FIX file and they should match (5)', () => {
           result = compareTextFiles(currentFile, beforeFixFile)
           expect(result).to.be.true
         })
@@ -314,7 +324,7 @@ describe('e2e', function () {
         })
       })
 
-      xit('should check FOO1 does not change after test (5)', () => {
+      it('should check FOO1 does not change after test (5)', () => {
         result = compareTextFiles(currentFile, beforeFixFile)
         expect(result).to.be.true
       })