chore: run 'npm lint:fix'

Author: bradenmacdonald

.github/pull_request_template.md

@@ -5,8 +5,9 @@ Design decisions and their rationales should be documented in the repo (docstrin
 [OEP-19](https://open-edx-proposals.readthedocs.io/en/latest/oep-0019-bp-developer-documentation.html), and can be linked here.
 
 Useful information to include:
+
 - Which user roles will this change impact? Common user roles are "Learner", "Course Author",
-"Developer", and "Operator".
+  "Developer", and "Operator".
 - Include screenshots for changes to the UI (ideally, both "before" and "after" screenshots, if applicable).
 
 ## Supporting information
@@ -21,6 +22,7 @@ Please provide detailed step-by-step instructions for manually testing this chan
 ## Other information
 
 Include anything else that will help reviewers and consumers understand the change.
+
 - Does this change depend on other changes elsewhere?
 - Any special concerns or limitations? For example: deprecations, migrations, security, or accessibility.
 

.oxlintrc.json

@@ -7,7 +7,7 @@
   "rules": {
     "eslint/no-unused-vars": ["warn", {
       // Allow using {ignoredProp, ...keepTheRest} to omit a prop like 'ignoredProp' from an object.
-      "ignoreRestSiblings": true,
+      "ignoreRestSiblings": true
     }],
     // We disable exhaustive-deps because: it's noisy, and we often include extra deps when we want a memoized thing to
     // re-calculate after some change, even if we're not using that thing in the calculation.
@@ -21,11 +21,11 @@
         // complete, but we rarely if ever want that; we usually want to
         // continue invalidating more things immediately. So we don't usually
         // want to await this.
-        "invalidateQueries",
+        "invalidateQueries"
       ]
     }]
   },
   "ignorePatterns": [
-    "webpack.dev-tutor.config.js",
+    "webpack.dev-tutor.config.js"
   ]
-}
+}

plugins/course-apps/calculator/package.json

