chore: fix circleci on vite build (#10214)

- Switch to pnpm based build
- Switch circleci from docker to machine to have more memory
- Fix frontend and backend tests

Fixes
https://linear.app/chatwoot/issue/CW-3610/fix-circle-ci-for-vite-build
---------

Co-authored-by: Shivam Mishra <scm.mymail@gmail.com>
Co-authored-by: Pranav <pranavrajs@gmail.com>
Co-authored-by: Pranav <pranav@chatwoot.com>

app/javascript/dashboard/composables/index.js

@@ -1,5 +1,5 @@
 import { emitter } from 'shared/helpers/mitt';
-import analyticsHelper from '/dashboard/helper/AnalyticsHelper/index';
+import analyticsHelper from 'dashboard/helper/AnalyticsHelper/index';
 
 /**
  * Custom hook to track events

app/javascript/dashboard/composables/spec/emitter.spec.js

@@ -1,6 +1,7 @@
 import { shallowMount } from '@vue/test-utils';
 import { emitter } from 'shared/helpers/mitt';
 import { useEmitter } from '../emitter';
+import { defineComponent } from 'vue';
 
 vi.mock('shared/helpers/mitt', () => ({
   emitter: {
@@ -10,31 +11,34 @@ vi.mock('shared/helpers/mitt', () => ({
 }));
 
 describe('useEmitter', () => {
-  let wrapper;
   const eventName = 'my-event';
   const callback = vi.fn();
 
+  let wrapper;
+
+  const TestComponent = defineComponent({
+    setup() {
+      return {
+        cleanup: useEmitter(eventName, callback),
+      };
+    },
+    template: '<div>Hello world</div>',
+  });
+
   beforeEach(() => {
-    wrapper = shallowMount({
-      template: `
-        <div>
-          Hello world
-        </div>
-      `,
-      setup() {
-        return {
-          cleanup: useEmitter(eventName, callback),
-        };
-      },
-    });
+    wrapper = shallowMount(TestComponent);
+  });
+
+  afterEach(() => {
+    vi.clearAllMocks();
   });
 
   it('should add an event listener on mount', () => {
     expect(emitter.on).toHaveBeenCalledWith(eventName, callback);
   });
 
-  it('should remove the event listener when the component is unmounted', () => {
-    wrapper.destroy();
+  it('should remove the event listener when the component is unmounted', async () => {
+    await wrapper.unmount();
     expect(emitter.off).toHaveBeenCalledWith(eventName, callback);
   });
 

app/javascript/dashboard/composables/spec/index.spec.js

@@ -1,20 +1,26 @@
-import { getCurrentInstance } from 'vue';
 import { emitter } from 'shared/helpers/mitt';
+import analyticsHelper from 'dashboard/helper/AnalyticsHelper';
 import { useTrack, useAlert } from '../index';
 
-vi.mock('vue', () => ({
-  getCurrentInstance: vi.fn(),
-}));
 vi.mock('shared/helpers/mitt', () => ({
   emitter: {
     emit: vi.fn(),
   },
 }));
 
+vi.mock('dashboard/helper/AnalyticsHelper/index', async importOriginal => {
+  const actual = await importOriginal();
+  actual.default = {
+    track: vi.fn(),
+  };
+  return actual;
+});
+
 describe('useTrack', () => {
-  it('should return a function', () => {
-    const track = useTrack();
-    expect(typeof track).toBe('function');
+  it('should call analyticsHelper.track and return a function', () => {
+    const eventArgs = ['event-name', { some: 'data' }];
+    useTrack(...eventArgs);
+    expect(analyticsHelper.track).toHaveBeenCalledWith(...eventArgs);
   });
 });
 

app/javascript/dashboard/composables/spec/useAI.spec.js

@@ -4,14 +4,20 @@ import {
   useStoreGetters,
   useMapGetter,
 } from 'dashboard/composables/store';
-import { useAlert, useTrack } from 'dashboard/composables';
 import { useI18n } from 'vue-i18n';
 import OpenAPI from 'dashboard/api/integrations/openapi';
+import analyticsHelper from 'dashboard/helper/AnalyticsHelper/index';
 
 vi.mock('dashboard/composables/store');
-vi.mock('dashboard/composables');
 vi.mock('vue-i18n');
 vi.mock('dashboard/api/integrations/openapi');
+vi.mock('dashboard/helper/AnalyticsHelper/index', async importOriginal => {
+  const actual = await importOriginal();
+  actual.default = {
+    track: vi.fn(),
+  };
+  return actual;
+});
 vi.mock('dashboard/helper/AnalyticsHelper/events', () => ({
   OPEN_AI_EVENTS: {
     TEST_EVENT: 'open_ai_test_event',
@@ -40,9 +46,7 @@ describe('useAI', () => {
       };
       return { value: mockValues[getter] };
     });
-    useTrack.mockReturnValue(vi.fn());
     useI18n.mockReturnValue({ t: vi.fn() });
-    useAlert.mockReturnValue(vi.fn());
   });
 
   it('initializes computed properties correctly', async () => {
@@ -78,13 +82,12 @@ describe('useAI', () => {
   });
 
   it('records analytics correctly', async () => {
-    const mockTrack = vi.fn();
-    useTrack.mockReturnValue(mockTrack);
+    // const mockTrack = analyticsHelper.track;
     const { recordAnalytics } = useAI();
 
     await recordAnalytics('TEST_EVENT', { data: 'test' });
 
-    expect(mockTrack).toHaveBeenCalledWith('open_ai_test_event', {
+    expect(analyticsHelper.track).toHaveBeenCalledWith('open_ai_test_event', {
       type: 'TEST_EVENT',
       data: 'test',
     });

app/javascript/dashboard/composables/spec/useAutomation.spec.js

@@ -1,7 +1,7 @@
 import { useAutomation } from '../useAutomation';
 import { useStoreGetters, useMapGetter } from 'dashboard/composables/store';
 import { useAlert } from 'dashboard/composables';
-import { useI18n } from '../useI18n';
+import { useI18n } from 'vue-i18n';
 import * as automationHelper from 'dashboard/helper/automationHelper';
 import {
   customAttributes,
@@ -20,7 +20,7 @@ import { MESSAGE_CONDITION_VALUES } from 'dashboard/constants/automation';
 
 vi.mock('dashboard/composables/store');
 vi.mock('dashboard/composables');
-vi.mock('../useI18n');
+vi.mock('vue-i18n');
 vi.mock('dashboard/helper/automationHelper');
 
 describe('useAutomation', () => {
@@ -120,8 +120,8 @@ describe('useAutomation', () => {
   });
 
   it('appends new condition and action correctly', () => {
-    const { appendNewCondition, appendNewAction } = useAutomation();
-    const mockAutomation = {
+    const { appendNewCondition, appendNewAction, automation } = useAutomation();
+    automation.value = {
       event_name: 'message_created',
       conditions: [],
       actions: [],
@@ -130,36 +130,37 @@ describe('useAutomation', () => {
     automationHelper.getDefaultConditions.mockReturnValue([{}]);
     automationHelper.getDefaultActions.mockReturnValue([{}]);
 
-    appendNewCondition(mockAutomation);
-    appendNewAction(mockAutomation);
+    appendNewCondition();
+    appendNewAction();
 
     expect(automationHelper.getDefaultConditions).toHaveBeenCalledWith(
       'message_created'
     );
     expect(automationHelper.getDefaultActions).toHaveBeenCalled();
-    expect(mockAutomation.conditions).toHaveLength(1);
-    expect(mockAutomation.actions).toHaveLength(1);
+    expect(automation.value.conditions).toHaveLength(1);
+    expect(automation.value.actions).toHaveLength(1);
   });
 
   it('removes filter and action correctly', () => {
-    const { removeFilter, removeAction } = useAutomation();
-    const mockAutomation = {
+    const { removeFilter, removeAction, automation } = useAutomation();
+    automation.value = {
       conditions: [{ id: 1 }, { id: 2 }],
       actions: [{ id: 1 }, { id: 2 }],
     };
 
-    removeFilter(mockAutomation, 0);
-    removeAction(mockAutomation, 0);
+    removeFilter(0);
+    removeAction(0);
 
-    expect(mockAutomation.conditions).toHaveLength(1);
-    expect(mockAutomation.actions).toHaveLength(1);
-    expect(mockAutomation.conditions[0].id).toBe(2);
-    expect(mockAutomation.actions[0].id).toBe(2);
+    expect(automation.value.conditions).toHaveLength(1);
+    expect(automation.value.actions).toHaveLength(1);
+    expect(automation.value.conditions[0].id).toBe(2);
+    expect(automation.value.actions[0].id).toBe(2);
   });
 
   it('resets filter and action correctly', () => {
-    const { resetFilter, resetAction } = useAutomation();
-    const mockAutomation = {
+    const { resetFilter, resetAction, automation, automationTypes } =
+      useAutomation();
+    automation.value = {
       event_name: 'message_created',
       conditions: [
         {
@@ -170,77 +171,37 @@ describe('useAutomation', () => {
       ],
       actions: [{ action_name: 'assign_agent', action_params: [1] }],
     };
-    const mockAutomationTypes = {
-      message_created: {
-        conditions: [
-          { key: 'status', filterOperators: [{ value: 'not_equal_to' }] },
-        ],
-      },
+    automationTypes.message_created = {
+      conditions: [
+        { key: 'status', filterOperators: [{ value: 'not_equal_to' }] },
+      ],
     };
 
-    resetFilter(
-      mockAutomation,
-      mockAutomationTypes,
-      0,
-      mockAutomation.conditions[0]
-    );
-    resetAction(mockAutomation, 0);
+    resetFilter(0, automation.value.conditions[0]);
+    resetAction(0);
 
-    expect(mockAutomation.conditions[0].filter_operator).toBe('not_equal_to');
-    expect(mockAutomation.conditions[0].values).toBe('');
-    expect(mockAutomation.actions[0].action_params).toEqual([]);
-  });
-
-  it('formats automation correctly', () => {
-    const { formatAutomation } = useAutomation();
-    const mockAutomation = {
-      conditions: [{ attribute_key: 'status', values: ['open'] }],
-      actions: [{ action_name: 'assign_agent', action_params: [1] }],
-    };
-    const mockAutomationTypes = {};
-    const mockAutomationActionTypes = [
-      { key: 'assign_agent', inputType: 'search_select' },
-    ];
-
-    automationHelper.getConditionOptions.mockReturnValue([
-      { id: 'open', name: 'open' },
-    ]);
-    automationHelper.getActionOptions.mockReturnValue([
-      { id: 1, name: 'Agent 1' },
-    ]);
-
-    const result = formatAutomation(
-      mockAutomation,
-      customAttributes,
-      mockAutomationTypes,
-      mockAutomationActionTypes
-    );
-
-    expect(result.conditions[0].values).toEqual([{ id: 'open', name: 'open' }]);
-    expect(result.actions[0].action_params).toEqual([
-      { id: 1, name: 'Agent 1' },
-    ]);
+    expect(automation.value.conditions[0].filter_operator).toBe('not_equal_to');
+    expect(automation.value.conditions[0].values).toBe('');
+    expect(automation.value.actions[0].action_params).toEqual([]);
   });
 
   it('manifests custom attributes correctly', () => {
-    const { manifestCustomAttributes } = useAutomation();
-    const mockAutomationTypes = {
-      message_created: { conditions: [] },
-      conversation_created: { conditions: [] },
-      conversation_updated: { conditions: [] },
-      conversation_opened: { conditions: [] },
-    };
+    const { manifestCustomAttributes, automationTypes } = useAutomation();
+    automationTypes.message_created = { conditions: [] };
+    automationTypes.conversation_created = { conditions: [] };
+    automationTypes.conversation_updated = { conditions: [] };
+    automationTypes.conversation_opened = { conditions: [] };
 
     automationHelper.generateCustomAttributeTypes.mockReturnValue([]);
     automationHelper.generateCustomAttributes.mockReturnValue([]);
 
-    manifestCustomAttributes(mockAutomationTypes);
+    manifestCustomAttributes();
 
     expect(automationHelper.generateCustomAttributeTypes).toHaveBeenCalledTimes(
       2
     );
     expect(automationHelper.generateCustomAttributes).toHaveBeenCalledTimes(1);
-    Object.values(mockAutomationTypes).forEach(type => {
+    Object.values(automationTypes).forEach(type => {
       expect(type.conditions).toHaveLength(0);
     });
   });
@@ -273,8 +234,8 @@ describe('useAutomation', () => {
   });
 
   it('handles event change correctly', () => {
-    const { onEventChange } = useAutomation();
-    const mockAutomation = {
+    const { onEventChange, automation } = useAutomation();
+    automation.value = {
       event_name: 'message_created',
       conditions: [],
       actions: [],
@@ -283,13 +244,13 @@ describe('useAutomation', () => {
     automationHelper.getDefaultConditions.mockReturnValue([{}]);
     automationHelper.getDefaultActions.mockReturnValue([{}]);
 
-    onEventChange(mockAutomation);
+    onEventChange();
 
     expect(automationHelper.getDefaultConditions).toHaveBeenCalledWith(
       'message_created'
     );
     expect(automationHelper.getDefaultActions).toHaveBeenCalled();
-    expect(mockAutomation.conditions).toHaveLength(1);
-    expect(mockAutomation.actions).toHaveLength(1);
+    expect(automation.value.conditions).toHaveLength(1);
+    expect(automation.value.actions).toHaveLength(1);
   });
 });