fix(require-returns-check): check WithStatement for return statements

Author: brettz9

README.md

@@ -6622,6 +6622,16 @@ function quux () {
   }
 }
 
+/**
+ * @returns {true}
+ */
+function quux () {
+  var a = {};
+  with (a) {
+    return true;
+  }
+}
+
 /**
  * @returns {true}
  */

src/jsdocUtils.js

@@ -245,7 +245,8 @@ const STATEMENTS_WITH_CHILDREN = [
   'SwitchStatement',
   'IfStatement',
   'BlockStatement',
-  'TryStatement'
+  'TryStatement',
+  'WithStatement'
 ];
 
 const RETURNFREE_STATEMENTS = [
@@ -257,7 +258,6 @@ const RETURNFREE_STATEMENTS = [
   'LabeledStatement',
   'DebuggerStatement',
   'EmptyStatement',
-  'WithStatement',
   'ThrowStatement',
   'ExpressionStatement'
 ];
@@ -285,6 +285,14 @@ const lookupTable = {
       return true;
     }
   },
+  WithStatement: {
+    is (node) {
+      return node.type === 'WithStatement';
+    },
+    check (node, context) {
+      return lookupTable.BlockStatement.check(node.body, context);
+    }
+  },
   IfStatement: {
     is (node) {
       return node.type === 'IfStatement';

test/rules/assertions/requireReturnsCheck.js

@@ -457,6 +457,19 @@ export default {
           }
       `
     },
+    {
+      code: `
+          /**
+           * @returns {true}
+           */
+          function quux () {
+            var a = {};
+            with (a) {
+              return true;
+            }
+          }
+      `
+    },
     {
       code: `
           /**