Add InstanceCommand and MrtCommand integration tests

Integration tests provide API contract validation beyond code coverage:
- Catch flag definition errors (wrong type, missing env var)
- Validate baseFlags inheritance works correctly
- Exercise full oclif command lifecycle (discovery, parse, init, run)
- Test commands the way consumers actually use them

New fixtures:
- test-instance.js: Tests server, instance flags and hasServer check
- test-mrt.js: Tests api-key, project, environment, cloud-origin flags

New integration tests:
- instance-command.integration.test.ts (5 tests)
- mrt-command.integration.test.ts (8 tests)

Total integration tests: 18 (BaseCommand + InstanceCommand + MrtCommand)

Author: clavery

packages/b2c-tooling-sdk/test/cli/instance-command.integration.test.ts

@@ -0,0 +1,66 @@
+/*
+ * Copyright (c) 2025, Salesforce, Inc.
+ * SPDX-License-Identifier: Apache-2
+ * For full license text, see the license.txt file in the repo root or http://www.apache.org/licenses/LICENSE-2.0
+ */
+import {expect} from 'chai';
+import {runCommand} from '@oclif/test';
+import path from 'node:path';
+import {fileURLToPath} from 'node:url';
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const fixtureRoot = path.join(__dirname, '../fixtures/test-cli');
+
+interface TestInstanceResult {
+  server?: string;
+  hasServer: boolean;
+  instance?: string;
+}
+
+describe('InstanceCommand integration', () => {
+  it('runs test-instance command without errors', async () => {
+    const {error} = await runCommand(['test-instance'], {root: fixtureRoot});
+    expect(error).to.be.undefined;
+  });
+
+  it('handles --server flag', async () => {
+    const {error, result} = await runCommand<TestInstanceResult>(
+      ['test-instance', '--server', 'test.demandware.net', '--json'],
+      {root: fixtureRoot},
+    );
+
+    expect(error).to.be.undefined;
+    expect(result?.server).to.equal('test.demandware.net');
+    expect(result?.hasServer).to.be.true;
+  });
+
+  it('reports hasServer false when no server provided', async () => {
+    const {error, result} = await runCommand<TestInstanceResult>(['test-instance', '--json'], {root: fixtureRoot});
+
+    expect(error).to.be.undefined;
+    expect(result?.hasServer).to.be.false;
+    expect(result?.server).to.be.undefined;
+  });
+
+  it('creates instance when server is provided', async () => {
+    const {error, result} = await runCommand<TestInstanceResult>(
+      ['test-instance', '--server', 'test.demandware.net', '--json'],
+      {root: fixtureRoot},
+    );
+
+    expect(error).to.be.undefined;
+    expect(result?.instance).to.equal('test.demandware.net');
+  });
+
+  it('handles --instance flag for config selection', async () => {
+    // The --instance flag is for selecting a named instance from dw.json
+    // Without a dw.json, it just sets the flag value
+    const {error, result} = await runCommand<TestInstanceResult>(['test-instance', '--instance', 'staging', '--json'], {
+      root: fixtureRoot,
+    });
+
+    expect(error).to.be.undefined;
+    // Instance flag is for config selection, not server name
+    expect(result?.hasServer).to.be.false;
+  });
+});

packages/b2c-tooling-sdk/test/cli/mrt-command.integration.test.ts

