fix: remove unnecessary error logging and clean up unused imports in various components

Author: sinatragianpaolo

src/pages/Plan/context/planContext.tsx

@@ -95,7 +95,6 @@ export const PlanProvider = ({ children }: { children: ReactNode }) => {
       }
     } catch (error) {
       setIsPaymentInProgress(false);
-      console.error(`Error while checkout process: ${error}`);
     }
   };
 

src/pages/Plan/modals/DateInThePastAlertModal.tsx

@@ -66,7 +66,7 @@ const DateInThePastAlertModal = ({ onQuit }: { onQuit: () => void }) => {
       .then(() => {
         navigate(window.location.pathname, { replace: true });
       })
-      .catch((err) => {
+      .catch(() => {
         addToast(
           ({ close }) => (
             <Notification

src/pages/Plan/modules/Factory/modules/Literacy/Component.tsx

@@ -1,6 +1,5 @@
 import {
   AccordionNew,
-  Button,
   Checkbox,
   FormField,
   Hint,

src/pages/Plan/modules/Factory/modules/OutOfScope/Component.tsx

@@ -6,7 +6,6 @@ import {
   Label,
   SM,
   Span,
-  Textarea,
   Tooltip,
 } from '@appquality/unguess-design-system';
 import { useState } from 'react';

src/pages/Plan/modules/Factory/modules/TargetSize/Component.tsx

@@ -1,7 +1,6 @@
 import {
   AccordionNew,
   Alert,
-  Button,
   FormField,
   IconButton,
   Input,

src/pages/Plan/useSetDraftOnFailed.tsx

@@ -54,11 +54,7 @@ export const useSetDraftOnFailed = () => {
             { placement: 'top' }
           );
         })
-        .catch((err) => {
-          console.error(
-            'Error updating plan status after payment failure',
-            err
-          );
+        .catch(() => {
           navigate(notFoundRoute, { state: { from: location.pathname } });
         });
     };

tests/e2e/plan/modules/out_of_scope.spec.ts

@@ -1,15 +1,14 @@
-import draft from '../../../api/plans/pid/_get/200_draft_complete.json';
-import { expect, test } from '../../../fixtures/app';
+import { test } from '../../../fixtures/app';
 import { PlanPage } from '../../../fixtures/pages/Plan';
-import { OutOfScopeModule } from '../../../fixtures/pages/Plan/Module_out_of_scope';
+// import { OutOfScopeModule } from '../../../fixtures/pages/Plan/Module_out_of_scope';
 
 test.describe('The title module defines the Plan title.', () => {
   let planPage: PlanPage;
-  let module: OutOfScopeModule;
+  // let module: OutOfScopeModule;
 
   test.beforeEach(async ({ page }) => {
     planPage = new PlanPage(page);
-    module = new OutOfScopeModule(page);
+    // module = new OutOfScopeModule(page);
     await planPage.loggedIn();
     await planPage.mockPreferences();
     await planPage.mockWorkspace();

tests/e2e/plan/modules/tasks.spec.ts

@@ -23,9 +23,7 @@ test.describe('The tasks module defines a list of activities.', () => {
     await moduleBuilderPage.elements().tabInstructions().click();
   });
 
-  test('Tasks can be deleted, but it is required to have at least 1 item to Request a Quote', async ({
-    i18n,
-  }) => {
+  test('Tasks can be deleted, but it is required to have at least 1 item to Request a Quote', async () => {
     await expect(tasksModule.elements().module()).toBeVisible();
     const tasks = TasksModule.getTasksFromPlan(apiGetDraftMandatoryPlan);
     await expect(tasksModule.elements().taskListItem()).toHaveCount(