fix: apply dprint formatting and fix i18n template literal in messages.ts

- Run dprint fmt on all new advanced-settings files to comply with the
  project's code formatting configuration
- Fix i18n extraction error in src/generic/create-or-rerun-course/messages.ts:
  replace template literal defaultMessage with FormatJS {maxLength} placeholder;
  update hooks.jsx to pass maxLength value at runtime; update test assertion

Author: SantiagoSuHe

src/advanced-settings/AdvancedSettings.test.tsx

@@ -16,14 +16,15 @@ import filterMessages from './settings-filters/messages';
 
 // Mock CodeMirror-based JsonInput with a controlled textarea so tests can
 // interact with JSON fields without a real browser layout engine.
-jest.mock('./setting-card/inputs/JsonInput', () => jest.fn(({ initialValue, onChange, onBlur }) => (
-  <textarea
-    data-testid="json-input"
-    value={initialValue}
-    onChange={(e) => onChange(e.target.value)}
-    onBlur={onBlur}
-  />
-)));
+jest.mock('./setting-card/inputs/JsonInput', () =>
+  jest.fn(({ initialValue, onChange, onBlur }) => (
+    <textarea
+      data-testid="json-input"
+      value={initialValue}
+      onChange={(e) => onChange(e.target.value)}
+      onBlur={onBlur}
+    />
+  )));
 
 jest.mock('@src/authz/data/apiHooks', () => ({
   useUserPermissions: jest.fn(),
@@ -65,9 +66,11 @@ describe('<AdvancedSettings />', () => {
 
   it('should render without errors', async () => {
     render();
-    expect(await screen.findByText(messages.headingSubtitle.defaultMessage, {
-      selector: 'small.sub-header-title-subtitle',
-    })).toBeInTheDocument();
+    expect(
+      await screen.findByText(messages.headingSubtitle.defaultMessage, {
+        selector: 'small.sub-header-title-subtitle',
+      }),
+    ).toBeInTheDocument();
     expect(screen.getByText(messages.headingTitle.defaultMessage, {
       selector: 'h2.sub-header-title',
     })).toBeInTheDocument();
@@ -177,9 +180,11 @@ describe('<AdvancedSettings />', () => {
       data: { canManageAdvancedSettings: true },
     } as unknown as ReturnType<typeof useUserPermissions>);
     render();
-    expect(await screen.findByText(messages.headingSubtitle.defaultMessage, {
-      selector: 'small.sub-header-title-subtitle',
-    })).toBeInTheDocument();
+    expect(
+      await screen.findByText(messages.headingSubtitle.defaultMessage, {
+        selector: 'small.sub-header-title-subtitle',
+      }),
+    ).toBeInTheDocument();
     expect(screen.getByText(messages.headingTitle.defaultMessage, {
       selector: 'h2.sub-header-title',
     })).toBeInTheDocument();

src/advanced-settings/AdvancedSettings.tsx

@@ -282,7 +282,9 @@ const AdvancedSettings = () => {
                           forceOpen={filterText.trim() !== ''}
                           expandAll={expandAll}
                           subcategoryMap={category === CATEGORY_CONTENT_BLOCKS ? CONTENT_BLOCKS_SUBCATEGORY_MAP : null}
-                          subcategoryOrder={category === CATEGORY_CONTENT_BLOCKS ? CONTENT_BLOCKS_SUBCATEGORY_ORDER : []}
+                          subcategoryOrder={category === CATEGORY_CONTENT_BLOCKS
+                            ? CONTENT_BLOCKS_SUBCATEGORY_ORDER
+                            : []}
                         />
                       ),
                     )}

src/advanced-settings/data/fieldTypeMessages.ts

@@ -31,12 +31,14 @@ const enumOptionMessages = defineMessages({
   finished: {
     id: 'course-authoring.advanced-settings.enum.finished',
     defaultMessage: 'Finished',
-    description: 'Option label for Show Answer: show the answer after the learner has used all attempts or the problem is past due',
+    description:
+      'Option label for Show Answer: show the answer after the learner has used all attempts or the problem is past due',
   },
   correctOrPastDue: {
     id: 'course-authoring.advanced-settings.enum.correct-or-past-due',
     defaultMessage: 'Correct or past due',
-    description: 'Option label for Show Answer: show the answer once the learner answers correctly or the due date passes',
+    description:
+      'Option label for Show Answer: show the answer once the learner answers correctly or the due date passes',
   },
   pastDue: {
     id: 'course-authoring.advanced-settings.enum.past-due',
@@ -61,7 +63,8 @@ const enumOptionMessages = defineMessages({
   afterAllAttemptsOrCorrect: {
     id: 'course-authoring.advanced-settings.enum.after-all-attempts-or-correct',
     defaultMessage: 'After all attempts or correct',
-    description: 'Option label for Show Answer: show the answer after all attempts are used or the learner answers correctly',
+    description:
+      'Option label for Show Answer: show the answer after all attempts are used or the learner answers correctly',
   },
   attemptedNoPastDue: {
     id: 'course-authoring.advanced-settings.enum.attempted-no-past-due',
@@ -141,12 +144,14 @@ const enumOptionMessages = defineMessages({
   earlyNoInfo: {
     id: 'course-authoring.advanced-settings.enum.early-no-info',
     defaultMessage: 'Early (no info)',
-    description: 'Option label for Certificates Display Behavior: certificates are displayed early with no additional information',
+    description:
+      'Option label for Certificates Display Behavior: certificates are displayed early with no additional information',
   },
   earlyWithInfo: {
     id: 'course-authoring.advanced-settings.enum.early-with-info',
     defaultMessage: 'Early (with info)',
-    description: 'Option label for Certificates Display Behavior: certificates are displayed early with additional information shown to learners',
+    description:
+      'Option label for Certificates Display Behavior: certificates are displayed early with additional information shown to learners',
   },
 });
 
@@ -155,7 +160,7 @@ const enumOptionMessages = defineMessages({
  * Used by EnumInput to render localized option labels.
  * Keys match the `value` field of each option in ENUM_OPTIONS (fieldTypes.ts).
  */
-export const ENUM_LABEL_MESSAGES: Record<string, Record<string, { id: string; defaultMessage: string }>> = {
+export const ENUM_LABEL_MESSAGES: Record<string, Record<string, { id: string; defaultMessage: string; }>> = {
   showanswer: {
     always: enumOptionMessages.always,
     answered: enumOptionMessages.answered,
@@ -349,7 +354,8 @@ export const FIELD_PLACEHOLDER_MESSAGES = defineMessages({
   topLevelDownstreamParentKey: {
     id: 'course-authoring.advanced-settings.placeholder.top-level-downstream-parent-key',
     defaultMessage: 'e.g. block-v1:Org+Course+Run+type@vertical+block@id',
-    description: 'Placeholder for the Top-Level Downstream Parent Key field in Advanced Settings (Open edX block key format)',
+    description:
+      'Placeholder for the Top-Level Downstream Parent Key field in Advanced Settings (Open edX block key format)',
   },
   certNameLong: {
     id: 'course-authoring.advanced-settings.placeholder.cert-name-long',

src/advanced-settings/data/fieldTypes.test.ts

@@ -77,8 +77,14 @@ describe('serializeValue()', () => {
 describe('ENUM_OPTIONS', () => {
   it('contains options for all 8 enum fields', () => {
     const expectedFields = [
-      'showanswer', 'rerandomize', 'catalogVisibility', 'courseVisibility',
-      'showCorrectness', 'videoSharingOptions', 'courseEditMethod', 'certificatesDisplayBehavior',
+      'showanswer',
+      'rerandomize',
+      'catalogVisibility',
+      'courseVisibility',
+      'showCorrectness',
+      'videoSharingOptions',
+      'courseEditMethod',
+      'certificatesDisplayBehavior',
     ];
     expectedFields.forEach((field) => {
       expect(ENUM_OPTIONS[field]).toBeDefined();

src/advanced-settings/data/fieldTypes.ts

@@ -113,20 +113,47 @@ export const ENUM_OPTIONS: Record<string, EnumOption[]> = {
  * Used when the runtime value could be null/empty but the field has a known type.
  */
 const NUMBER_KEYS = new Set([
-  'maxAttempts', 'maxStudentEnrollmentsAllowed', 'entranceExamMinimumScorePct',
-  'daysEarlyForBeta', 'relativeWeeksDue', 'minimumGradeCredit',
-  'upstreamVersion', 'upstreamVersionDeclined',
+  'maxAttempts',
+  'maxStudentEnrollmentsAllowed',
+  'entranceExamMinimumScorePct',
+  'daysEarlyForBeta',
+  'relativeWeeksDue',
+  'minimumGradeCredit',
+  'upstreamVersion',
+  'upstreamVersionDeclined',
 ]);
 
 const BOOLEAN_KEYS = new Set([
-  'certHtmlViewEnabled', 'selfPaced', 'enableTimedExams', 'enableProctoredExams',
-  'showResetButton', 'mobileAvailable', 'invitationOnly', 'isNew', 'noGrade',
-  'highlightsEnabledForMessaging', 'allowUnsupportedXblocks', 'allowProctoringOptOut',
-  'createZendeskTickets', 'enableSubsectionGating', 'entranceExamEnabled',
-  'allowPublicWikiAccess', 'disableProgressGraph', 'showCalculator', 'hideProgressTab',
-  'edxnotes', 'videoAutoAdvance', 'videoSpeedOptimizations', 'useLatexCompiler',
-  'courseSurveyRequired', 'allowAnonymous', 'allowAnonymousToPeers', 'enableCcx',
-  'forceOnFlexiblePeerOpenassessments', 'downstreamCustomized', 'certificatesShowBeforeEnd',
+  'certHtmlViewEnabled',
+  'selfPaced',
+  'enableTimedExams',
+  'enableProctoredExams',
+  'showResetButton',
+  'mobileAvailable',
+  'invitationOnly',
+  'isNew',
+  'noGrade',
+  'highlightsEnabledForMessaging',
+  'allowUnsupportedXblocks',
+  'allowProctoringOptOut',
+  'createZendeskTickets',
+  'enableSubsectionGating',
+  'entranceExamEnabled',
+  'allowPublicWikiAccess',
+  'disableProgressGraph',
+  'showCalculator',
+  'hideProgressTab',
+  'edxnotes',
+  'videoAutoAdvance',
+  'videoSpeedOptimizations',
+  'useLatexCompiler',
+  'courseSurveyRequired',
+  'allowAnonymous',
+  'allowAnonymousToPeers',
+  'enableCcx',
+  'forceOnFlexiblePeerOpenassessments',
+  'downstreamCustomized',
+  'certificatesShowBeforeEnd',
 ]);
 
 /**

src/advanced-settings/data/types.ts

@@ -14,5 +14,5 @@ export type SettingEntry = [string, SettingData];
 
 /** Callback signatures shared across input components and section components. */
 export type SetEditedSettings = (
-  updater: (prev: Record<string, string>) => Record<string, string>
+  updater: (prev: Record<string, string>) => Record<string, string>,
 ) => void;

src/advanced-settings/setting-card/SettingCard.tsx

@@ -14,7 +14,9 @@ import { useIntl } from '@edx/frontend-platform/i18n';
 
 import messages from './messages';
 import {
-  FIELD_TYPE, getFieldType, serializeValue,
+  FIELD_TYPE,
+  getFieldType,
+  serializeValue,
 } from '../data/fieldTypes';
 import { FIELD_PLACEHOLDER_MESSAGES } from '../data/fieldTypeMessages';
 import BooleanInput from './inputs/BooleanInput';

src/advanced-settings/setting-card/inputs/BooleanInput.test.tsx

@@ -2,13 +2,16 @@ import { render, fireEvent, initializeMocks } from '@src/testUtils';
 import BooleanInput from './BooleanInput';
 
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
-const renderInput = (props: Record<string, any> = {}) => render(
-  // eslint-disable-next-line react/jsx-props-no-spreading
-  <BooleanInput name="selfPaced" displayName="Self Paced" {...(props as any)} />,
-);
+const renderInput = (props: Record<string, any> = {}) =>
+  render(
+    // eslint-disable-next-line react/jsx-props-no-spreading
+    <BooleanInput name="selfPaced" displayName="Self Paced" {...(props as any)} />,
+  );
 
 describe('<BooleanInput />', () => {
-  beforeEach(() => { initializeMocks(); });
+  beforeEach(() => {
+    initializeMocks();
+  });
 
   it('renders as checked when value is true', () => {
     const { getByRole } = renderInput({ value: true, onChange: jest.fn() });

src/advanced-settings/setting-card/inputs/BooleanInput.tsx

@@ -8,7 +8,10 @@ interface BooleanInputProps {
 }
 
 const BooleanInput = ({
-  value, name, displayName, onChange,
+  value,
+  name,
+  displayName,
+  onChange,
 }: BooleanInputProps) => (
   <Form.Switch
     id={`setting-bool-${name}`}

src/advanced-settings/setting-card/inputs/EnumInput.test.tsx

@@ -3,13 +3,16 @@ import EnumInput from './EnumInput';
 import { ENUM_OPTIONS } from '../../data/fieldTypes';
 
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
-const renderInput = (props: Record<string, any> = {}) => render(
-  // eslint-disable-next-line react/jsx-props-no-spreading
-  <EnumInput name="showanswer" displayName="Show Answer" {...(props as any)} />,
-);
+const renderInput = (props: Record<string, any> = {}) =>
+  render(
+    // eslint-disable-next-line react/jsx-props-no-spreading
+    <EnumInput name="showanswer" displayName="Show Answer" {...(props as any)} />,
+  );
 
 describe('<EnumInput />', () => {
-  beforeEach(() => { initializeMocks(); });
+  beforeEach(() => {
+    initializeMocks();
+  });
 
   it('renders the correct options for the given field name', () => {
     const { getAllByRole } = renderInput({ value: 'always', onChange: jest.fn() });