🚚(frontend) api/joanie as joanieLegacyClient

We'll implement a api client generated from joanie's openApi schema.
The current client needs to change place.

src/frontend/js/api/joanie/index.ts

@@ -0,0 +1,15 @@
+import context from 'utils/context';
+
+/**
+ * Flag which determines if joanie is enabled.
+ */
+export const isJoanieEnabled = !!context.joanie_backend;
+
+export {
+  default as JoanieLegacyClient,
+  getResponseBody as getResponseBodyLegacy,
+  checkStatus as checkStatusLegacy,
+  getAPIEndpoint as getAPIEndpointLegacy,
+  getRoutes as getRoutesLegacy,
+  buildApiUrl as buildApiUrlLegacy,
+} from './joanieLegacyClient';

src/frontend/js/api/joanie.spec.ts → src/frontend/js/api/joanie/joanieLegacyClient.spec.ts

@@ -1,7 +1,7 @@
 import fetchMock from 'fetch-mock';
 import { ResourcesQuery } from 'hooks/useResources';
 import { HttpStatusCode } from 'utils/errors/HttpError';
-import { buildApiUrl, getResponseBody } from './joanie';
+import { buildApiUrl, getResponseBody } from './joanieLegacyClient';
 
 describe('api/joanie', () => {
   it('getResponse should handle empty response body', async () => {

src/frontend/js/api/joanie.ts → src/frontend/js/api/joanie/joanieLegacyClient.ts

@@ -206,11 +206,6 @@ export const getRoutes = () => {
   };
 };
 
-/**
- * Flag which determines if joanie is enabled.
- */
-export const isJoanieEnabled = !!context.joanie_backend;
-
 const filterEmptyEntry = ([, value]: [PropertyKey, any]) => {
   if (value == null) return false; // Value is null/undefined
   if (value?.length !== undefined && value.length === 0) return false; // Value is an empty array/string

src/frontend/js/api/lms/joanie.ts

@@ -1,5 +1,5 @@
 import { matchPath, PathMatch } from 'react-router-dom';
-import JoanieApi from 'api/joanie';
+import { JoanieLegacyClient as JoanieApi } from 'api/joanie';
 import { LMSBackend } from 'types/commonDataProps';
 import { APIBackend, APILms } from 'types/api';
 

src/frontend/js/contexts/JoanieApiContext/index.tsx

@@ -1,7 +1,7 @@
 import type { PropsWithChildren } from 'react';
 import { createContext, useContext } from 'react';
 import type * as Joanie from 'types/Joanie';
-import API from 'api/joanie';
+import { JoanieLegacyClient as JoanieApi } from 'api/joanie';
 import type { Maybe } from 'types/utils';
 
 const JoanieApiContext = createContext<Maybe<Joanie.API>>(undefined);
@@ -10,7 +10,7 @@ const JoanieApiContext = createContext<Maybe<Joanie.API>>(undefined);
  * Provider to access to the Joanie API interface.
  */
 const JoanieApiProvider = ({ children }: PropsWithChildren<{}>) => {
-  const api = API();
+  const api = JoanieApi();
 
   return <JoanieApiContext.Provider value={api}>{children}</JoanieApiContext.Provider>;
 };

src/frontend/js/hooks/useCourseProductUnion/index.spec.tsx

@@ -8,7 +8,7 @@ import { History, HistoryContext } from 'hooks/useHistory';
 import { RichieContextFactory as mockRichieContextFactory } from 'utils/test/factories/richie';
 import { createTestQueryClient } from 'utils/test/createTestQueryClient';
 import { SessionProvider } from 'contexts/SessionContext';
-import { getRoutes } from 'api/joanie';
+import { getRoutesLegacy } from 'api/joanie';
 import { mockPaginatedResponse } from 'utils/test/mockPaginatedResponse';
 import { CourseListItemFactory, CourseProductRelationFactory } from 'utils/test/factories/joanie';
 import { useCourseProductUnion } from '.';
@@ -74,7 +74,7 @@ describe('useCourseProductUnion', () => {
   });
 
   it('should call courses and coursesProductRelation endpoints', async () => {
-    const ROUTES = getRoutes();
+    const ROUTES = getRoutesLegacy();
     const coursesUrl = ROUTES.courses.get.replace(':id/', '');
     const courseProductRelationsUrl = ROUTES.courseProductRelations.get.replace(':id/', '');
     fetchMock.get(
@@ -100,7 +100,7 @@ describe('useCourseProductUnion', () => {
 
   it('should call organization courses and organization coursesProductRelation endpoints', async () => {
     const organizationId = 'DUMMY_ORGANIZATION_ID';
-    const ROUTES = getRoutes();
+    const ROUTES = getRoutesLegacy();
     const organizationCoursesUrl = ROUTES.organizations.courses.get
       .replace(':organization_id', organizationId)
       .replace(':id/', '');

src/frontend/js/utils/react-query/useSessionMutation/index.spec.tsx

@@ -5,7 +5,7 @@ import { renderHook, waitFor } from '@testing-library/react';
 import { RichieContextFactory as mockRichieContextFactory } from 'utils/test/factories/richie';
 import BaseSessionProvider from 'contexts/SessionContext/BaseSessionProvider';
 import { useSession } from 'contexts/SessionContext';
-import { checkStatus } from 'api/joanie';
+import { checkStatusLegacy } from 'api/joanie';
 import { createTestQueryClient } from 'utils/test/createTestQueryClient';
 import { HttpStatusCode } from 'utils/errors/HttpError';
 import { useSessionMutation } from '.';
@@ -41,7 +41,7 @@ describe('useSessionMutation', () => {
       const session = useSession();
       const mutation = useSessionMutation<unknown, void, unknown>({
         mutationFn: () =>
-          fetch('http://api.endpoint/orders/create', { method: 'POST' }).then(checkStatus),
+          fetch('http://api.endpoint/orders/create', { method: 'POST' }).then(checkStatusLegacy),
         onError: handleError,
       });
 

src/frontend/js/utils/react-query/useSessionQuery/index.spec.tsx

@@ -4,7 +4,7 @@ import { PropsWithChildren } from 'react';
 import { renderHook, waitFor } from '@testing-library/react';
 import { RichieContextFactory as mockRichieContextFactory } from 'utils/test/factories/richie';
 import BaseSessionProvider from 'contexts/SessionContext/BaseSessionProvider';
-import { checkStatus } from 'api/joanie';
+import { checkStatusLegacy } from 'api/joanie';
 import { useSession } from 'contexts/SessionContext';
 import { createTestQueryClient } from 'utils/test/createTestQueryClient';
 import { HttpError, HttpStatusCode } from 'utils/errors/HttpError';
@@ -43,7 +43,9 @@ describe('useSessionQuery', () => {
 
     const useHooks = () => {
       const session = useSession();
-      useSessionQuery(['orders'], () => fetch('http://api.endpoint/orders/').then(checkStatus));
+      useSessionQuery(['orders'], () =>
+        fetch('http://api.endpoint/orders/').then(checkStatusLegacy),
+      );
 
       return session;
     };
@@ -73,7 +75,9 @@ describe('useSessionQuery', () => {
 
     const useHooks = () => {
       const session = useSession();
-      useSessionQuery(['orders'], () => fetch('http://api.endpoint/orders/').then(checkStatus));
+      useSessionQuery(['orders'], () =>
+        fetch('http://api.endpoint/orders/').then(checkStatusLegacy),
+      );
 
       return session;
     };

src/frontend/mocks/handlers/contracts.ts

@@ -1,11 +1,11 @@
 import { http, HttpResponse } from 'msw';
-import { getAPIEndpoint } from 'api/joanie';
+import { getAPIEndpointLegacy } from 'api/joanie';
 import { Contract, PaginatedResponse } from 'types/Joanie';
 import { ContractFactory } from 'utils/test/factories/joanie';
 import { PER_PAGE } from 'settings';
 
 export default [
-  http.get(`${getAPIEndpoint()}/contracts/`, () => {
+  http.get(`${getAPIEndpointLegacy()}/contracts/`, () => {
     return HttpResponse.json<PaginatedResponse<Contract>>({
       count: 250,
       results: ContractFactory().many(PER_PAGE.teacherContractList),