test: fix export & import ct, entries test cases

Author: aman19K

packages/contentstack-export/test/unit/export/modules/content-types.test.ts

@@ -84,6 +84,9 @@ describe('ExportContentTypes', () => {
     // Stub FsUtility methods
     sinon.stub(FsUtility.prototype, 'writeFile').resolves();
     sinon.stub(FsUtility.prototype, 'makeDirectory').resolves();
+    // Stub FsUtility.prototype.readdir and readFile for readContentTypeSchemas support
+    sinon.stub(FsUtility.prototype, 'readdir').returns([]);
+    sinon.stub(FsUtility.prototype, 'readFile').returns(undefined);
   });
 
   afterEach(() => {

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

@@ -140,6 +140,11 @@ describe('EntriesExport', () => {
     createFolderStub.callsFake(() => {
       // Do nothing - prevent actual directory creation
     });
+    
+    // Stub FsUtility.prototype.readdir and readFile for readContentTypeSchemas support
+    // readContentTypeSchemas creates its own FsUtility instance, so we need to stub the prototype
+    sandbox.stub(FsUtility.prototype, 'readdir').returns([]);
+    sandbox.stub(FsUtility.prototype, 'readFile').returns(undefined);
 
     entriesExport = new EntriesExport({
       exportConfig: mockExportConfig,
@@ -173,16 +178,15 @@ describe('EntriesExport', () => {
         mockExportConfig.modules.locales.dirName,
         mockExportConfig.modules.locales.fileName,
       );
-      const expectedSchemaPath = path.resolve(
+      const expectedContentTypesDirPath = path.resolve(
         mockExportConfig.exportDir,
         mockExportConfig.branchName || '',
         mockExportConfig.modules.content_types.dirName,
-        'schema.json',
       );
 
       expect(entriesExport.entriesDirPath).to.equal(expectedEntriesPath);
       expect(entriesExport.localesFilePath).to.equal(expectedLocalesPath);
-      expect(entriesExport.schemaFilePath).to.equal(expectedSchemaPath);
+      expect(entriesExport.contentTypesDirPath).to.equal(expectedContentTypesDirPath);
     });
 
     it('should initialize ExportProjects instance', () => {
@@ -197,27 +201,29 @@ describe('EntriesExport', () => {
 
   describe('start() method - Early Returns', () => {
     it('should return early when no content types are found', async () => {
-      mockFsUtil.readFile
-        .onFirstCall()
-        .returns([{ code: 'en-us' }]) // locales
-        .onSecondCall()
-        .returns([]); // content types
+      // Stub mockFsUtil.readFile for locales
+      mockFsUtil.readFile.returns([{ code: 'en-us' }]);
+      
+      // Stub FsUtility.prototype for readContentTypeSchemas to return empty
+      (FsUtility.prototype.readdir as sinon.SinonStub).returns([]); // No content type files
 
       await entriesExport.start();
 
       // Should not attempt to fetch entries
       expect(mockStackAPIClient.contentType.called).to.be.false;
-      // Should read both locales and content types files
-      expect(mockFsUtil.readFile.calledTwice).to.be.true;
+      // Should read locales file
+      expect(mockFsUtil.readFile.called).to.be.true;
     });
 
     it('should handle empty locales array gracefully', async () => {
-      const contentTypes = [{ uid: 'ct-1', title: 'Content Type 1' }];
-      mockFsUtil.readFile
-        .onFirstCall()
-        .returns([]) // empty locales
-        .onSecondCall()
-        .returns(contentTypes);
+      const contentTypes = [{ uid: 'ct-1', title: 'Content Type 1', schema: [] as any }];
+      
+      // Stub mockFsUtil.readFile for locales
+      mockFsUtil.readFile.returns([]); // empty locales
+      
+      // Stub FsUtility.prototype for readContentTypeSchemas to return content types
+      (FsUtility.prototype.readdir as sinon.SinonStub).returns(['ct-1.json']);
+      (FsUtility.prototype.readFile as sinon.SinonStub).returns(contentTypes[0]);
 
       await entriesExport.start();
 
@@ -226,14 +232,14 @@ describe('EntriesExport', () => {
     });
 
     it('should handle non-array locales gracefully', async () => {
-      const contentTypes = [{ uid: 'ct-1', title: 'Content Type 1' }];
-      // Use empty array instead of null to avoid Object.keys error
-      // The code checks !Array.isArray first, so empty array will work
-      mockFsUtil.readFile
-        .onFirstCall()
-        .returns([]) // empty locales array
-        .onSecondCall()
-        .returns(contentTypes);
+      const contentTypes = [{ uid: 'ct-1', title: 'Content Type 1', schema: [] as any }];
+      
+      // Stub mockFsUtil.readFile for locales
+      mockFsUtil.readFile.returns([]); // empty locales array
+      
+      // Stub FsUtility.prototype for readContentTypeSchemas to return content types
+      (FsUtility.prototype.readdir as sinon.SinonStub).returns(['ct-1.json']);
+      (FsUtility.prototype.readFile as sinon.SinonStub).returns(contentTypes[0]);
 
       // Mock entry query for when entries are processed
       const mockEntryQuery = {

packages/contentstack-import/test/unit/import/modules/content-types.test.ts

@@ -1218,9 +1218,9 @@ describe('ImportContentTypes', () => {
       // Configure FsUtility stubs to make readContentTypeSchemas return mock content types
       (FsUtility.prototype.readdir as sinon.SinonStub).returns(['ct-1.json', 'ct-2.json']);
       (FsUtility.prototype.readFile as sinon.SinonStub).callsFake((filePath: string) => {
-        if (filePath.includes('ct-1.json')) return JSON.stringify(mockContentTypes[0]);
-        if (filePath.includes('ct-2.json')) return JSON.stringify(mockContentTypes[1]);
-        return '{}';
+        if (filePath.includes('ct-1.json')) return mockContentTypes[0];
+        if (filePath.includes('ct-2.json')) return mockContentTypes[1];
+        return {};
       });
 
       fsUtilStub.readFile.returns([]);

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

@@ -29,11 +29,13 @@ describe('EntriesImport', () => {
     });
 
     // Stub FsUtility prototype to support readContentTypeSchemas
-    // readContentTypeSchemas reads individual JSON files and returns parsed objects
+    // readContentTypeSchemas creates its own FsUtility instance, so we need to stub the prototype
     sinon.stub(FsUtility.prototype, 'readdir').returns([]);
-    sinon.stub(FsUtility.prototype, 'readFile').returns(undefined);
 
-    fsUtilityReadFileStub = sinon.stub(fsUtil, 'readFile');
+    // Stub FsUtility.prototype.readFile for readContentTypeSchemas (returns parsed objects)
+    // This also stubs fsUtil.readFile since fsUtil is an instance of FsUtility
+    // Don't set a default return value - let individual tests configure it
+    fsUtilityReadFileStub = sinon.stub(FsUtility.prototype, 'readFile');
     fsUtilityWriteFileStub = sinon.stub(fsUtil, 'writeFile').callsFake(() => {
       return Promise.resolve();
     });
@@ -509,12 +511,14 @@ describe('EntriesImport', () => {
           if (path.includes('schema.json')) {
             return [mockData.contentTypeWithFieldRules];
           }
-          return {};
+          if (path.includes('field_rules_ct.json')) {
+            return mockData.contentTypeWithFieldRules;
+          }
+          return undefined;
         });
 
-        // Override FsUtility stubs to return field_rules_ct
+        // Override FsUtility.prototype.readdir for readContentTypeSchemas
         (FsUtility.prototype.readdir as sinon.SinonStub).returns(['field_rules_ct.json']);
-        (FsUtility.prototype.readFile as sinon.SinonStub).returns(mockData.contentTypeWithFieldRules);
       });
 
       it('should update field rules with new UIDs', async () => {