Merge pull request #1335 from stof/fix_eslint_warning

Fix ESLint warning

Author: stof

package.json

@@ -7,7 +7,7 @@
     "test": "yarn run test:main && yarn run test:persistent-cache",
     "test:main": "mocha --reporter spec test --recursive --ignore test/persistent-cache/*",
     "test:persistent-cache": "node run-persistent-tests",
-    "lint": "eslint lib test index.js .eslintrc.js --report-unused-disable-directives",
+    "lint": "eslint lib test index.js .eslintrc.js --report-unused-disable-directives --max-warnings=0",
     "travis:lint": "yarn run lint"
   },
   "bin": {

test/functional.js

@@ -1248,20 +1248,17 @@ module.exports = {
                     );
 
                     // Test that the generated scripts work fine
-                    await new Promise(resolve => {
-                        testSetup.requestTestPage(
-                            browser,
-                            path.join(config.getContext(), 'www'),
-                            [
-                                'build/runtime.js',
-                                `build/${scriptName}`,
-                            ],
-                            async({ page }) => {
-                                expect(await page.evaluate(() => document.body.textContent)).to.contains('[1,2,3,4]');
-                                resolve();
-                            }
-                        );
-                    });
+                    await testSetup.requestTestPage(
+                        browser,
+                        path.join(config.getContext(), 'www'),
+                        [
+                            'build/runtime.js',
+                            `build/${scriptName}`,
+                        ],
+                        async({ page }) => {
+                            expect(await page.evaluate(() => document.body.textContent)).to.contains('[1,2,3,4]');
+                        }
+                    );
                 }
 
                 done();