Merge pull request #111 from snyk/snyk-fix-93368b07b9de27f1dbf0560f8b…

…a14c21 [Snyk] Security upgrade acorn from 5.7.1 to 5.7.4
snyk · Apr 26, 2020 · ade9436 · ade9436
2 parents a502cbb + 4b5269a
commit ade9436
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 6 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -43,7 +43,7 @@
     "tap": "^14.6.1"
   },
   "dependencies": {
-    "acorn": "5.7.1",
+    "acorn": "5.7.4",
     "debug": "^4.1.1",
     "needle": "^2.4.0",
     "semver": "^6.3.0",

diff --git a/test/method-detection.test.js b/test/method-detection.test.js
@@ -27,15 +27,17 @@ test('test bootstrap +function method detection', function (t) {
 test('test clashing variable/function declaration', function (t) {
   const contents = `
 var foo;
-function foo() {
+{
+  function foo() {
+  }
 }
 `;
   const methods = ['foo'];
   const found = ast.findAllVulnerableFunctionsInScript(
     contents, methods,
   );
   t.same(sorted(Object.keys(found)), sorted(methods));
-  t.equal(found[methods[0]].start.line, 3, 'foo');
+  t.equal(found[methods[0]].start.line, 4, 'foo');
   t.end();
 });