@@ -1,17 +1,17 @@
 {
-    "name": "@openedx-plugins/course-app-calculator",
-    "version": "0.1.0",
-    "description": "Calculator configuration for courses using it",
-    "peerDependencies": {
-        "@edx/frontend-app-authoring": "*",
-        "@edx/frontend-platform": "*",
-        "@openedx/paragon": "*",
-        "prop-types": "*",
-        "react": "*"
-    },
-    "peerDependenciesMeta": {
-        "@edx/frontend-app-authoring": {
-            "optional": true
-        }
+  "name": "@openedx-plugins/course-app-calculator",
+  "version": "0.1.0",
+  "description": "Calculator configuration for courses using it",
+  "peerDependencies": {
+    "@edx/frontend-app-authoring": "*",
+    "@edx/frontend-platform": "*",
+    "@openedx/paragon": "*",
+    "prop-types": "*",
+    "react": "*"
+  },
+  "peerDependenciesMeta": {
+    "@edx/frontend-app-authoring": {
+      "optional": true
     }
+  }
 }

plugins/course-apps/dates/package.json

@@ -1,17 +1,17 @@
 {
-    "name": "@openedx-plugins/course-app-dates",
-    "version": "0.1.0",
-    "description": "Dates configuration for courses using it",
-    "peerDependencies": {
-        "@edx/frontend-app-authoring": "*",
-        "@edx/frontend-platform": "*",
-        "@openedx/paragon": "*",
-        "prop-types": "*",
-        "react": "*"
-    },
-    "peerDependenciesMeta": {
-        "@edx/frontend-app-authoring": {
-            "optional": true
-        }
+  "name": "@openedx-plugins/course-app-dates",
+  "version": "0.1.0",
+  "description": "Dates configuration for courses using it",
+  "peerDependencies": {
+    "@edx/frontend-app-authoring": "*",
+    "@edx/frontend-platform": "*",
+    "@openedx/paragon": "*",
+    "prop-types": "*",
+    "react": "*"
+  },
+  "peerDependenciesMeta": {
+    "@edx/frontend-app-authoring": {
+      "optional": true
     }
+  }
 }

plugins/course-apps/edxnotes/package.json

@@ -1,17 +1,17 @@
 {
-    "name": "@openedx-plugins/course-app-edxnotes",
-    "version": "0.1.0",
-    "description": "edxnotes configuration for courses using it",
-    "peerDependencies": {
-        "@edx/frontend-app-authoring": "*",
-        "@edx/frontend-platform": "*",
-        "@openedx/paragon": "*",
-        "prop-types": "*",
-        "react": "*"
-    },
-    "peerDependenciesMeta": {
-        "@edx/frontend-app-authoring": {
-            "optional": true
-        }
+  "name": "@openedx-plugins/course-app-edxnotes",
+  "version": "0.1.0",
+  "description": "edxnotes configuration for courses using it",
+  "peerDependencies": {
+    "@edx/frontend-app-authoring": "*",
+    "@edx/frontend-platform": "*",
+    "@openedx/paragon": "*",
+    "prop-types": "*",
+    "react": "*"
+  },
+  "peerDependenciesMeta": {
+    "@edx/frontend-app-authoring": {
+      "optional": true
     }
+  }
 }

plugins/course-apps/learning_assistant/Settings.jsx

@@ -15,7 +15,7 @@ const LearningAssistantSettings = ({ onClose }) => {
   const intl = useIntl();
 
   // We need to render more than one link, so we use the bodyChildren prop.
-  const bodyChildren = (
+  const bodyChildren =
     appInfo?.documentationLinks?.learnMoreOpenaiDataPrivacy && appInfo?.documentationLinks?.learnMoreOpenai
       ? (
         <div className="d-flex flex-column">
@@ -41,8 +41,7 @@ const LearningAssistantSettings = ({ onClose }) => {
           )}
         </div>
       )
-      : null
-  );
+      : null;
 
   return (
     <AppSettingsModal

plugins/course-apps/learning_assistant/Settings.test.jsx

@@ -5,15 +5,14 @@ import { RequestStatus } from 'CourseAuthoring/data/constants';
 import { initializeMocks, render } from 'CourseAuthoring/testUtils';
 import LearningAssistantSettings from './Settings';
 
-const onClose = () => { };
+const onClose = () => {};
 
 describe('Learning Assistant Settings', () => {
   it('renders', async () => {
     const initialState = {
       models: {
         courseApps: {
-          learning_assistant:
-          {
+          learning_assistant: {
             id: 'learning_assistant',
             enabled: true,
             name: 'Learning Assistant',
@@ -38,10 +37,12 @@ describe('Learning Assistant Settings', () => {
     render(<LearningAssistantSettings onClose={onClose} />);
 
     const toggleDescription = 'Reinforce learning concepts by sharing text-based course content '
-    + 'with OpenAI (via API) to power an in-course Learning Assistant. Learners can leave feedback about the quality '
-    + 'of the AI-powered experience for use by edX to improve the performance of the tool.';
+      + 'with OpenAI (via API) to power an in-course Learning Assistant. Learners can leave feedback about the quality '
+      + 'of the AI-powered experience for use by edX to improve the performance of the tool.';
 
-    await waitFor(() => expect(screen.getByRole('heading', { name: 'Configure Learning Assistant' })).toBeInTheDocument());
+    await waitFor(() =>
+      expect(screen.getByRole('heading', { name: 'Configure Learning Assistant' })).toBeInTheDocument()
+    );
     await waitFor(() => expect(screen.getByText(toggleDescription)).toBeInTheDocument());
     await waitFor(() => expect(screen.getByText('Learn more about how OpenAI handles data')).toBeInTheDocument());
     await waitFor(() => expect(screen.getByText('Learn more about OpenAI API data privacy')).toBeInTheDocument());

plugins/course-apps/learning_assistant/package.json

@@ -1,19 +1,18 @@
 {
-    "name": "@openedx-plugins/course-app-learning_assistant",
-    "version": "0.1.0",
-    "description": "Learning Assistant configuration for courses using it",
-    "peerDependencies": {
-        "@edx/frontend-app-authoring": "*",
-        "@edx/frontend-platform": "*",
-        "@openedx/paragon": "*",
-        "prop-types": "*",
-        "react": "*",
-        "yup": "*"
-    },
-    "peerDependenciesMeta": {
-        "@edx/frontend-app-authoring": {
-            "optional": true
-        }
+  "name": "@openedx-plugins/course-app-learning_assistant",
+  "version": "0.1.0",
+  "description": "Learning Assistant configuration for courses using it",
+  "peerDependencies": {
+    "@edx/frontend-app-authoring": "*",
+    "@edx/frontend-platform": "*",
+    "@openedx/paragon": "*",
+    "prop-types": "*",
+    "react": "*",
+    "yup": "*"
+  },
+  "peerDependenciesMeta": {
+    "@edx/frontend-app-authoring": {
+      "optional": true
     }
+  }
 }
-  

plugins/course-apps/live/BBBSettings.jsx

@@ -42,15 +42,17 @@ const BbbSettings = ({
   };
   return (
     <>
-      {isPiiDisabled ? (
-        <p data-testid="request-pii-sharing">
-          {intl.formatMessage(messages.requestPiiSharingEnableForBbb, { provider: providerNames[values.provider] })}
-        </p>
-      ) : (
-        <p data-testid="helper-text">
-          {intl.formatMessage(messages.providerHelperText, { providerName: providerNames[values.provider] })}
-        </p>
-      )}
+      {isPiiDisabled ?
+        (
+          <p data-testid="request-pii-sharing">
+            {intl.formatMessage(messages.requestPiiSharingEnableForBbb, { provider: providerNames[values.provider] })}
+          </p>
+        ) :
+        (
+          <p data-testid="helper-text">
+            {intl.formatMessage(messages.providerHelperText, { providerName: providerNames[values.provider] })}
+          </p>
+        )}
 
       <Form.Group controlId="bbs-settings" data-testid="plansDropDown">
         <Form.Label as="planSelector" className="h6">
@@ -78,33 +80,35 @@ const BbbSettings = ({
         showLaunchIcon
         className="text-primary-500 pt-2"
       >
-        { intl.formatMessage(messages.learnMore, { providerName: 'plans' }) }
+        {intl.formatMessage(messages.learnMore, { providerName: 'plans' })}
       </Hyperlink>
       <>
         <AppConfigFormDivider thick marginAdj={{ default: 0, sm: 2 }} />
-        {isPiiDisabled ? (
-          <p data-testid="help-request-pii-sharing">
-            {intl.formatMessage(messages.piiSharingEnableHelpTextBbb)}
-          </p>
-        ) : (
-          <>
-            {bbbPlan === bbbPlanTypes.commercial && <LiveCommonFields values={values} />}
-            {bbbPlan === bbbPlanTypes.free && (
-              <span data-testid="free-plan-message">
-                {intl.formatMessage(messages.freePlanMessage)}
-                <Hyperlink
-                  destination={getExternalLinkUrl('https://bigbluebutton.org/privacy-policy/')}
-                  target="_blank"
-                  rel="noopener noreferrer"
-                  showLaunchIcon
-                  className="text-gray-700 ml-1"
-                >
-                  {intl.formatMessage(messages.privacyPolicy)}
-                </Hyperlink>
-              </span>
-            )}
-          </>
-        )}
+        {isPiiDisabled ?
+          (
+            <p data-testid="help-request-pii-sharing">
+              {intl.formatMessage(messages.piiSharingEnableHelpTextBbb)}
+            </p>
+          ) :
+          (
+            <>
+              {bbbPlan === bbbPlanTypes.commercial && <LiveCommonFields values={values} />}
+              {bbbPlan === bbbPlanTypes.free && (
+                <span data-testid="free-plan-message">
+                  {intl.formatMessage(messages.freePlanMessage)}
+                  <Hyperlink
+                    destination={getExternalLinkUrl('https://bigbluebutton.org/privacy-policy/')}
+                    target="_blank"
+                    rel="noopener noreferrer"
+                    showLaunchIcon
+                    className="text-gray-700 ml-1"
+                  >
+                    {intl.formatMessage(messages.privacyPolicy)}
+                  </Hyperlink>
+                </span>
+              )}
+            </>
+          )}
       </>
     </>
   );

plugins/course-apps/live/BbbSettings.test.jsx

@@ -63,8 +63,14 @@ const mockStore = async ({
   const fetchProviderConfigUrl = `${providersApiUrl}/${courseId}/`;
   const fetchLiveConfigUrl = `${providerConfigurationApiUrl}/${courseId}/`;
 
-  axiosMock.onGet(fetchProviderConfigUrl).reply(200, configurationProviders(emailSharing, usernameSharing, 'big_blue_button', isFreeTier));
-  axiosMock.onGet(fetchLiveConfigUrl).reply(200, generateLiveConfigurationApiResponse(enabled, piiSharingAllowed, 'bigBlueButton', isFreeTier));
+  axiosMock.onGet(fetchProviderConfigUrl).reply(
+    200,
+    configurationProviders(emailSharing, usernameSharing, 'big_blue_button', isFreeTier),
+  );
+  axiosMock.onGet(fetchLiveConfigUrl).reply(
+    200,
+    generateLiveConfigurationApiResponse(enabled, piiSharingAllowed, 'bigBlueButton', isFreeTier),
+  );
 
   await executeThunk(fetchLiveProviders(courseId), store.dispatch);
   await executeThunk(fetchLiveConfiguration(courseId), store.dispatch);
@@ -88,14 +94,17 @@ describe('BBB Settings', () => {
     expect(container.querySelector('select[name="tierType"]')).not.toBeDisabled();
   });
 
-  test.each([[true, 3], [false, 2]])('Plan dropdown should display correct number of options', async (isFreeTier, noOfOptions) => {
-    await mockStore({ emailSharing: true, isFreeTier });
-    renderComponent();
-    const spinner = getByRole(container, 'status');
-    await waitForElementToBeRemoved(spinner);
-    const dropDown = queryByTestId(container, 'plansDropDown');
-    expect(getAllByRole(dropDown, 'option').length).toBe(noOfOptions);
-  });
+  test.each([[true, 3], [false, 2]])(
+    'Plan dropdown should display correct number of options',
+    async (isFreeTier, noOfOptions) => {
+      await mockStore({ emailSharing: true, isFreeTier });
+      renderComponent();
+      const spinner = getByRole(container, 'status');
+      await waitForElementToBeRemoved(spinner);
+      const dropDown = queryByTestId(container, 'plansDropDown');
+      expect(getAllByRole(dropDown, 'option').length).toBe(noOfOptions);
+    },
+  );
 
   test(
     'Connect to support and PII sharing message is visible and plans selection is disabled, When pii sharing is disabled, ',