Assume methods are non-optional for now

This makes e.g. `app.commandLine.appendSwitch` non-optional and does not
result in false positives.

Author: poiru

lib/utils.js

@@ -122,8 +122,10 @@ const isOptional = (param) => {
     return !param.required
   }
 
+  // Assume that methods are never optional because electron-docs-linter
+  // doesn't currently mark them as required.
   debug(`Could not determine optionality for ${param.name}`)
-  return true
+  return param.type !== 'Function'
 }
 
 const genMethodString = (paramInterfaces, module, moduleMethod, parameters, returns, includeType, paramTypePrefix) => {

test/utils_spec.js

@@ -107,5 +107,13 @@ describe('utils', () => {
     it('should return false if param is required', () => {
       expect(utils.isOptional({ required: true })).to.eq(false)
     })
+
+    it('should default to true if param is a non-function', () => {
+      expect(utils.isOptional({ type: 'Foo' })).to.eq(true)
+    })
+
+    it('should default to false if param is a function', () => {
+      expect(utils.isOptional({ type: 'Function' })).to.eq(false)
+    })
   })
 })