richie-education 3.1.3-dev12 → 3.1.3-dev17

package/js/pages/TeacherDashboardContractsLayout/hooks/useTeacherContractFilters/index.spec.tsx

@@ -7,7 +7,7 @@ import { MemoryRouter, Route, Routes } from 'react-router';
 import { RichieContextFactory as mockRichieContextFactory } from 'utils/test/factories/richie';
 import { createTestQueryClient } from 'utils/test/createTestQueryClient';
 import JoanieSessionProvider from 'contexts/SessionContext/JoanieSessionProvider';
-import { OfferFactory, OrganizationFactory } from 'utils/test/factories/joanie';
+import { OfferingFactory, OrganizationFactory } from 'utils/test/factories/joanie';
 import { ContractState } from 'types/Joanie';
 import useTeacherContractFilters from '.';
 
@@ -67,13 +67,13 @@ describe('useTeacherContractFilters', () => {
       expect(result.current.initialFilters).toStrictEqual({
         contract_ids: [],
         organization_id: defaultOrganization.id,
-        offer_id: undefined,
+        offering_id: undefined,
         signature_state: ContractState.SIGNED,
       });
       expect(result.current.filters).toStrictEqual({
         contract_ids: [],
         organization_id: defaultOrganization.id,
-        offer_id: undefined,
+        offering_id: undefined,
         signature_state: ContractState.SIGNED,
       });
     });
