Skip to content

Commit 521ba9e

Browse files
author
naman-contentstack
committed
chore: update test cases
1 parent 4536bd2 commit 521ba9e

6 files changed

Lines changed: 16 additions & 32 deletions

File tree

packages/contentstack-export/test/unit/export/modules/assets.test.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -136,6 +136,9 @@ describe('ExportAssets', () => {
136136
enableDownloadStatus: false,
137137
includeVersionedAssets: false,
138138
},
139+
'asset-management': {
140+
chunkFileSizeMb: 1,
141+
},
139142
content_types: {
140143
dirName: 'content_types',
141144
fileName: 'content_types.json',

packages/contentstack-export/test/unit/export/modules/base-class.test.ts

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -154,6 +154,9 @@ describe('BaseClass', () => {
154154
enableDownloadStatus: false,
155155
includeVersionedAssets: false,
156156
},
157+
'asset-management': {
158+
chunkFileSizeMb: 1,
159+
},
157160
content_types: {
158161
dirName: 'content_types',
159162
fileName: 'content_types.json',

packages/contentstack-export/test/unit/export/modules/marketplace-apps.test.ts

Lines changed: 5 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ import * as utilities from '@contentstack/cli-utilities';
55
import ExportMarketplaceApps from '../../../../src/export/modules/marketplace-apps';
66
import ExportConfig from '../../../../src/types/export-config';
77
import * as marketplaceAppHelper from '../../../../src/utils/marketplace-app-helper';
8-
import * as utils from '../../../../src/utils';
98

109
describe('ExportMarketplaceApps', () => {
1110
let exportMarketplaceApps: any;
@@ -106,10 +105,6 @@ describe('ExportMarketplaceApps', () => {
106105
sinon.stub(marketplaceAppHelper, 'getDeveloperHubUrl').resolves('https://developer-api.contentstack.io');
107106
sinon.stub(marketplaceAppHelper, 'createNodeCryptoInstance').resolves(mockNodeCrypto);
108107
sinon.stub(marketplaceAppHelper, 'askEncryptionKey').resolves('test-encryption-key');
109-
sinon.stub(utils, 'getOrgUid').resolves('test-org-uid');
110-
sinon.stub(utils, 'getDeveloperHubUrl').resolves('https://developer-api.contentstack.io');
111-
sinon.stub(utils, 'createNodeCryptoInstance').resolves(mockNodeCrypto);
112-
sinon.stub(utils, 'askEncryptionKey').resolves('test-encryption-key');
113108
});
114109

115110
afterEach(() => {
@@ -300,14 +295,7 @@ describe('ExportMarketplaceApps', () => {
300295

301296
await exportMarketplaceApps.start();
302297

303-
// Source imports from utils barrel; resolution may use utils or marketplaceAppHelper depending on env (CI vs local)
304-
const helperCalled = (marketplaceAppHelper.createNodeCryptoInstance as sinon.SinonStub).calledOnce;
305-
const utilsCalled = (utils.createNodeCryptoInstance as sinon.SinonStub).calledOnce;
306-
expect(helperCalled || utilsCalled, 'createNodeCryptoInstance should be called exactly once').to.be.true;
307-
expect(
308-
(marketplaceAppHelper.createNodeCryptoInstance as sinon.SinonStub).callCount +
309-
(utils.createNodeCryptoInstance as sinon.SinonStub).callCount,
310-
).to.equal(1);
298+
expect((marketplaceAppHelper.createNodeCryptoInstance as sinon.SinonStub).calledOnce).to.be.true;
311299

312300
getAppsCountStub.restore();
313301
exportAppsStub.restore();
@@ -409,12 +397,12 @@ describe('ExportMarketplaceApps', () => {
409397
const getAppManifestAndAppConfigStub = sinon.stub(exportMarketplaceApps, 'getAppManifestAndAppConfig').resolves();
410398

411399
// Reset the stub call count since it might have been called in previous tests
412-
(utils.createNodeCryptoInstance as sinon.SinonStub).resetHistory();
400+
(marketplaceAppHelper.createNodeCryptoInstance as sinon.SinonStub).resetHistory();
413401

414402
await exportMarketplaceApps.exportApps();
415403

416404
// NodeCrypto should not be initialized if no configurations
417-
expect((utils.createNodeCryptoInstance as sinon.SinonStub).called).to.be.false;
405+
expect((marketplaceAppHelper.createNodeCryptoInstance as sinon.SinonStub).called).to.be.false;
418406

419407
getStackSpecificAppsStub.restore();
420408
getAppManifestAndAppConfigStub.restore();
@@ -785,11 +773,11 @@ describe('ExportMarketplaceApps', () => {
785773
});
786774

787775
// Reset the stub call count since it was called in beforeEach
788-
(utils.createNodeCryptoInstance as sinon.SinonStub).resetHistory();
776+
(marketplaceAppHelper.createNodeCryptoInstance as sinon.SinonStub).resetHistory();
789777

790778
await exportMarketplaceApps.getAppConfigurations(0, exportMarketplaceApps.installedApps[0]);
791779

792-
expect((utils.createNodeCryptoInstance as sinon.SinonStub).called).to.be.true;
780+
expect((marketplaceAppHelper.createNodeCryptoInstance as sinon.SinonStub).called).to.be.true;
793781
expect(exportMarketplaceApps.nodeCrypto).to.exist;
794782
expect((marketplaceAppHelper.createNodeCryptoInstance as sinon.SinonStub).called).to.be.true;
795783
});

packages/contentstack-import/test/unit/import/module-importer.test.ts

Lines changed: 3 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -509,7 +509,7 @@ describe('ModuleImporter', () => {
509509

510510
await importer.start();
511511

512-
expect(importer['stackAPIClient'].locale.calledOnce).to.be.true;
512+
expect(masterLocalDetailsStub.calledOnce).to.be.true;
513513
expect(importer['importConfig'].master_locale).to.deep.equal({ code: 'en-us' });
514514
expect(importer['importConfig'].masterLocale).to.deep.equal({ code: 'en-us' });
515515
});
@@ -534,13 +534,7 @@ describe('ModuleImporter', () => {
534534

535535
it('should set both master_locale and masterLocale', async () => {
536536
mockImportConfig.master_locale = undefined;
537-
538-
const localeMock = {
539-
query: sandbox.stub().returnsThis(),
540-
find: sandbox.stub().resolves({ items: [{ code: 'de-de' }] }),
541-
};
542-
mockStackClient.locale = sandbox.stub().returns(localeMock);
543-
mockStackClient._localeMock = localeMock;
537+
masterLocalDetailsStub.resolves({ code: 'de-de' });
544538

545539
const importer = new ModuleImporter(mockManagementClient as any, mockImportConfig);
546540

@@ -552,13 +546,7 @@ describe('ModuleImporter', () => {
552546

553547
it('should handle error when masterLocalDetails fails', async () => {
554548
mockImportConfig.master_locale = undefined;
555-
556-
const localeMock = {
557-
query: sandbox.stub().returnsThis(),
558-
find: sandbox.stub().rejects(new Error('Master locale fetch failed')),
559-
};
560-
mockStackClient.locale = sandbox.stub().returns(localeMock);
561-
mockStackClient._localeMock = localeMock;
549+
masterLocalDetailsStub.rejects(new Error('Master locale fetch failed'));
562550

563551
const importer = new ModuleImporter(mockManagementClient as any, mockImportConfig);
564552

packages/contentstack-import/test/unit/import/modules/locales.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ describe('ImportLocales', () => {
2626
management_token: 'test-token',
2727
contentDir: tempDir,
2828
modules: {
29+
apiConcurrency: 5,
2930
types: [],
3031
locales: {
3132
dirName: 'locales',

packages/contentstack-import/test/unit/utils/extension-helper.test.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ describe('Extension Helper', () => {
3434
contentDir: '/test/content',
3535
data: '/test/content',
3636
modules: {
37+
apiConcurrency: 5,
3738
'composable-studio': {
3839
dirName: 'composable_studio',
3940
fileName: 'composable_studio.json',

0 commit comments

Comments
 (0)