@capillarytech/creatives-library 8.0.263 → 8.0.265

package/v2Containers/EmailWrapper/tests/useEmailWrapper.edgeCases.test.js

@@ -1,956 +0,0 @@
-/**
- * useEmailWrapper Hook Tests - Edge Cases Coverage
- *
- * Additional tests to cover edge cases and remaining uncovered lines
- */
-
-import { renderHook, waitFor, act as reactAct } from '@testing-library/react';
-import { STEPS, EMAIL_CREATE_MODES } from '../constants';
-import useEmailWrapper from '../hooks/useEmailWrapper';
-import { EmailWrapperMockData } from '../mockdata/mockdata';
-import { gtmPush } from '../../../utils/gtmTrackers';
-
-// Mock dependencies
-jest.mock('lodash', () => ({
-  ...jest.requireActual('lodash'),
-  isEmpty: jest.fn().mockImplementation((val) => {
-    if (val === null || val === undefined) return true;
-    if (typeof val === 'object') return Object.keys(val).length === 0;
-    if (typeof val === 'string') return val.trim().length === 0;
-    return !val;
-  }),
-  find: jest.fn(),
-  get: jest.fn((obj, path, defaultValue) => {
-    const keys = path.split('.');
-    let result = obj;
-    for (const key of keys) {
-      if (result && typeof result === 'object' && key in result) {
-        result = result[key];
-      } else {
-        return defaultValue;
-      }
-    }
-    return result;
-  }),
-}));
-
-let mockCapNotificationError;
-jest.mock('@capillarytech/cap-ui-library/CapNotification', () => {
-  mockCapNotificationError = jest.fn();
-  return {
-    error: mockCapNotificationError,
-  };
-});
-
-jest.mock('../../../utils/gtmTrackers', () => ({
-  gtmPush: jest.fn(),
-}));
-
-jest.mock('../../../utils/common', () => ({
-  hasSupportCKEditor: jest.fn(() => false),
-}));
-
-const mockStopTimer = jest.fn();
-
-jest.mock('@capillarytech/cap-ui-utils', () => ({
-  ...jest.requireActual('@capillarytech/cap-ui-utils'),
-  GA: {
-    timeTracker: {
-      startTimer: jest.fn(),
-      stopTimer: jest.fn(() => 1000),
-    },
-  },
-}));
-
-describe('useEmailWrapper - Edge Cases', () => {
-  let mockProps;
-
-  beforeEach(() => {
-    jest.clearAllMocks();
-    mockCapNotificationError.mockClear();
-    gtmPush.mockClear();
-    mockStopTimer.mockClear();
-
-    mockProps = {
-      ...EmailWrapperMockData,
-      intl: { formatMessage: jest.fn((msg) => msg.defaultMessage || msg.id || '') },
-      onEmailModeChange: jest.fn(),
-      showNextStep: jest.fn(),
-      onResetStep: jest.fn(),
-      onEnterTemplateName: jest.fn(),
-      onRemoveTemplateName: jest.fn(),
-      templatesActions: {
-        resetTemplateData: jest.fn(),
-        getDefaultBeeTemplates: jest.fn(),
-        handleZipUpload: jest.fn((file, onSuccess, onError) => {
-          setTimeout(() => onSuccess(), 0);
-        }),
-        handleHtmlUpload: jest.fn(),
-        setEdmTemplate: jest.fn(),
-        setBEETemplate: jest.fn(),
-      },
-      EmailLayout: null,
-      CmsTemplates: null,
-      SelectedEdmDefaultTemplate: null,
-      isUploading: false,
-      getCmsTemplatesInProgress: false,
-      step: STEPS.MODE_SELECTION,
-      emailCreateMode: '',
-      setIsLoadingContent: jest.fn(),
-      isGetFormData: false,
-      getFormdata: jest.fn(),
-      type: 'someType',
-      isFullMode: false,
-      cap: { currentOrgDetails: { id: 'org1' } },
-      showTemplateName: true,
-      showLiquidErrorInFooter: false,
-      onValidationFail: jest.fn(),
-      forwardedTags: [],
-      selectedOfferDetails: null,
-      onPreviewContentClicked: jest.fn(),
-      onTestContentClicked: jest.fn(),
-      editor: 'HTML',
-      moduleType: null,
-      eventContextTags: [],
-      isLoyaltyModule: false,
-      cmsTemplatesLoader: jest.fn(),
-      currentOrgDetails: {},
-      showTestAndPreviewSlidebox: false,
-      handleTestAndPreview: jest.fn(),
-      handleCloseTestAndPreview: jest.fn(),
-      isTestAndPreviewMode: false,
-      location: { query: {}, pathname: '/email/create' },
-      emailActions: {
-        getCmsAccounts: jest.fn(),
-        getTemplateDetails: jest.fn(),
-        getCmsSetting: jest.fn(),
-      },
-      Email: {
-        isBeeEnabled: undefined,
-        fetchingCmsAccountsError: null,
-        templateDetails: null,
-        BEETemplate: null,
-        getTemplateDetailsInProgress: false,
-        fetchingCmsData: false,
-        fetchingCmsSettings: false,
-      },
-      templateData: null,
-      params: {},
-    };
-  });
-
-  describe('File Upload Edge Cases', () => {
-    it('handles file upload when already uploading', () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: { ...mockProps, isUploading: true },
-      });
-
-      const file = { name: 'test.zip', size: 1024, originFileObj: new Blob() };
-      reactAct(() => {
-        result.current.useFileUpload({ file });
-      });
-
-      // Should not process upload when already uploading
-      expect(mockProps.templatesActions.handleZipUpload).not.toHaveBeenCalled();
-    });
-
-    it('handles file upload with missing file', () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: mockProps,
-      });
-
-      reactAct(() => {
-        result.current.useFileUpload({ file: null });
-      });
-
-      expect(mockCapNotificationError).toHaveBeenCalled();
-    });
-
-    it('handles file upload with missing file name', () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: mockProps,
-      });
-
-      const file = { size: 1024, originFileObj: new Blob() };
-      reactAct(() => {
-        result.current.useFileUpload({ file });
-      });
-
-      expect(mockCapNotificationError).toHaveBeenCalled();
-    });
-
-    it('handles file upload with file size exceeding limit', () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: mockProps,
-      });
-
-      const file = {
-        name: 'large.zip',
-        size: 6 * 1024 * 1024, // 6MB
-        originFileObj: new Blob(),
-      };
-      reactAct(() => {
-        result.current.useFileUpload({ file });
-      });
-
-      expect(mockCapNotificationError).toHaveBeenCalled();
-    });
-
-    it('handles zip upload with missing originFileObj', () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: mockProps,
-      });
-
-      const file = { name: 'test.zip', size: 1024 };
-      reactAct(() => {
-        result.current.useFileUpload({ file });
-      });
-
-      expect(mockCapNotificationError).toHaveBeenCalled();
-    });
-
-    it('handles HTML file upload with missing originFileObj', () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: mockProps,
-      });
-
-      const file = { name: 'test.html', size: 1024 };
-      reactAct(() => {
-        result.current.useFileUpload({ file });
-      });
-
-      expect(mockCapNotificationError).toHaveBeenCalled();
-    });
-
-    it('handles HTML file upload with FileReader error', async () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: mockProps,
-      });
-
-      const mockFile = new Blob(['<html></html>'], { type: 'text/html' });
-      mockFile.name = 'test.html';
-      const file = {
-        name: 'test.html',
-        size: 1024,
-        originFileObj: mockFile,
-      };
-
-      // Mock FileReader to trigger error
-      const originalFileReader = global.FileReader;
-      global.FileReader = jest.fn(() => ({
-        readAsText: jest.fn(function () {
-          setTimeout(() => {
-            this.onerror(new Error('Read error'));
-          }, 0);
-        }),
-        onerror: null,
-        onload: null,
-      }));
-
-      reactAct(() => {
-        result.current.useFileUpload({ file });
-      });
-
-      await waitFor(() => {
-        expect(mockCapNotificationError).toHaveBeenCalled();
-      });
-
-      global.FileReader = originalFileReader;
-    });
-
-    it('handles HTML file upload with empty content', async () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: mockProps,
-      });
-
-      const mockFile = new Blob([''], { type: 'text/html' });
-      mockFile.name = 'test.html';
-      const file = {
-        name: 'test.html',
-        size: 1024,
-        originFileObj: mockFile,
-      };
-
-      // Mock FileReader to return empty content
-      const originalFileReader = global.FileReader;
-      global.FileReader = jest.fn(() => ({
-        readAsText: jest.fn(function () {
-          setTimeout(() => {
-            this.result = '';
-            if (this.onload) this.onload();
-          }, 0);
-        }),
-        onerror: null,
-        onload: null,
-      }));
-
-      reactAct(() => {
-        result.current.useFileUpload({ file });
-      });
-
-      await waitFor(() => {
-        expect(mockCapNotificationError).toHaveBeenCalled();
-      });
-
-      global.FileReader = originalFileReader;
-    });
-
-    it('handles unsupported file format', () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: mockProps,
-      });
-
-      const file = {
-        name: 'test.txt',
-        size: 1024,
-        originFileObj: new Blob(),
-      };
-      reactAct(() => {
-        result.current.useFileUpload({ file });
-      });
-
-      expect(mockCapNotificationError).toHaveBeenCalled();
-    });
-
-    it('handles file with zero size', () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: mockProps,
-      });
-
-      const file = {
-        name: 'test.zip',
-        size: 0,
-        originFileObj: new Blob(),
-      };
-      reactAct(() => {
-        result.current.useFileUpload({ file });
-      });
-
-      // Should handle zero size file
-      expect(mockProps.templatesActions.handleZipUpload).toHaveBeenCalled();
-    });
-  });
-
-  describe('Template Name Edge Cases', () => {
-    it('handles template name with only whitespace', () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: mockProps,
-      });
-
-      // Clear any previous calls
-      mockProps.onEnterTemplateName.mockClear();
-      mockProps.onRemoveTemplateName.mockClear();
-
-      reactAct(() => {
-        result.current.onTemplateNameChange({ target: { value: '   ' } });
-      });
-
-      // When value is only whitespace, value?.trim() returns empty string
-      // So isEmptyTemplateName should be true
-      expect(result.current.isTemplateNameEmpty).toBe(true);
-      // However, the code checks `if (value && onEnterTemplateName)` first
-      // Since '   ' is truthy, onEnterTemplateName is called, not onRemoveTemplateName
-      // This is the actual behavior of the code
-      expect(mockProps.onEnterTemplateName).toHaveBeenCalledTimes(1);
-      expect(mockProps.onRemoveTemplateName).not.toHaveBeenCalled();
-    });
-
-    it('handles empty template name', () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: mockProps,
-      });
-
-      reactAct(() => {
-        result.current.onTemplateNameChange({ target: { value: '' } });
-      });
-
-      expect(mockProps.onRemoveTemplateName).toHaveBeenCalled();
-    });
-
-    it('handles null template name', () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: mockProps,
-      });
-
-      reactAct(() => {
-        result.current.onTemplateNameChange({ target: { value: null } });
-      });
-
-      expect(mockProps.onRemoveTemplateName).toHaveBeenCalled();
-    });
-  });
-
-  describe('Mode Selection Edge Cases', () => {
-    it('prevents DRAG_DROP selection when BEE is disabled', () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          Email: {
-            ...mockProps.Email,
-            isBeeEnabled: false,
-          },
-        },
-      });
-
-      reactAct(() => {
-        result.current.onChange({ target: { value: EMAIL_CREATE_MODES.DRAG_DROP } });
-      });
-
-      // Should not change mode
-      expect(mockProps.onEmailModeChange).not.toHaveBeenCalled();
-    });
-
-    it('allows DRAG_DROP selection when BEE is enabled', () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          Email: {
-            ...mockProps.Email,
-            isBeeEnabled: true,
-          },
-        },
-      });
-
-      reactAct(() => {
-        result.current.onChange({ target: { value: EMAIL_CREATE_MODES.DRAG_DROP } });
-      });
-
-      expect(mockProps.onEmailModeChange).toHaveBeenCalled();
-    });
-  });
-
-  describe('Cleanup Edge Cases', () => {
-    it('calls cleanup functions on unmount', () => {
-      const { unmount } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: mockProps,
-      });
-
-      unmount();
-
-      expect(mockProps.onResetStep).toHaveBeenCalled();
-      expect(mockProps.templatesActions.resetTemplateData).toHaveBeenCalled();
-    });
-  });
-
-  describe('BEE Template Setting Edge Cases', () => {
-    it('handles BEE template with drag_drop_id in active tab', () => {
-      const { result, rerender } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          params: { id: '123' },
-          Email: {
-            ...mockProps.Email,
-            templateDetails: {
-              _id: '123',
-              versions: {
-                base: {
-                  activeTab: 'en',
-                  en: {
-                    is_drag_drop: true,
-                    drag_drop_id: 'drag-drop-123',
-                  },
-                },
-              },
-            },
-          },
-        },
-      });
-
-      rerender({
-        ...mockProps,
-        params: { id: '123' },
-        Email: {
-          ...mockProps.Email,
-          templateDetails: {
-            _id: '123',
-            versions: {
-              base: {
-                activeTab: 'en',
-                en: {
-                  is_drag_drop: true,
-                  drag_drop_id: 'drag-drop-123',
-                },
-              },
-            },
-          },
-        },
-      });
-
-      expect(mockProps.templatesActions.setBEETemplate).toHaveBeenCalled();
-    });
-
-    it('handles BEE template with is_drag_drop at root level', () => {
-      const { rerender } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          params: { id: '123' },
-          Email: {
-            ...mockProps.Email,
-            templateDetails: {
-              _id: '123',
-              is_drag_drop: true,
-            },
-          },
-        },
-      });
-
-      rerender({
-        ...mockProps,
-        params: { id: '123' },
-        Email: {
-          ...mockProps.Email,
-          templateDetails: {
-            _id: '123',
-            is_drag_drop: true,
-          },
-        },
-      });
-
-      expect(mockProps.templatesActions.setBEETemplate).toHaveBeenCalled();
-    });
-
-    it('handles BEE template with is_drag_drop as number 1', () => {
-      const { rerender } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          params: { id: '123' },
-          Email: {
-            ...mockProps.Email,
-            templateDetails: {
-              _id: '123',
-              versions: {
-                base: {
-                  is_drag_drop: 1,
-                },
-              },
-            },
-          },
-        },
-      });
-
-      rerender({
-        ...mockProps,
-        params: { id: '123' },
-        Email: {
-          ...mockProps.Email,
-          templateDetails: {
-            _id: '123',
-            versions: {
-              base: {
-                is_drag_drop: 1,
-              },
-            },
-          },
-        },
-      });
-
-      expect(mockProps.templatesActions.setBEETemplate).toHaveBeenCalled();
-    });
-  });
-
-  describe('Step Handling Edge Cases', () => {
-    it('handles CREATE_TEMPLATE_CONTENT step with UPLOAD mode and EmailLayout', async () => {
-      // The effect at line 485-486 sets selectedCreateMode when:
-      // step === CREATE_TEMPLATE_CONTENT && emailCreateMode === UPLOAD && !isEmpty(EmailLayout)
-      // The isEmpty mock checks if object has keys, so { html: '<p>Content</p>' } is not empty
-      // Note: selectedCreateMode is not returned from the hook, so we can't test it directly
-      // This test verifies the effect runs without errors
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          step: STEPS.CREATE_TEMPLATE_CONTENT,
-          emailCreateMode: EMAIL_CREATE_MODES.UPLOAD,
-          EmailLayout: { html: '<p>Content</p>' },
-        },
-      });
-
-      // Wait for effects to run
-      await waitFor(() => {
-        // Verify hook returns expected values
-        expect(result.current).toBeDefined();
-      }, { timeout: 1000 });
-
-      // The effect should have run (we can't directly test selectedCreateMode as it's not returned)
-      // But we can verify the hook doesn't crash and returns expected structure
-      expect(result.current.modes).toBeDefined();
-      expect(result.current.onChange).toBeDefined();
-    });
-
-    it('handles template selection when template already selected', () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          step: STEPS.CREATE_TEMPLATE_CONTENT,
-          emailCreateMode: EMAIL_CREATE_MODES.EDITOR,
-          SelectedEdmDefaultTemplate: { _id: 'template-1' },
-        },
-      });
-
-      // Should handle already selected template
-    });
-  });
-
-  describe('Template Data Reset Edge Cases', () => {
-    it('resets selectedCreateMode when templateData is cleared in MODE_SELECTION', () => {
-      const { result, rerender } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          templateData: { _id: '123' },
-          step: STEPS.CREATE_TEMPLATE_CONTENT,
-        },
-      });
-
-      // Set a mode first
-      reactAct(() => {
-        result.current.onChange({ target: { value: EMAIL_CREATE_MODES.HTML_EDITOR } });
-      });
-
-      // Clear template data and go back to MODE_SELECTION
-      rerender({
-        ...mockProps,
-        templateData: null,
-        step: STEPS.MODE_SELECTION,
-        emailCreateMode: '',
-      });
-
-      // selectedCreateMode should be reset
-    });
-
-    it('does not reset selectedCreateMode in CREATE_TEMPLATE_CONTENT step', () => {
-      const { result, rerender } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          templateData: { _id: '123' },
-          step: STEPS.CREATE_TEMPLATE_CONTENT,
-        },
-      });
-
-      // Set a mode first
-      reactAct(() => {
-        result.current.onChange({ target: { value: EMAIL_CREATE_MODES.HTML_EDITOR } });
-      });
-
-      // Clear template data but stay in CREATE_TEMPLATE_CONTENT
-      rerender({
-        ...mockProps,
-        templateData: null,
-        step: STEPS.CREATE_TEMPLATE_CONTENT,
-      });
-
-      // selectedCreateMode should NOT be reset
-    });
-  });
-
-  describe('getStepFromTemplateStep Edge Cases (lines 130-131)', () => {
-    it('handles unknown numeric step value - returns MODE_SELECTION (default case)', () => {
-      // Test with numeric step value that doesn't match 1, 2, or 3
-      // This triggers the default case at lines 130-131
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          step: 99, // Unknown numeric step
-        },
-      });
-
-      // The hook should default to MODE_SELECTION behavior
-      expect(result.current.isShowEmailCreate).toBe(false);
-    });
-
-    it('handles step value of 0 - returns MODE_SELECTION (default case)', () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          step: 0,
-        },
-      });
-
-      expect(result.current.isShowEmailCreate).toBe(false);
-    });
-
-    it('handles negative step value - returns MODE_SELECTION (default case)', () => {
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          step: -1,
-        },
-      });
-
-      expect(result.current.isShowEmailCreate).toBe(false);
-    });
-  });
-
-  describe('beeTemplateSetRef reset Edge Cases (lines 288-290)', () => {
-    it('resets beeTemplateSetRef when template details are cleared', async () => {
-      // First render with template details to set beeTemplateSetRef
-      const { result, rerender } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          params: { id: '123' },
-          Email: {
-            ...mockProps.Email,
-            isBeeEnabled: true,
-            templateDetails: {
-              _id: '123',
-              versions: {
-                base: {
-                  activeTab: 'en',
-                  en: {
-                    is_drag_drop: true,
-                    drag_drop_id: 'drag-drop-123',
-                  },
-                },
-              },
-            },
-          },
-        },
-      });
-
-      // Wait for initial effects
-      await waitFor(() => {
-        expect(mockProps.templatesActions.setBEETemplate).toHaveBeenCalled();
-      }, { timeout: 2000 });
-
-      // Clear template details - this should trigger lines 288-290
-      rerender({
-        ...mockProps,
-        params: { id: '123' },
-        Email: {
-          ...mockProps.Email,
-          isBeeEnabled: true,
-          templateDetails: null, // Cleared
-          BEETemplate: null,
-        },
-      });
-
-      // The ref should be reset (we can't test ref directly, but we verify no errors)
-      expect(result.current).toBeDefined();
-    });
-
-    it('does not reset beeTemplateSetRef when template details exist', async () => {
-      const templateDetails = {
-        _id: '123',
-        versions: {
-          base: {
-            activeTab: 'en',
-            en: { is_drag_drop: true },
-          },
-        },
-      };
-
-      const { result, rerender } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          params: { id: '123' },
-          Email: {
-            ...mockProps.Email,
-            isBeeEnabled: true,
-            templateDetails,
-          },
-        },
-      });
-
-      // Rerender with same template details
-      rerender({
-        ...mockProps,
-        params: { id: '123' },
-        Email: {
-          ...mockProps.Email,
-          isBeeEnabled: true,
-          templateDetails,
-        },
-      });
-
-      // Should not cause errors
-      expect(result.current).toBeDefined();
-    });
-  });
-
-  describe('Legacy flow auto-navigate (lines 327-330)', () => {
-    it('calls showNextStep immediately when EDITOR mode is selected in legacy flow', () => {
-      // Enable legacy flow
-      const { hasSupportCKEditor } = require('../../../utils/common');
-      hasSupportCKEditor.mockReturnValue(true);
-
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: mockProps,
-      });
-
-      reactAct(() => {
-        result.current.onChange({ target: { value: EMAIL_CREATE_MODES.EDITOR } });
-      });
-
-      // In legacy flow, showNextStep should be called immediately
-      expect(mockProps.showNextStep).toHaveBeenCalled();
-      expect(mockProps.onEmailModeChange).toHaveBeenCalledWith(EMAIL_CREATE_MODES.EDITOR, EMAIL_CREATE_MODES.EDITOR);
-
-      // Reset mock
-      hasSupportCKEditor.mockReturnValue(false);
-    });
-
-    it('does NOT call showNextStep in new flow when EDITOR mode is selected', () => {
-      // New flow (hasSupportCKEditor returns false)
-      const { hasSupportCKEditor } = require('../../../utils/common');
-      hasSupportCKEditor.mockReturnValue(false);
-
-      const showNextStepMock = jest.fn();
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          showNextStep: showNextStepMock,
-        },
-      });
-
-      reactAct(() => {
-        result.current.onChange({ target: { value: EMAIL_CREATE_MODES.HTML_EDITOR } });
-      });
-
-      // In new flow, showNextStep should NOT be called for HTML_EDITOR
-      expect(showNextStepMock).not.toHaveBeenCalled();
-    });
-
-    it('does NOT call showNextStep in legacy flow when showNextStep is not provided', () => {
-      const { hasSupportCKEditor } = require('../../../utils/common');
-      hasSupportCKEditor.mockReturnValue(true);
-
-      const { result } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          showNextStep: null, // Not provided
-        },
-      });
-
-      // Should not throw error
-      reactAct(() => {
-        result.current.onChange({ target: { value: EMAIL_CREATE_MODES.EDITOR } });
-      });
-
-      // Verify onEmailModeChange is still called
-      expect(mockProps.onEmailModeChange).toHaveBeenCalled();
-
-      // Reset mock
-      hasSupportCKEditor.mockReturnValue(false);
-    });
-  });
-
-  describe('setSelectedCreateMode for HTML_EDITOR (line 501)', () => {
-    it('sets selectedCreateMode to HTML_EDITOR in CREATE_TEMPLATE_CONTENT step when conditions are met', async () => {
-      const { hasSupportCKEditor } = require('../../../utils/common');
-      hasSupportCKEditor.mockReturnValue(false); // New flow
-
-      const { result, rerender } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          step: STEPS.MODE_SELECTION,
-          emailCreateMode: '',
-        },
-      });
-
-      // First select HTML_EDITOR mode
-      reactAct(() => {
-        result.current.onChange({ target: { value: EMAIL_CREATE_MODES.HTML_EDITOR } });
-      });
-
-      // Move to CREATE_TEMPLATE_CONTENT step
-      rerender({
-        ...mockProps,
-        step: STEPS.CREATE_TEMPLATE_CONTENT,
-        emailCreateMode: EMAIL_CREATE_MODES.EDITOR, // Mapped value
-      });
-
-      // Wait for effects
-      await waitFor(() => {
-        // The emailProps should reflect HTML editor mode
-        const emailProps = result.current.emailProps;
-        expect(emailProps.editor).toBe('HTML');
-        expect(emailProps.selectedEditorMode).toBe(EMAIL_CREATE_MODES.HTML_EDITOR);
-      }, { timeout: 2000 });
-    });
-  });
-
-  describe('setSelectedCreateMode for DRAG_DROP (line 508)', () => {
-    it('sets selectedCreateMode to DRAG_DROP when template is already selected', async () => {
-      const { hasSupportCKEditor } = require('../../../utils/common');
-      hasSupportCKEditor.mockReturnValue(false); // New flow - supportCKEditor will be false
-
-      const { result, rerender } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          step: STEPS.MODE_SELECTION,
-          emailCreateMode: '',
-          Email: {
-            ...mockProps.Email,
-            isBeeEnabled: true,
-          },
-        },
-      });
-
-      // First select DRAG_DROP mode - this sets selectedCreateMode internally
-      reactAct(() => {
-        result.current.onChange({ target: { value: EMAIL_CREATE_MODES.DRAG_DROP } });
-      });
-
-      // Verify DRAG_DROP selection was made
-      // Note: onEmailModeChange is called with (mappedValue, originalValue)
-      // For DRAG_DROP, mappedValue is 'editor' and originalValue is 'drag_drop'
-      expect(mockProps.onEmailModeChange).toHaveBeenCalledWith(
-        EMAIL_CREATE_MODES.EDITOR,
-        EMAIL_CREATE_MODES.DRAG_DROP
-      );
-
-      // Move to CREATE_TEMPLATE_CONTENT step with template already selected
-      // This should trigger line 508: setSelectedCreateMode(EMAIL_CREATE_MODES.DRAG_DROP)
-      rerender({
-        ...mockProps,
-        step: STEPS.CREATE_TEMPLATE_CONTENT,
-        emailCreateMode: EMAIL_CREATE_MODES.DRAG_DROP, // Use DRAG_DROP to enter the correct branch
-        SelectedEdmDefaultTemplate: { _id: 'template-123' }, // Template already selected
-        Email: {
-          ...mockProps.Email,
-          isBeeEnabled: true,
-        },
-      });
-
-      // Wait for effects - the component renders correctly when DRAG_DROP path is triggered
-      await waitFor(() => {
-        // The component should be in a valid state when line 508 is executed
-        expect(result.current.modes).toBeDefined();
-      }, { timeout: 2000 });
-    });
-
-    it('ensures DRAG_DROP is set when in new flow with DRAG_DROP selection', async () => {
-      const { hasSupportCKEditor } = require('../../../utils/common');
-      hasSupportCKEditor.mockReturnValue(false); // New flow
-
-      const { result, rerender } = renderHook((props) => useEmailWrapper(props), {
-        initialProps: {
-          ...mockProps,
-          step: STEPS.MODE_SELECTION,
-          emailCreateMode: '',
-          Email: {
-            ...mockProps.Email,
-            isBeeEnabled: true,
-          },
-        },
-      });
-
-      // Select DRAG_DROP mode
-      reactAct(() => {
-        result.current.onChange({ target: { value: EMAIL_CREATE_MODES.DRAG_DROP } });
-      });
-
-      // Move to template selection step
-      rerender({
-        ...mockProps,
-        step: STEPS.TEMPLATE_SELECTION,
-        emailCreateMode: EMAIL_CREATE_MODES.EDITOR, // Mapped value
-        Email: {
-          ...mockProps.Email,
-          isBeeEnabled: true,
-        },
-      });
-
-      // Verify the internal state maintains DRAG_DROP
-      expect(result.current.modes).toBeDefined();
-    });
-  });
-});