@@ -83,7 +83,7 @@ describe('useTeacherContractFilters', () => {
     const defaultOrganization = OrganizationFactory({ id: 'default' }).one();
     const filteredOrganization = OrganizationFactory({ id: 'filtered' }).one();
     const routeOrganization = OrganizationFactory({ id: 'route' }).one();
-    const routeOffer = OfferFactory().one();
+    const routeOffering = OfferingFactory().one();
     // fetching user's organizations to initialize default organizationId.
     fetchMock.get('https://joanie.test/api/v1.0/organizations/', [
       defaultOrganization,
@@ -92,8 +92,8 @@ describe('useTeacherContractFilters', () => {
     const { result } = renderHook(useTeacherContractFilters, {
       wrapper: ({ children }) => (
         <Wrapper
-          routePath="/:organizationId/:offerId"
-          initialEntry={`/${routeOrganization.id}/${routeOffer.id}?organization_id=${filteredOrganization.id}&signature_state=${ContractState?.UNSIGNED}&contract_ids=1&contract_ids=2`}
+          routePath="/:organizationId/:offeringId"
+          initialEntry={`/${routeOrganization.id}/${routeOffering.id}?organization_id=${filteredOrganization.id}&signature_state=${ContractState?.UNSIGNED}&contract_ids=1&contract_ids=2`}
         >
           {children}
         </Wrapper>
@@ -104,13 +104,13 @@ describe('useTeacherContractFilters', () => {
       expect(result.current.initialFilters).toStrictEqual({
         contract_ids: ['1', '2'],
         organization_id: routeOrganization.id,
-        offer_id: routeOffer.id,
+        offering_id: routeOffering.id,
         signature_state: ContractState.UNSIGNED,
       });
       expect(result.current.filters).toStrictEqual({
         contract_ids: ['1', '2'],
         organization_id: routeOrganization.id,
-        offer_id: routeOffer.id,
+        offering_id: routeOffering.id,
         signature_state: ContractState.UNSIGNED,
       });
     });
@@ -119,7 +119,7 @@ describe('useTeacherContractFilters', () => {
   it("should use organizationId from query parameters when it's not in route params", async () => {
     const defaultOrganization = OrganizationFactory({ id: 'default' }).one();
     const filteredOrganization = OrganizationFactory({ id: 'filtered' }).one();
-    const routeOffer = OfferFactory({ id: 'route' }).one();
+    const routeOffering = OfferingFactory({ id: 'route' }).one();
     // fetching user's organizations to initialize default organizationId.
     fetchMock.get('https://joanie.test/api/v1.0/organizations/', [
       defaultOrganization,
@@ -128,8 +128,8 @@ describe('useTeacherContractFilters', () => {
     const { result } = renderHook(useTeacherContractFilters, {
       wrapper: ({ children }) => (
         <Wrapper
-          routePath="/:offerId"
-          initialEntry={`/${routeOffer.id}/?organization_id=${filteredOrganization.id}&signature_state=${ContractState?.UNSIGNED}&contract_ids=1&contract_ids=2`}
+          routePath="/:offeringId"
+          initialEntry={`/${routeOffering.id}/?organization_id=${filteredOrganization.id}&signature_state=${ContractState?.UNSIGNED}&contract_ids=1&contract_ids=2`}
         >
           {children}
         </Wrapper>
@@ -140,13 +140,13 @@ describe('useTeacherContractFilters', () => {
       expect(result.current.initialFilters).toStrictEqual({
         contract_ids: ['1', '2'],
         organization_id: filteredOrganization.id,
-        offer_id: routeOffer.id,
+        offering_id: routeOffering.id,
         signature_state: ContractState.UNSIGNED,
       });
       expect(result.current.filters).toStrictEqual({
         contract_ids: ['1', '2'],
         organization_id: filteredOrganization.id,
-        offer_id: routeOffer.id,
+        offering_id: routeOffering.id,
         signature_state: ContractState.UNSIGNED,
       });
     });
@@ -155,7 +155,7 @@ describe('useTeacherContractFilters', () => {
   it('setFilters should update filter state', async () => {
     const defaultOrganization = OrganizationFactory({ id: 'default' }).one();
     const routeOrganization = OrganizationFactory({ id: 'route' }).one();
-    const routeOffer = OfferFactory().one();
+    const routeOffering = OfferingFactory().one();
     // fetching user's organizations to initialize default organizationId.
     fetchMock.get('https://joanie.test/api/v1.0/organizations/', [defaultOrganization]);
     const { result } = renderHook(useTeacherContractFilters, {
@@ -169,7 +169,7 @@ describe('useTeacherContractFilters', () => {
     const expectedInitialFilters = {
       contract_ids: [],
       organization_id: defaultOrganization.id,
-      offer_id: undefined,
+      offering_id: undefined,
       signature_state: ContractState.SIGNED,
     };
     await waitFor(() => {
@@ -179,7 +179,7 @@ describe('useTeacherContractFilters', () => {
     const newFilters = {
       contract_ids: ['1', '2'],
       organization_id: routeOrganization.id,
-      offer_id: routeOffer.id,
+      offering_id: routeOffering.id,
       signature_state: ContractState.UNSIGNED,
     };
     act(() => {

package/js/pages/TeacherDashboardContractsLayout/hooks/useTeacherContractFilters/index.tsx

@@ -5,16 +5,16 @@ import { ContractResourceQuery, ContractState } from 'types/Joanie';
 
 export type TeacherDashboardContractsParams = {
   organizationId?: string;
-  offerId?: string;
+  offeringId?: string;
 };
 
 const useTeacherContractFilters = () => {
-  const { offerId } = useParams<TeacherDashboardContractsParams>();
+  const { offeringId } = useParams<TeacherDashboardContractsParams>();
   const [searchParams] = useSearchParams();
   const searchFilters: ContractResourceQuery = useMemo(() => {
     return {
       organization_id: searchParams.get('organization_id') || undefined,
-      offer_id: searchParams.get('offer_id') || undefined,
+      offering_id: searchParams.get('offering_id') || undefined,
       contract_ids: searchParams.getAll('contract_ids') || undefined,
       signature_state:
         (searchParams.get('signature_state') as ContractState) || ContractState.SIGNED,
@@ -28,7 +28,7 @@ const useTeacherContractFilters = () => {
     return {
       ...searchFilters,
       organization_id: defaultOrganizationId,
-      offer_id: offerId,
+      offering_id: offeringId,
     };
   }, [defaultOrganizationId]);
   const [filters, setFilters] = useState<ContractResourceQuery>(initialFilters);

package/js/pages/TeacherDashboardContractsLayout/hooks/useTeacherContractsToSign.tsx

@@ -1,19 +1,22 @@
 import useContractAbilities from 'hooks/useContractAbilities';
 import { useOrganizationContracts } from 'hooks/useContracts';
-import { ContractState, Organization, Offer } from 'types/Joanie';
+import { ContractState, Organization, Offering } from 'types/Joanie';
 import { ContractActions } from 'utils/AbilitiesHelper/types';
 
 interface UseTeacherContractsToSignProps {
   organizationId?: Organization['id'];
-  offerId?: Offer['id'];
+  offeringId?: Offering['id'];
 }
 
-const useTeacherContractsToSign = ({ organizationId, offerId }: UseTeacherContractsToSignProps) => {
+const useTeacherContractsToSign = ({
+  organizationId,
+  offeringId,
+}: UseTeacherContractsToSignProps) => {
   const { items: contractsToSign, meta: contractsToSignMeta } = useOrganizationContracts(
     {
       signature_state: ContractState.LEARNER_SIGNED,
       organization_id: organizationId,
-      offer_id: offerId,
+      offering_id: offeringId,
     },
     { enabled: !!organizationId },
   );

package/js/pages/TeacherDashboardCourseLearnersLayout/hooks/useCourseLearnersFilters/index.spec.tsx

@@ -1,7 +1,7 @@
 import fetchMock from 'fetch-mock';
 import { renderHook, waitFor, act } from '@testing-library/react';
 import { RichieContextFactory as mockRichieContextFactory } from 'utils/test/factories/richie';
-import { OfferFactory, OrganizationFactory } from 'utils/test/factories/joanie';
+import { OfferingFactory, OrganizationFactory } from 'utils/test/factories/joanie';
 import { JoanieAppWrapper, setupJoanieSession } from 'utils/test/wrappers/JoanieAppWrapper';
 import useCourseLearnersFilters from '.';
 
@@ -32,12 +32,12 @@ describe('useCourseLearnersFilters', () => {
       expect(result.current.initialFilters).toStrictEqual({
         organization_id: undefined,
         course_id: undefined,
-        offer_id: undefined,
+        offering_id: undefined,
       });
       expect(result.current.filters).toStrictEqual({
         organization_id: undefined,
         course_id: undefined,
-        offer_id: undefined,
+        offering_id: undefined,
       });
     });
   });
@@ -46,7 +46,7 @@ describe('useCourseLearnersFilters', () => {
     const defaultOrganization = OrganizationFactory().one();
     const filteredOrganization = OrganizationFactory({ id: 'filtered' }).one();
     const routeOrganization = OrganizationFactory({ id: 'route' }).one();
-    const routeOffer = OfferFactory().one();
+    const routeOffering = OfferingFactory().one();
     // fetching user's organizations to initialize default organizationId.
     fetchMock.get('https://joanie.endpoint/api/v1.0/organizations/', [
       defaultOrganization,
@@ -56,9 +56,9 @@ describe('useCourseLearnersFilters', () => {
       wrapper: ({ children }) => (
         <JoanieAppWrapper
           routerOptions={{
-            path: '/:organizationId/:courseId/:offerId',
+            path: '/:organizationId/:courseId/:offeringId',
             initialEntries: [
-              `/${routeOrganization.id}/${routeOffer.course.id}/${routeOffer.id}?organization_id=${filteredOrganization.id}`,
+              `/${routeOrganization.id}/${routeOffering.course.id}/${routeOffering.id}?organization_id=${filteredOrganization.id}`,
             ],
           }}
         >
@@ -70,13 +70,13 @@ describe('useCourseLearnersFilters', () => {
     await waitFor(() => {
       expect(result.current.initialFilters).toStrictEqual({
         organization_id: routeOrganization.id,
-        course_id: routeOffer.course.id,
-        offer_id: routeOffer.id,
+        course_id: routeOffering.course.id,
+        offering_id: routeOffering.id,
       });
       expect(result.current.filters).toStrictEqual({
         organization_id: routeOrganization.id,
-        course_id: routeOffer.course.id,
-        offer_id: routeOffer.id,
+        course_id: routeOffering.course.id,
+        offering_id: routeOffering.id,
       });
     });
   });
@@ -84,7 +84,7 @@ describe('useCourseLearnersFilters', () => {
   it("should use organizationId from query parameters when it's not in route params", async () => {
     const defaultOrganization = OrganizationFactory().one();
     const filteredOrganization = OrganizationFactory({ id: 'filtered' }).one();
-    const routeOffer = OfferFactory({ id: 'route' }).one();
+    const routeOffering = OfferingFactory({ id: 'route' }).one();
     // fetching user's organizations to initialize default organizationId.
     fetchMock.get('https://joanie.endpoint/api/v1.0/organizations/', [
       defaultOrganization,
@@ -94,9 +94,9 @@ describe('useCourseLearnersFilters', () => {
       wrapper: ({ children }) => (
         <JoanieAppWrapper
           routerOptions={{
-            path: '/:courseId/:offerId',
+            path: '/:courseId/:offeringId',
             initialEntries: [
-              `/${routeOffer.course.id}/${routeOffer.id}/?organization_id=${filteredOrganization.id}`,
+              `/${routeOffering.course.id}/${routeOffering.id}/?organization_id=${filteredOrganization.id}`,
             ],
           }}
         >
@@ -108,13 +108,13 @@ describe('useCourseLearnersFilters', () => {
     await waitFor(() => {
       expect(result.current.initialFilters).toStrictEqual({
         organization_id: filteredOrganization.id,
-        course_id: routeOffer.course.id,
-        offer_id: routeOffer.id,
+        course_id: routeOffering.course.id,
+        offering_id: routeOffering.id,
       });
       expect(result.current.filters).toStrictEqual({
         organization_id: filteredOrganization.id,
-        course_id: routeOffer.course.id,
-        offer_id: routeOffer.id,
+        course_id: routeOffering.course.id,
+        offering_id: routeOffering.id,
       });
     });
   });
@@ -122,7 +122,7 @@ describe('useCourseLearnersFilters', () => {
   it('setFilters should update filter state', async () => {
     const defaultOrganization = OrganizationFactory({ id: 'all' }).one();
     const routeOrganization = OrganizationFactory({ id: 'route' }).one();
-    const routeOffer = OfferFactory().one();
+    const routeOffering = OfferingFactory().one();
     // fetching user's organizations to initialize default organizationId.
     fetchMock.get('https://joanie.endpoint/api/v1.0/organizations/', [defaultOrganization]);
     const { result } = renderHook(useCourseLearnersFilters, {
@@ -136,7 +136,7 @@ describe('useCourseLearnersFilters', () => {
     const expectedInitialFilters = {
       organization_id: defaultOrganization.id,
       course_id: undefined,
-      offer_id: undefined,
+      offering_id: undefined,
     };
     await waitFor(() => {
       expect(result.current.initialFilters).toStrictEqual(expectedInitialFilters);
@@ -144,8 +144,8 @@ describe('useCourseLearnersFilters', () => {
 
     const newFilters = {
       organization_id: routeOrganization.id,
-      course_id: routeOffer.course.id,
-      offer_id: routeOffer.id,
+      course_id: routeOffering.course.id,
+      offering_id: routeOffering.id,
     };
     act(() => {
       result.current.setFilters(newFilters);

package/js/pages/TeacherDashboardCourseLearnersLayout/hooks/useCourseLearnersFilters/index.ts

@@ -1,22 +1,22 @@
 import { useEffect, useMemo, useState } from 'react';
 import { useParams, useSearchParams } from 'react-router';
 import useDefaultOrganizationId from 'hooks/useDefaultOrganizationId';
-import { CourseListItem, CourseOrderResourceQuery, Offer, Organization } from 'types/Joanie';
+import { CourseListItem, CourseOrderResourceQuery, Offering, Organization } from 'types/Joanie';
 
 export type CourseLearnersParams = {
   courseId: CourseListItem['id'];
-  offerId?: Offer['id'];
+  offeringId?: Offering['id'];
   organizationId?: Organization['id'];
 };
 
 const useCourseLearnersFilters = () => {
-  const { courseId, offerId } = useParams<CourseLearnersParams>();
+  const { courseId, offeringId } = useParams<CourseLearnersParams>();
   const [searchParams] = useSearchParams();
   const searchFilters: CourseOrderResourceQuery = useMemo(() => {
     return {
       course_id: courseId,
       organization_id: searchParams.get('organization_id') || undefined,
-      offer_id: searchParams.get('offer_id') || undefined,
+      offering_id: searchParams.get('offering_id') || undefined,
     };
   }, Array.from(searchParams.entries()));
 
@@ -27,7 +27,7 @@ const useCourseLearnersFilters = () => {
     return {
       ...searchFilters,
       organization_id: defaultOrganizationId,
-      offer_id: offerId,
+      offering_id: offeringId,
     };
   }, [defaultOrganizationId]);
   const [filters, setFilters] = useState<CourseOrderResourceQuery>(initialFilters);