Fixed the warning

Author: cs-raj

packages/contentstack-export/src/utils/export-config-handler.ts

@@ -42,13 +42,11 @@ const setupConfig = async (exportCmdFlags: any): Promise<ExportConfig> => {
   config.exportDir = config.exportDir.replace(/['"]/g, '');
   config.exportDir = path.resolve(config.exportDir);
 
-  if (exportCmdFlags['data-dir'] || exportCmdFlags['data']) {
-    if (isDirectoryNonEmpty(config.exportDir)) {
-      cliux.print(
-        '\nThe export directory is not empty. Existing files in this folder may be overwritten.',
-        { color: 'yellow' },
-      );
-    }
+  if (isDirectoryNonEmpty(config.exportDir)) {
+    cliux.print(
+      '\nThe export directory is not empty. Existing files in this folder may be overwritten.',
+      { color: 'yellow' },
+    );
   }
 
   const managementTokenAlias = exportCmdFlags['management-token-alias'] || exportCmdFlags['alias'];

packages/contentstack-export/test/unit/utils/export-config-handler.test.ts

@@ -63,7 +63,7 @@ describe('Export Config Handler', () => {
       expect(askExportDirStub.called).to.be.false;
     });
 
-    it('should print yellow warning when data-dir is set and export directory is not empty', async () => {
+    it('should print yellow warning when export directory is not empty (data-dir flag)', async () => {
       configHandlerGetStub.withArgs('authorisationType').returns('OAUTH');
       const isNonEmptyStub = sandbox.stub(fileHelper, 'isDirectoryNonEmpty').returns(true);
       const cliuxPrint = utilities.cliux.print as sinon.SinonStub;
@@ -81,6 +81,24 @@ describe('Export Config Handler', () => {
       ).to.be.true;
     });
 
+    it('should print yellow warning when export directory is not empty (interactive path)', async () => {
+      configHandlerGetStub.withArgs('authorisationType').returns('OAUTH');
+      const isNonEmptyStub = sandbox.stub(fileHelper, 'isDirectoryNonEmpty').returns(true);
+      const cliuxPrint = utilities.cliux.print as sinon.SinonStub;
+      cliuxPrint.resetHistory();
+
+      const flags = {};
+      await setupConfig(flags);
+
+      expect(isNonEmptyStub.calledWith(path.resolve('/default/export/dir'))).to.be.true;
+      expect(
+        cliuxPrint.calledWith(
+          '\nThe export directory is not empty. Existing files in this folder may be overwritten.',
+          { color: 'yellow' },
+        ),
+      ).to.be.true;
+    });
+
     it('should ask for export directory when not provided', async () => {
       // Set authenticated: isAuthenticated() checks configHandler.get('authorisationType')
       // Returns 'OAUTH' or 'AUTH' for authenticated, undefined for not authenticated