Add missing check for broken passThrough

lib/command.js

@@ -272,6 +272,8 @@ class Command extends EventEmitter {
 
     this.commands.push(cmd);
     cmd.parent = this;
+    this._checkForBrokenPassThrough(cmd, this);
+
     return this;
   }
 
@@ -735,12 +737,22 @@ Expecting one of '${allowedValues.join("', '")}'`);
    */
   passThroughOptions(passThrough = true) {
     this._passThroughOptions = !!passThrough;
-    if (!!this.parent && passThrough && !this.parent._enablePositionalOptions) {
-      throw new Error('passThroughOptions can not be used without turning on enablePositionalOptions for parent command(s)');
-    }
+    this._checkForBrokenPassThrough(this, this.parent);
     return this;
   }
 
+  /**
+   * @param {Command} command
+   * @param {Command | null} parent
+   * @api private
+   */
+
+  _checkForBrokenPassThrough(command, parent) {
+    if (parent && command._passThroughOptions && !parent._enablePositionalOptions) {
+      throw new Error(`passThroughOptions cannot be used for '${command._name}' without turning on enablePositionalOptions for parent command${parent._name ? ` '${parent._name}'` : ''}`);
+    }
+  }
+
   /**
     * Whether to store option values as properties on command object,
     * or store separately (specify false). In both cases the option values can be accessed using .opts().

tests/command.positionalOptions.test.js

@@ -362,13 +362,25 @@ describe('program with action handler and positionalOptions and subcommand', ()
 
 // ------------------------------------------------------------------------------
 
-test('when program not positional and turn on passthrough in subcommand then error', () => {
-  const program = new commander.Command();
-  const sub = program.command('sub');
+describe('broken passThrough', () => {
+  test('when program not positional and turn on passThroughOptions in subcommand then error', () => {
+    const program = new commander.Command();
+    const sub = program.command('sub');
+
+    expect(() => {
+      sub.passThroughOptions();
+    }).toThrow();
+  });
 
-  expect(() => {
-    sub.passThroughOptions();
-  }).toThrow();
+  test('when program not positional and add subcommand with passThroughOptions then error', () => {
+    const program = new commander.Command();
+    const sub = new commander.Command('sub')
+      .passThroughOptions();
+
+    expect(() => {
+      program.addCommand(sub);
+    }).toThrow();
+  });
 });
 
 // ------------------------------------------------------------------------------