fix: update listBundles test to use new API signature and fix lint errors

- Update push.test.ts to use the new listBundles({projectSlug}, auth) signature
- Change import style in download.ts to use named imports per eslint rules

Author: clavery

packages/b2c-cli/src/commands/mrt/bundle/download.ts

@@ -3,8 +3,8 @@
  * 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 * as fs from 'node:fs';
-import * as path from 'node:path';
+import {createWriteStream, mkdirSync} from 'node:fs';
+import {dirname, resolve} from 'node:path';
 import {pipeline} from 'node:stream/promises';
 import {Args, Flags} from '@oclif/core';
 import {MrtCommand} from '@salesforce/b2c-tooling-sdk/cli';
@@ -92,7 +92,7 @@ export default class MrtBundleDownload extends MrtCommand<typeof MrtBundleDownlo
 
       // Download the file
       const outputPath = this.flags.output ?? `bundle-${bundleId}.tgz`;
-      const absolutePath = path.resolve(outputPath);
+      const absolutePath = resolve(outputPath);
 
       this.log(
         t('commands.mrt.bundle.download.downloading', 'Downloading bundle {{bundleId}} to {{filePath}}...', {
@@ -115,13 +115,13 @@ export default class MrtBundleDownload extends MrtCommand<typeof MrtBundleDownlo
       }
 
       // Ensure directory exists
-      const dir = path.dirname(absolutePath);
+      const dir = dirname(absolutePath);
       if (dir !== '.') {
-        fs.mkdirSync(dir, {recursive: true});
+        mkdirSync(dir, {recursive: true});
       }
 
       // Stream the response to file
-      const fileStream = fs.createWriteStream(absolutePath);
+      const fileStream = createWriteStream(absolutePath);
       await pipeline(response.body, fileStream);
 
       if (!this.jsonEnabled()) {

packages/b2c-tooling-sdk/test/operations/mrt/push.test.ts

@@ -174,13 +174,13 @@ describe('operations/mrt/push', () => {
       );
 
       const auth = new MockAuthStrategy();
-      const client = createMrtClient({}, auth);
 
-      const bundles = await listBundles(client, 'my-project');
+      const result = await listBundles({projectSlug: 'my-project'}, auth);
 
-      expect(bundles).to.have.length(2);
-      expect(bundles[0]).to.deep.include({id: 123, message: 'Bundle 1'});
-      expect(bundles[1]).to.deep.include({id: 124, message: 'Bundle 2'});
+      expect(result.bundles).to.have.length(2);
+      expect(result.bundles[0]).to.deep.include({id: 123, message: 'Bundle 1'});
+      expect(result.bundles[1]).to.deep.include({id: 124, message: 'Bundle 2'});
+      expect(result.count).to.equal(2);
     });
 
     it('passes pagination options', async () => {
@@ -189,14 +189,13 @@ describe('operations/mrt/push', () => {
       server.use(
         http.get(`${DEFAULT_BASE_URL}/api/projects/:projectSlug/bundles/`, ({request}) => {
           queryParams = new URL(request.url).searchParams;
-          return HttpResponse.json({results: []});
+          return HttpResponse.json({count: 0, results: []});
         }),
       );
 
       const auth = new MockAuthStrategy();
-      const client = createMrtClient({}, auth);
 
-      await listBundles(client, 'my-project', {limit: 10, offset: 20});
+      await listBundles({projectSlug: 'my-project', limit: 10, offset: 20}, auth);
 
       expect(queryParams?.get('limit')).to.equal('10');
       expect(queryParams?.get('offset')).to.equal('20');
@@ -213,11 +212,11 @@ describe('operations/mrt/push', () => {
       );
 
       const auth = new MockAuthStrategy();
-      const client = createMrtClient({}, auth);
 
-      const bundles = await listBundles(client, 'my-project');
+      const result = await listBundles({projectSlug: 'my-project'}, auth);
 
-      expect(bundles).to.have.length(0);
+      expect(result.bundles).to.have.length(0);
+      expect(result.count).to.equal(0);
     });
 
     it('throws error on API failure', async () => {
@@ -228,10 +227,9 @@ describe('operations/mrt/push', () => {
       );
 
       const auth = new MockAuthStrategy();
-      const client = createMrtClient({}, auth);
 
       try {
-        await listBundles(client, 'my-project');
+        await listBundles({projectSlug: 'my-project'}, auth);
         expect.fail('Should have thrown');
       } catch (error) {
         expect((error as Error).message).to.include('Failed to list bundles');