richie-education 3.1.3-dev12 → 3.1.3-dev17

package/js/pages/TeacherDashboardTraining/index.spec.tsx

@@ -11,7 +11,7 @@ import {
   UserFactory,
 } from 'utils/test/factories/richie';
 import JoanieSessionProvider from 'contexts/SessionContext/JoanieSessionProvider';
-import { OfferFactory, OrganizationFactory } from 'utils/test/factories/joanie';
+import { OfferingFactory, OrganizationFactory } from 'utils/test/factories/joanie';
 import { createTestQueryClient } from 'utils/test/createTestQueryClient';
 import { expectNoSpinner } from 'utils/test/expectSpinner';
 import { DashboardBreadcrumbsProvider } from 'widgets/Dashboard/contexts/DashboardBreadcrumbsContext';
@@ -46,9 +46,9 @@ describe('components/TeacherDashboardTrainingLoader', () => {
   });
 
   it('should render TeacherDashboardTrainingLoader page', async () => {
-    const offer = OfferFactory().one();
-    fetchMock.get(`https://joanie.endpoint/api/v1.0/organizations/?offer_id=${offer.id}`, []);
-    fetchMock.get(`https://joanie.endpoint/api/v1.0/offers/${offer.id}/`, offer);
+    const offering = OfferingFactory().one();
+    fetchMock.get(`https://joanie.endpoint/api/v1.0/organizations/?offering_id=${offering.id}`, []);
+    fetchMock.get(`https://joanie.endpoint/api/v1.0/offerings/${offering.id}/`, offering);
 
     const user = UserFactory().one();
     render(
@@ -61,12 +61,12 @@ describe('components/TeacherDashboardTrainingLoader', () => {
                   router={createMemoryRouter(
                     [
                       {
-                        path: ':offerId',
+                        path: ':offeringId',
                         element: <TeacherDashboardTrainingLoader />,
                       },
                     ],
                     {
-                      initialEntries: [`/${offer.id}`],
+                      initialEntries: [`/${offering.id}`],
                     },
                   )}
                 />
@@ -82,10 +82,10 @@ describe('components/TeacherDashboardTrainingLoader', () => {
     await expectNoSpinner('Loading course...');
 
     nbApiCalls += 1; // organizations api call
-    nbApiCalls += 1; // offers api call
+    nbApiCalls += 1; // offerings api call
     const calledUrls = fetchMock.calls().map((call) => call[0]);
     expect(calledUrls).toHaveLength(nbApiCalls);
-    expect(calledUrls).toContain(`https://joanie.endpoint/api/v1.0/offers/${offer.id}/`);
+    expect(calledUrls).toContain(`https://joanie.endpoint/api/v1.0/offerings/${offering.id}/`);
 
     // main titles
     expect(
@@ -94,28 +94,28 @@ describe('components/TeacherDashboardTrainingLoader', () => {
       }),
     ).toBeInTheDocument();
 
-    expect(screen.getAllByRole('heading', { name: capitalize(offer.product.title) })).toHaveLength(
-      2,
-    );
+    expect(
+      screen.getAllByRole('heading', { name: capitalize(offering.product.title) }),
+    ).toHaveLength(2);
 
-    const nbCourseRun = offer.product.target_courses.reduce(
+    const nbCourseRun = offering.product.target_courses.reduce(
       (acc, course) => acc + course.course_runs.length,
       0,
     );
     expect(screen.getAllByRole('link', { name: 'Go to course area' })).toHaveLength(nbCourseRun);
   });
 
-  it('should fetch offer with organization id if there is one in the path', async () => {
+  it('should fetch offering with organization id if there is one in the path', async () => {
     const organization = OrganizationFactory().one();
-    const offer = OfferFactory({
+    const offering = OfferingFactory({
       organizations: [organization],
     }).one();
     fetchMock.get(
-      `https://joanie.endpoint/api/v1.0/organizations/${organization.id}/offers/${offer.id}/`,
-      offer,
+      `https://joanie.endpoint/api/v1.0/organizations/${organization.id}/offerings/${offering.id}/`,
+      offering,
     );
     fetchMock.get(
-      `https://joanie.endpoint/api/v1.0/organizations/${organization.id}/contracts/?offer_id=${offer.id}&signature_state=half_signed&page=1&page_size=25`,
+      `https://joanie.endpoint/api/v1.0/organizations/${organization.id}/contracts/?offering_id=${offering.id}&signature_state=half_signed&page=1&page_size=25`,
       [],
     );
 
@@ -130,12 +130,12 @@ describe('components/TeacherDashboardTrainingLoader', () => {
                   router={createMemoryRouter(
                     [
                       {
-                        path: '/:organizationId/:offerId',
+                        path: '/:organizationId/:offeringId',
                         element: <TeacherDashboardTrainingLoader />,
                       },
                     ],
                     {
-                      initialEntries: [`/${organization.id}/${offer.id}`],
+                      initialEntries: [`/${organization.id}/${offering.id}`],
                     },
                   )}
                 />
@@ -151,11 +151,11 @@ describe('components/TeacherDashboardTrainingLoader', () => {
     await expectNoSpinner('Loading course...');
 
     nbApiCalls += 1; // contracts api call
-    nbApiCalls += 1; // offers api call
+    nbApiCalls += 1; // offerings api call
     const calledUrls = fetchMock.calls().map((call) => call[0]);
     expect(calledUrls).toHaveLength(nbApiCalls);
     expect(calledUrls).toContain(
-      `https://joanie.endpoint/api/v1.0/organizations/${organization.id}/offers/${offer.id}/`,
+      `https://joanie.endpoint/api/v1.0/organizations/${organization.id}/offerings/${offering.id}/`,
     );
 
     // main titles
@@ -165,11 +165,11 @@ describe('components/TeacherDashboardTrainingLoader', () => {
       }),
     ).toBeInTheDocument();
 
-    expect(screen.getAllByRole('heading', { name: capitalize(offer.product.title) })).toHaveLength(
-      2,
-    );
+    expect(
+      screen.getAllByRole('heading', { name: capitalize(offering.product.title) }),
+    ).toHaveLength(2);
 
-    const nbCourseRun = offer.product.target_courses.reduce(
+    const nbCourseRun = offering.product.target_courses.reduce(
       (acc, course) => acc + course.course_runs.length,
       0,
     );

package/js/pages/TeacherDashboardTraining/index.tsx

@@ -6,33 +6,33 @@ import { DashboardLayout } from 'widgets/Dashboard/components/DashboardLayout';
 import { DashboardCard } from 'widgets/Dashboard/components/DashboardCard';
 import { Icon, IconTypeEnum } from 'components/Icon';
 import Banner, { BannerType } from 'components/Banner';
-import { Offer } from 'types/Joanie';
+import { Offering } from 'types/Joanie';
 
 const messages = defineMessages({
-  errorNoOffer: {
+  errorNoOffering: {
     defaultMessage: "This product doesn't exist",
-    description: 'Message displayed when requested offer is not found',
-    id: 'components.TeacherDashboardTraining.errorNoOffer',
+    description: 'Message displayed when requested offering is not found',
+    id: 'components.TeacherDashboardTraining.errorNoOffering',
   },
 });
 
 interface TeacherDashboardTrainingProps {
-  offer: Offer;
+  offering: Offering;
 }
 
-export const TeacherDashboardTraining = ({ offer }: TeacherDashboardTrainingProps) => {
+export const TeacherDashboardTraining = ({ offering }: TeacherDashboardTrainingProps) => {
   const intl = useIntl();
-  return offer ? (
+  return offering ? (
     <div className="teacher-course-page">
       <DashboardCard
         className="icon-arrow-right-rounded"
         header={
           <div>
             <div className="dashboard__title_container--small">
-              <h2 className="dashboard__title--large">{capitalize(offer.product.title)}</h2>
+              <h2 className="dashboard__title--large">{capitalize(offering.product.title)}</h2>
             </div>
-            {offer.product.description && (
-              <div className="dashboard__quote">{offer.product.description}</div>
+            {offering.product.description && (
+              <div className="dashboard__quote">{offering.product.description}</div>
             )}
           </div>
         }
@@ -40,7 +40,7 @@ export const TeacherDashboardTraining = ({ offer }: TeacherDashboardTrainingProp
         fullWidth
       />
       <DashboardLayout.NestedSection>
-        {offer.product.target_courses.map((course) => (
+        {offering.product.target_courses.map((course) => (
           <DashboardLayout.Section key={`course_target_${course.code}`}>
             <DashboardCard
               className="icon-arrow-right-rounded"
@@ -62,7 +62,11 @@ export const TeacherDashboardTraining = ({ offer }: TeacherDashboardTrainingProp
       </DashboardLayout.NestedSection>
     </div>
   ) : (
-    <Banner message={intl.formatMessage(messages.errorNoOffer)} type={BannerType.ERROR} rounded />
+    <Banner
+      message={intl.formatMessage(messages.errorNoOffering)}
+      type={BannerType.ERROR}
+      rounded
+    />
   );
 };
 

package/js/types/Joanie.ts

@@ -38,7 +38,7 @@ export interface Organization {
 }
 
 export interface OrganizationResourceQuery extends ResourcesQuery {
-  offer_id?: Offer['id'];
+  offering_id?: Offering['id'];
 }
 
 export interface ContractDefinition {
@@ -174,7 +174,7 @@ export interface DefinitionResourcesProduct {
   contract_definition_id: Nullable<ContractDefinition['id']>;
 }
 
-export interface OfferLight {
+export interface OfferingLight {
   id: string;
   course: CourseLight;
   organizations: Organization[];
@@ -182,18 +182,25 @@ export interface OfferLight {
   created_on: string;
 }
 
-export interface Offer extends OfferLight {
-  is_withdrawable: boolean;
+export interface OfferingRule {
   discounted_price: Nullable<number>;
   discount_rate: Nullable<number>;
   discount_amount: Nullable<number>;
   discount_start: Nullable<string>;
   discount_end: Nullable<string>;
   description: Nullable<string>;
-  nb_seats_available: Nullable<number>;
-  seats: Nullable<number>;
+  nb_available_seats: Nullable<number>;
+  has_seat_limit: boolean;
+  has_seats_left: boolean;
+}
+
+export interface Offering extends OfferingLight {
+  is_withdrawable: boolean;
+  rules: OfferingRule;
 }
-export function isOffer(entity: CourseListItem | OfferLight | RichieCourse): entity is OfferLight {
+export function isOffering(
+  entity: CourseListItem | OfferingLight | RichieCourse,
+): entity is OfferingLight {
   return 'course' in entity && 'product' in entity;
 }
 
@@ -238,7 +245,7 @@ export interface Enrollment {
   was_created_by_order: boolean;
   created_on: string;
   orders: OrderEnrollment[];
-  offers: Offer[];
+  offerings: Offering[];
   certificate_id: Nullable<string>;
 }
 export const isEnrollment = (obj: unknown | Enrollment | OpenEdXEnrollment): obj is Enrollment => {
@@ -252,7 +259,7 @@ export const isEnrollment = (obj: unknown | Enrollment | OpenEdXEnrollment): obj
     'was_created_by_order' in obj &&
     'created_on' in obj &&
     'orders' in obj &&
-    'offers' in obj &&
+    'offerings' in obj &&
     'certificate_id' in obj
   );
 };
@@ -402,7 +409,7 @@ export interface NestedCourseOrder {
 
 export interface CourseOrderResourceQuery extends PaginatedResourceQuery {
   course_id?: CourseListItem['id'];
-  offer_id?: Offer['id'];
+  offering_id?: Offering['id'];
   organization_id?: Organization['id'];
   product_id?: Product['id'];
 }
@@ -537,8 +544,8 @@ export interface CourseProductQueryFilters extends ResourcesQuery {
   id?: Product['id'];
   course_id?: CourseListItem['id'];
 }
-export interface OfferQueryFilters extends PaginatedResourceQuery {
-  id?: Offer['id'];
+export interface OfferingQueryFilters extends PaginatedResourceQuery {
+  id?: Offering['id'];
   organization_id?: Organization['id'];
   product_type?: ProductType;
   query?: string;
@@ -551,7 +558,7 @@ export enum ContractState {
 }
 export interface ContractResourceQuery extends PaginatedResourceQuery {
   organization_id?: Organization['id'];
-  offer_id?: Offer['id'];
+  offering_id?: Offering['id'];
   contract_ids?: Contract['id'][];
   signature_state?: ContractState;
 }
@@ -559,7 +566,7 @@ export interface ContractResourceQuery extends PaginatedResourceQuery {
 export interface OrganizationContractSignatureLinksFilters {
   contracts_ids?: string[];
   organization_id: Organization['id'];
-  offer_ids?: Offer['id'][];
+  offering_ids?: Offering['id'][];
 }
 
 export interface ContractInvitationLinkResponse {
@@ -653,10 +660,10 @@ interface APIUser {
       check: (id: string) => Promise<Response>;
       create: ({
         organization_id,
-        offer_id,
+        offering_id,
       }: {
         organization_id?: Organization['id'];
-        offer_id?: Offer['id'];
+        offering_id?: Offering['id'];
       }) => Promise<{ url: string }>;
       get: (id: string) => Promise<File>;
     };
@@ -672,7 +679,7 @@ export interface API {
       ? Promise<Nullable<CourseListItem>>
       : Promise<PaginatedResponse<CourseListItem>>;
     products: {
-      get(filters?: CourseProductQueryFilters): Promise<Nullable<Offer>>;
+      get(filters?: CourseProductQueryFilters): Promise<Nullable<Offering>>;
       paymentSchedule: {
         get(filters?: CourseProductQueryFilters): Promise<Nullable<PaymentSchedule>>;
       };
@@ -705,12 +712,12 @@ export interface API {
       filters?: CourseRunFilters,
     ): CourseRunFilters extends { id: string } ? Promise<Nullable<CourseRun>> : Promise<CourseRun>;
   };
-  offers: {
+  offerings: {
     get<Filters extends PaginatedResourceQuery = PaginatedResourceQuery>(
       filters?: Filters,
     ): Filters extends { id: string }
-      ? Promise<Nullable<Offer>>
-      : Promise<PaginatedResponse<OfferLight>>;
+      ? Promise<Nullable<Offering>>
+      : Promise<PaginatedResponse<OfferingLight>>;
   };
   contractDefinitions: {
     previewTemplate(id: string): Promise<File>;

package/js/utils/test/factories/joanie.ts

@@ -12,7 +12,7 @@ import {
   CourseLight,
   CourseListItem,
   CourseProduct,
-  Offer,
+  Offering,
   CourseRun,
   CredentialOrder,
   CredentialProduct,
@@ -89,7 +89,7 @@ export const EnrollmentFactory = factory((): Enrollment => {
     id: faker.string.uuid(),
     course_run: CourseRunWithCourseFactory().one(),
     is_active: true,
-    offers: OfferFactory().many(1),
+    offerings: OfferingFactory().many(1),
     state: EnrollmentState.SET,
     was_created_by_order: false,
     created_on: faker.date.past({ years: 1 }).toISOString(),
@@ -309,7 +309,7 @@ export const NestedCourseOrderFactory = factory((): NestedCourseOrder => {
   };
 });
 
-export const OfferFactory = factory((): Offer => {
+export const OfferingFactory = factory((): Offering => {
   return {
     id: faker.string.uuid(),
     created_on: faker.date.past().toISOString(),
@@ -317,14 +317,17 @@ export const OfferFactory = factory((): Offer => {
     product: ProductFactory().one(),
     organizations: OrganizationFactory().many(1),
     is_withdrawable: true,
-    discounted_price: null,
-    discount_rate: null,
-    discount_amount: null,
-    discount_start: null,
-    discount_end: null,
-    description: null,
-    seats: null,
-    nb_seats_available: null,
+    rules: {
+      discounted_price: null,
+      discount_rate: null,
+      discount_amount: null,
+      discount_start: null,
+      discount_end: null,
+      description: null,
+      nb_available_seats: null,
+      has_seat_limit: false,
+      has_seats_left: true,
+    },
   };
 });
 

package/js/utils/test/mockCourseProductWithOrder.ts

@@ -3,14 +3,14 @@ import { CredentialOrder } from 'types/Joanie';
 import {
   ContractDefinitionFactory,
   CourseFactory,
-  OfferFactory,
+  OfferingFactory,
   ProductFactory,
 } from 'utils/test/factories/joanie';
 
 export const mockCourseProductWithOrder = (order: CredentialOrder) => {
   const courseCode = order.course.code;
   const productId = order.product_id;
-  const offer = OfferFactory({
+  const offering = OfferingFactory({
     product: ProductFactory({
       id: order.product_id,
       contract_definition: order.contract ? ContractDefinitionFactory().one() : undefined,
@@ -22,7 +22,7 @@ export const mockCourseProductWithOrder = (order: CredentialOrder) => {
 
   fetchMock.get(
     `https://joanie.endpoint/api/v1.0/courses/${courseCode}/products/${productId}/`,
-    offer,
+    offering,
   );
-  return offer;
+  return offering;
 };

package/js/widgets/Dashboard/components/DashboardItem/Enrollment/DashboardItemEnrollment.tsx

@@ -33,7 +33,7 @@ export const DashboardItemEnrollment = ({ enrollment }: DashboardItemCourseRunPr
         </div>
       </div>,
     ];
-    enrollment.offers.forEach(({ product, is_withdrawable }) => {
+    enrollment.offerings.forEach(({ product, is_withdrawable }) => {
       if (isCertificateProduct(product)) {
         partialFooterList.push(
           <ProductCertificateFooter

package/js/widgets/Dashboard/components/DashboardItem/Enrollment/ProductCertificateFooter/index.spec.tsx

@@ -249,7 +249,7 @@ describe('<ProductCertificateFooter/>', () => {
         course,
       }).one(),
     }).one();
-    enrollment.offers[0].product = CertificateProductFactory().one();
+    enrollment.offerings[0].product = CertificateProductFactory().one();
 
     fetchMock.get(
       `https://joanie.endpoint/api/v1.0/enrollments/?was_created_by_order=false&is_active=true&page=1&page_size=${PER_PAGE.useOrdersEnrollments}`,

package/js/widgets/Dashboard/components/DashboardItem/Order/DashboardItemOrder.tsx

@@ -67,15 +67,15 @@ export const DashboardItemOrder = ({
 }: DashboardItemOrderProps) => {
   const { course } = order;
   const intl = useIntl();
-  const { item: offer } = useCourseProduct({
+  const { item: offering } = useCourseProduct({
     product_id: order.product_id,
     course_id: course.code,
   });
-  const { product } = offer || {};
+  const { product } = offering || {};
   const needsSignature = OrderHelper.orderNeedsSignature(order);
   const needsPaymentMethod = order.state === OrderState.TO_SAVE_PAYMENT_METHOD;
   const isActive = OrderHelper.isActive(order);
-  const isProductPurchasable = ProductHelper.isPurchasable(offer?.product);
+  const isProductPurchasable = ProductHelper.isPurchasable(offering?.product);
   const isNotResumable = !isActive && !isProductPurchasable;
   const canEnroll = OrderHelper.allowEnrollment(order);
 

package/js/widgets/Dashboard/components/DashboardItem/Order/DashboardItemOrderContract.useUnionResource.cache.spec.tsx

@@ -57,7 +57,7 @@ describe('<DashboardItemOrder/> Contract', () => {
         contract: ContractFactory({ student_signed_on: null }).one(),
       }).one();
 
-      // learner dashboard course page do one call to offer per order
+      // learner dashboard course page do one call to offering per order
       const { product } = mockCourseProductWithOrder(order);
 
       // overwrite useOmniscientOrders call

package/js/widgets/Dashboard/components/DashboardItem/Order/Installment/index.tsx

@@ -94,7 +94,7 @@ const Installment = ({ order }: Props) => {
 
 const PaymentMethodManager = ({ order }: Props) => {
   const needsPaymentMethod = order.state === OrderState.TO_SAVE_PAYMENT_METHOD;
-  const { item: offer, states } = useCourseProduct({
+  const { item: offering, states } = useCourseProduct({
     course_id: order.course.code,
     product_id: order.product_id,
   });
@@ -118,9 +118,9 @@ const PaymentMethodManager = ({ order }: Props) => {
       )}
       <SaleTunnel
         {...modal}
-        product={offer.product as CredentialProduct}
-        course={offer.course}
-        isWithdrawable={offer.is_withdrawable}
+        product={offering.product as CredentialProduct}
+        course={offering.course}
+        isWithdrawable={offering.is_withdrawable}
       />
     </>
   );

package/js/widgets/Dashboard/components/DashboardItem/stories.mock.ts

@@ -25,6 +25,6 @@ export const enrollment: Enrollment = {
     },
     languages: ['en'],
   },
-  offers: [],
+  offerings: [],
   certificate_id: null,
 };

package/js/widgets/Dashboard/components/DashboardSidebar/components/ContractNavLink/index.spec.tsx

@@ -28,7 +28,7 @@ describe('<ContractNavLink />', () => {
     fetchMock.get('https://joanie.endpoint/api/v1.0/organizations/', []);
   });
 
-  it('should render a ContractNavLink with route and label when neither organizationId and offerId are given', () => {
+  it('should render a ContractNavLink with route and label when neither organizationId and offeringId are given', () => {
     const link: MenuLink = {
       to: '/dummy/url/',
       label: 'My contract navigation link',
@@ -45,31 +45,31 @@ describe('<ContractNavLink />', () => {
     it.each([
       {
         organizationId: faker.string.uuid(),
-        offerId: undefined,
+        offeringId: undefined,
       },
       {
         organizationId: faker.string.uuid(),
-        offerId: faker.string.uuid(),
+        offeringId: faker.string.uuid(),
       },
       {
         organizationId: undefined,
-        offerId: faker.string.uuid(),
+        offeringId: faker.string.uuid(),
       },
       {
         organizationId: undefined,
-        offerId: undefined,
+        offeringId: undefined,
       },
     ])(
-      'should never render Badge for organizationId: $organizationId and courseProductId: $offerId',
-      async ({ organizationId, offerId }) => {
+      'should never render Badge for organizationId: $organizationId and courseProductId: $offeringId',
+      async ({ organizationId, offeringId }) => {
         let contractQueryParams: ContractResourceQuery = {
           signature_state: ContractState.LEARNER_SIGNED,
           page: 1,
           page_size: PER_PAGE.teacherContractList,
         };
-        if (offerId) {
+        if (offeringId) {
           contractQueryParams = {
-            offer_id: offerId,
+            offering_id: offeringId,
             ...contractQueryParams,
           };
         }
@@ -94,7 +94,7 @@ describe('<ContractNavLink />', () => {
               label: 'My contract navigation link',
             }}
             organizationId={organizationId}
-            offerId={offerId}
+            offeringId={offeringId}
           />,
         );
 
@@ -112,25 +112,25 @@ describe('<ContractNavLink />', () => {
       // with 1 contracts to sign
       {
         organizationId: faker.string.uuid(),
-        offerId: undefined,
+        offeringId: undefined,
         nbContractsToSign: 1,
         expectedBadgeCount: 1,
       },
       {
         organizationId: faker.string.uuid(),
-        offerId: faker.string.uuid(),
+        offeringId: faker.string.uuid(),
         nbContractsToSign: 1,
         expectedBadgeCount: 1,
       },
       {
         organizationId: undefined,
-        offerId: faker.string.uuid(),
+        offeringId: faker.string.uuid(),
         nbContractsToSign: 1,
         expectedBadgeCount: undefined,
       },
       {
         organizationId: undefined,
-        offerId: undefined,
+        offeringId: undefined,
         nbContractsToSign: 1,
         expectedBadgeCount: undefined,
       },
@@ -138,39 +138,39 @@ describe('<ContractNavLink />', () => {
       // with 0 contracts to sign
       {
         organizationId: faker.string.uuid(),
-        offerId: undefined,
+        offeringId: undefined,
         nbContractsToSign: 0,
         expectedBadgeCount: undefined,
       },
       {
         organizationId: faker.string.uuid(),
-        offerId: faker.string.uuid(),
+        offeringId: faker.string.uuid(),
         nbContractsToSign: 0,
         expectedBadgeCount: undefined,
       },
       {
         organizationId: undefined,
-        offerId: faker.string.uuid(),
+        offeringId: faker.string.uuid(),
         nbContractsToSign: 0,
         expectedBadgeCount: undefined,
       },
       {
         organizationId: undefined,
-        offerId: undefined,
+        offeringId: undefined,
         nbContractsToSign: 0,
         expectedBadgeCount: undefined,
       },
     ])(
-      'should render Badge (count: $expectedBadgeCount) for nb contracts to sign: $nbContractsToSign, organizationId: $organizationId and courseProductId: $offerId',
-      async ({ nbContractsToSign, organizationId, offerId, expectedBadgeCount }) => {
+      'should render Badge (count: $expectedBadgeCount) for nb contracts to sign: $nbContractsToSign, organizationId: $organizationId and courseProductId: $offeringId',
+      async ({ nbContractsToSign, organizationId, offeringId, expectedBadgeCount }) => {
         let contractQueryParams: ContractResourceQuery = {
           signature_state: ContractState.LEARNER_SIGNED,
           page: 1,
           page_size: PER_PAGE.teacherContractList,
         };
-        if (offerId) {
+        if (offeringId) {
           contractQueryParams = {
-            offer_id: offerId,
+            offering_id: offeringId,
             ...contractQueryParams,
           };
         }
@@ -194,7 +194,7 @@ describe('<ContractNavLink />', () => {
               label: 'My contract navigation link',
             }}
             organizationId={organizationId}
-            offerId={offerId}
+            offeringId={offeringId}
           />,
         );