feat: Unenroll survey is configurable through environment variable

Port of master PR #738 to frontend-base.

Differences from master:
- Config added to src/app.ts instead of .env files and src/config/index.js
  (frontend-base convention)
- Uses useAppConfig() hook instead of importing configuration object
  directly (idiomatic for code running inside the provider tree)
- Default is false (master defaults to true)

Co-Authored-By: Adolfo R. Brandes <adolfo@opencraft.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: 3 people

src/app.ts

@@ -17,6 +17,7 @@ const app: App = {
     ECOMMERCE_BASE_URL: '',
     ORDER_HISTORY_URL: '',
     SUPPORT_URL: '',
+    SHOW_UNENROLL_SURVEY: false,
   }
 };
 

src/containers/UnenrollConfirmModal/hooks/index.js

@@ -1,6 +1,9 @@
 import React from 'react';
 
+import { useAppConfig } from '@openedx/frontend-base';
 import { StrictDict } from '@src/utils';
+import { useCourseData } from '@src/hooks';
+import { useUnenrollFromCourse } from '@src/data/hooks';
 
 import { useUnenrollReasons } from './reasons';
 import * as module from '.';
@@ -17,12 +20,23 @@ export const modalStates = StrictDict({
 
 export const useUnenrollData = ({ closeModal, cardId }) => {
   const [isConfirmed, setIsConfirmed] = module.state.confirmed(false);
-  const confirm = () => setIsConfirmed(true);
   const reason = useUnenrollReasons({ cardId });
+  const appConfig = useAppConfig();
+  const courseData = useCourseData(cardId);
+  const courseId = courseData?.courseRun?.courseId;
+
+  const { mutate: unenrollFromCourse } = useUnenrollFromCourse();
+
+  const confirm = () => {
+    if (!appConfig.SHOW_UNENROLL_SURVEY) {
+      unenrollFromCourse({ courseId });
+    }
+    setIsConfirmed(true);
+  };
 
   let modalState;
   if (isConfirmed) {
-    modalState = (reason.isSubmitted)
+    modalState = (reason.isSubmitted || !appConfig.SHOW_UNENROLL_SURVEY)
       ? modalStates.finished : modalStates.reason;
   } else {
     modalState = modalStates.confirm;

src/containers/UnenrollConfirmModal/hooks/index.test.js

@@ -1,4 +1,8 @@
 import { MockUseState } from '@src/testUtils';
+import { useAppConfig } from '@openedx/frontend-base';
+
+import { useCourseData } from '@src/hooks';
+import { useUnenrollFromCourse } from '@src/data/hooks';
 
 import * as reasons from './reasons';
 import * as hooks from '.';
@@ -7,8 +11,23 @@ jest.mock('./reasons', () => ({
   useUnenrollReasons: jest.fn(),
 }));
 
+jest.mock('@src/data/hooks', () => ({
+  useUnenrollFromCourse: jest.fn(),
+}));
+
+jest.mock('@src/hooks', () => ({
+  useCourseData: jest.fn(),
+}));
+
+jest.mock('@openedx/frontend-base', () => ({
+  useAppConfig: jest.fn(),
+}));
+
 const state = new MockUseState(hooks);
 const testValue = 'test-value';
+const unenrollFromCourse = jest.fn();
+useUnenrollFromCourse.mockReturnValue({ mutate: unenrollFromCourse });
+useCourseData.mockReturnValue({ courseRun: { courseId: 'test-course-id' } });
 let out;
 
 const mockReason = {
@@ -22,6 +41,7 @@ const useUnenrollReasons = jest.fn(() => mockReason);
 describe('UnenrollConfirmModal hooks', () => {
   beforeEach(() => {
     reasons.useUnenrollReasons.mockImplementation(useUnenrollReasons);
+    useAppConfig.mockReturnValue({ SHOW_UNENROLL_SURVEY: true });
   });
   const closeModal = jest.fn();
   const cardId = 'test-card-id';
@@ -66,22 +86,67 @@ describe('UnenrollConfirmModal hooks', () => {
         expect(mockReason.handleClear).toHaveBeenCalled();
       });
     });
-    describe('modalState', () => {
-      it('returns modalStates.finished if confirmed and submitted', () => {
+  });
+
+  describe('SHOW_UNENROLL_SURVEY configuration tests', () => {
+    beforeEach(() => {
+      state.mock();
+      jest.clearAllMocks();
+    });
+    afterEach(() => {
+      state.restore();
+    });
+
+    describe('when SHOW_UNENROLL_SURVEY is true (default)', () => {
+      beforeEach(() => {
+        useAppConfig.mockReturnValue({ SHOW_UNENROLL_SURVEY: true });
+        useCourseData.mockReturnValue({ courseRun: { courseId: 'test-course-id' } });
+        useUnenrollFromCourse.mockReturnValue({ mutate: unenrollFromCourse });
+        reasons.useUnenrollReasons.mockImplementation(useUnenrollReasons);
+      });
+
+      test('confirm does not call unenrollFromCourse immediately', () => {
+        out = createUseUnenrollData();
+        out.confirm();
+        expect(unenrollFromCourse).not.toHaveBeenCalled();
+        expect(state.setState.confirmed).toHaveBeenCalledWith(true);
+      });
+
+      test('modalState returns reason when confirmed but not submitted', () => {
+        state.mockVal(state.keys.confirmed, true);
+        reasons.useUnenrollReasons.mockReturnValueOnce({ ...mockReason, isSubmitted: false });
+        out = createUseUnenrollData();
+        expect(out.modalState).toEqual(hooks.modalStates.reason);
+      });
+
+      test('modalState returns finished when confirmed and submitted', () => {
         state.mockVal(state.keys.confirmed, true);
         reasons.useUnenrollReasons.mockReturnValueOnce({ ...mockReason, isSubmitted: true });
         out = createUseUnenrollData();
         expect(out.modalState).toEqual(hooks.modalStates.finished);
       });
-      it('returns modalStates.reason if confirmed and not submitted', () => {
-        state.mockVal(state.keys.confirmed, true);
+    });
+
+    describe('when SHOW_UNENROLL_SURVEY is false', () => {
+      beforeEach(() => {
+        useAppConfig.mockReturnValue({ SHOW_UNENROLL_SURVEY: false });
+        useCourseData.mockReturnValue({ courseRun: { courseId: 'test-course-id' } });
+        useUnenrollFromCourse.mockReturnValue({ mutate: unenrollFromCourse });
+        reasons.useUnenrollReasons.mockImplementation(useUnenrollReasons);
+      });
+
+      test('confirm calls unenrollFromCourse immediately', () => {
         out = createUseUnenrollData();
-        expect(out.modalState).toEqual(hooks.modalStates.reason);
+        out.confirm();
+        expect(unenrollFromCourse).toHaveBeenCalled();
+        expect(state.setState.confirmed).toHaveBeenCalledWith(true);
       });
-      it('returns modalStates.confirm if not confirmed', () => {
-        state.mockVal(state.keys.confirmed, false);
+
+      test('modalState returns finished when confirmed regardless of submission status', () => {
+        state.mockVal(state.keys.confirmed, true);
+        reasons.useUnenrollReasons.mockReturnValueOnce({ ...mockReason, isSubmitted: false });
         out = createUseUnenrollData();
-        expect(out.modalState).toEqual(hooks.modalStates.confirm);
+        expect(out.modalState).toEqual(hooks.modalStates.finished);
       });
     });
   });