Merge pull request #994 from openedx/ea/ent-7216-2

chore: specify learner_credit subsidy type in useEnterpriseOffers
openedx · Jun 1, 2023 · 313171a · 313171a
2 parents de062e8 + d363452
commit 313171a
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 4 deletions.
diff --git a/src/components/EnterpriseSubsidiesContext/data/hooks.js b/src/components/EnterpriseSubsidiesContext/data/hooks.js
@@ -18,7 +18,7 @@ export const useEnterpriseOffers = ({ enablePortalLearnerCreditManagementScreen,
     const fetchOffers = async () => {
       try {
         const [enterpriseSubsidyResponse, ecommerceApiResponse] = await Promise.all([
-          SubsidyApiService.getSubsidyByCustomerUUID(enterpriseId),
+          SubsidyApiService.getSubsidyByCustomerUUID(enterpriseId, { subsidyType: 'learner_credit' }),
           EcommerceApiService.fetchEnterpriseOffers({
             isCurrent: true,
           }),

diff --git a/src/components/EnterpriseSubsidiesContext/data/tests/hooks.test.js b/src/components/EnterpriseSubsidiesContext/data/tests/hooks.test.js
@@ -108,7 +108,10 @@ describe('useEnterpriseOffers', () => {
 
     await waitForNextUpdate();
 
-    expect(SubsidyApiService.getSubsidyByCustomerUUID).toHaveBeenCalledWith(TEST_ENTERPRISE_UUID);
+    expect(SubsidyApiService.getSubsidyByCustomerUUID).toHaveBeenCalledWith(
+      TEST_ENTERPRISE_UUID,
+      { subsidyType: 'learner_credit' },
+    );
     expect(result.current).toEqual({
       offers: mockOffers,
       isLoading: false,
@@ -157,7 +160,10 @@ describe('useEnterpriseOffers', () => {
 
     await waitForNextUpdate();
 
-    expect(SubsidyApiService.getSubsidyByCustomerUUID).toHaveBeenCalledWith(TEST_ENTERPRISE_UUID);
+    expect(SubsidyApiService.getSubsidyByCustomerUUID).toHaveBeenCalledWith(
+      TEST_ENTERPRISE_UUID,
+      { subsidyType: 'learner_credit' },
+    );
     expect(result.current).toEqual({
       offers: mockOfferData,
       isLoading: false,

diff --git a/src/data/services/EnterpriseSubsidyApiService.js b/src/data/services/EnterpriseSubsidyApiService.js
@@ -1,4 +1,5 @@
 import { getAuthenticatedHttpClient } from '@edx/frontend-platform/auth';
+import { snakeCaseObject } from '@edx/frontend-platform';
 
 import { configuration } from '../../config';
 
@@ -10,7 +11,7 @@ class SubsidyApiService {
   static getSubsidyByCustomerUUID(uuid, options = {}) {
     const queryParams = new URLSearchParams({
       enterprise_customer_uuid: uuid,
-      ...options,
+      ...snakeCaseObject(options),
     });
     const url = `${SubsidyApiService.baseUrl}/subsidies/?${queryParams.toString()}`;
     return SubsidyApiService.apiClient({