@@ -0,0 +1,101 @@
+/*
+ * Copyright (c) 2025, Salesforce, Inc.
+ * SPDX-License-Identifier: Apache-2
+ * For full license text, see the license.txt file in the repo root or http://www.apache.org/licenses/LICENSE-2.0
+ */
+import {expect} from 'chai';
+import {runCommand} from '@oclif/test';
+import path from 'node:path';
+import {fileURLToPath} from 'node:url';
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const fixtureRoot = path.join(__dirname, '../fixtures/test-cli');
+
+interface TestMrtResult {
+  hasApiKey: boolean;
+  project?: string;
+  environment?: string;
+  cloudOrigin?: string;
+  credentialsFile?: string;
+  hasMrtCredentials: boolean;
+}
+
+describe('MrtCommand integration', () => {
+  it('runs test-mrt command without errors', async () => {
+    const {error} = await runCommand(['test-mrt'], {root: fixtureRoot});
+    expect(error).to.be.undefined;
+  });
+
+  it('handles --api-key flag', async () => {
+    const {error, result} = await runCommand<TestMrtResult>(['test-mrt', '--api-key', 'test-api-key-123', '--json'], {
+      root: fixtureRoot,
+    });
+
+    expect(error).to.be.undefined;
+    expect(result?.hasApiKey).to.be.true;
+    expect(result?.hasMrtCredentials).to.be.true;
+  });
+
+  it('handles --project flag', async () => {
+    const {error, result} = await runCommand<TestMrtResult>(['test-mrt', '--project', 'my-project', '--json'], {
+      root: fixtureRoot,
+    });
+
+    expect(error).to.be.undefined;
+    expect(result?.project).to.equal('my-project');
+  });
+
+  it('handles --environment flag', async () => {
+    const {error, result} = await runCommand<TestMrtResult>(['test-mrt', '--environment', 'staging', '--json'], {
+      root: fixtureRoot,
+    });
+
+    expect(error).to.be.undefined;
+    expect(result?.environment).to.equal('staging');
+  });
+
+  it('handles --cloud-origin flag', async () => {
+    const {error, result} = await runCommand<TestMrtResult>(
+      ['test-mrt', '--cloud-origin', 'https://custom.mobify.com', '--json'],
+      {root: fixtureRoot},
+    );
+
+    expect(error).to.be.undefined;
+    expect(result?.cloudOrigin).to.equal('https://custom.mobify.com');
+  });
+
+  it('handles --credentials-file flag', async () => {
+    const {error, result} = await runCommand<TestMrtResult>(
+      ['test-mrt', '--credentials-file', '/custom/path/.mobify', '--json'],
+      {root: fixtureRoot},
+    );
+
+    expect(error).to.be.undefined;
+    expect(result?.credentialsFile).to.equal('/custom/path/.mobify');
+  });
+
+  it('reports hasMrtCredentials false when no api-key provided', async () => {
+    // Use --credentials-file to isolate from developer's ~/.mobify
+    const {error, result} = await runCommand<TestMrtResult>(
+      ['test-mrt', '--credentials-file', '/dev/null', '--json'],
+      {root: fixtureRoot},
+    );
+
+    expect(error).to.be.undefined;
+    expect(result?.hasApiKey).to.be.false;
+    expect(result?.hasMrtCredentials).to.be.false;
+  });
+
+  it('handles multiple flags together', async () => {
+    const {error, result} = await runCommand<TestMrtResult>(
+      ['test-mrt', '--api-key', 'key123', '--project', 'proj', '--environment', 'prod', '--json'],
+      {root: fixtureRoot},
+    );
+
+    expect(error).to.be.undefined;
+    expect(result?.hasApiKey).to.be.true;
+    expect(result?.project).to.equal('proj');
+    expect(result?.environment).to.equal('prod');
+    expect(result?.hasMrtCredentials).to.be.true;
+  });
+});

packages/b2c-tooling-sdk/test/fixtures/test-cli/src/commands/test-instance.js

@@ -0,0 +1,42 @@
+/*
+ * Copyright (c) 2025, Salesforce, Inc.
+ * SPDX-License-Identifier: Apache-2
+ * For full license text, see the license.txt file in the repo root or http://www.apache.org/licenses/LICENSE-2.0
+ */
+// Use relative import since this fixture is inside the SDK's test folder
+import {InstanceCommand} from '../../../../../src/cli/index.js';
+
+/**
+ * Test command that extends InstanceCommand for integration testing.
+ * Exercises instance command features like server flag and instance getter.
+ */
+export default class TestInstance extends InstanceCommand {
+  static id = 'test-instance';
+  static description = 'Test command for InstanceCommand integration testing';
+  static enableJsonFlag = true;
+
+  async run() {
+    // Check server via resolvedConfig (no hasServer method on InstanceCommand)
+    const hasServer = Boolean(this.resolvedConfig.hostname);
+
+    // Return server/instance info without requiring server (for testing flags work)
+    const result = {
+      server: this.resolvedConfig.hostname,
+      hasServer,
+    };
+
+    // Only access instance if server is provided (avoids requireServer error)
+    if (hasServer) {
+      result.instance = this.instance.config.hostname;
+    }
+
+    if (this.jsonEnabled()) {
+      return result;
+    }
+
+    this.log('Server: ' + (result.server || 'not set'));
+    this.log('Has server: ' + result.hasServer);
+
+    return result;
+  }
+}

packages/b2c-tooling-sdk/test/fixtures/test-cli/src/commands/test-mrt.js

@@ -0,0 +1,40 @@
+/*
+ * Copyright (c) 2025, Salesforce, Inc.
+ * SPDX-License-Identifier: Apache-2
+ * For full license text, see the license.txt file in the repo root or http://www.apache.org/licenses/LICENSE-2.0
+ */
+// Use relative import since this fixture is inside the SDK's test folder
+import {MrtCommand} from '../../../../../src/cli/index.js';
+
+/**
+ * Test command that extends MrtCommand for integration testing.
+ * Exercises MRT command features like api-key, project, and environment flags.
+ */
+export default class TestMrt extends MrtCommand {
+  static id = 'test-mrt';
+  static description = 'Test command for MrtCommand integration testing';
+  static enableJsonFlag = true;
+
+  async run() {
+    const result = {
+      // Mask API key for security (just show if present)
+      hasApiKey: Boolean(this.flags['api-key']),
+      project: this.flags.project,
+      environment: this.flags.environment,
+      cloudOrigin: this.flags['cloud-origin'],
+      credentialsFile: this.flags['credentials-file'],
+      hasMrtCredentials: this.hasMrtCredentials(),
+    };
+
+    if (this.jsonEnabled()) {
+      return result;
+    }
+
+    this.log('Has API key: ' + result.hasApiKey);
+    this.log('Project: ' + (result.project || 'not set'));
+    this.log('Environment: ' + (result.environment || 'not set'));
+    this.log('Has MRT credentials: ' + result.hasMrtCredentials);
+
+    return result;
+  }
+}