Add validation for CLI arguments that don't take a value

Author: npaun

src/node_options-inl.h

@@ -268,6 +268,10 @@ inline std::string RequiresArgumentErr(const std::string& arg) {
   return arg + " requires an argument";
 }
 
+inline std::string UnexpectedArgumentErr(const std::string& arg) {
+  return arg + " does not take an argument";
+}
+
 inline std::string NegationImpliesBooleanError(const std::string& arg) {
   return arg + " is an invalid negation because it is not a boolean option";
 }
@@ -475,6 +479,12 @@ void OptionsParser<Options>::Parse(
             value = value.substr(1);  // Treat \- as escaping an -.
         }
       }
+    } else if (info.type == kBoolean) {
+        if (equals_index != std::string::npos) {
+          // Boolean options don't accept arguments
+          errors->push_back(UnexpectedArgumentErr(name));
+          break;
+        }
     }
 
     switch (info.type) {