fix: test logic to call commerce-coordinator

Author: Diane Kaplan

.env

@@ -2,6 +2,7 @@ NODE_ENV='production'
 BASE_URL=''
 LMS_BASE_URL=''
 CREDENTIALS_BASE_URL=''
+COMMERCE_COORDINATOR_BASE_URL=''
 ECOMMERCE_BASE_URL=''
 LOGIN_URL=''
 LOGOUT_URL=''

.env.development

@@ -3,6 +3,7 @@ NODE_ENV=development
 BASE_URL=localhost:1996
 LMS_BASE_URL=http://localhost:18000
 CREDENTIALS_BASE_URL=http://localhost:18150
+COMMERCE_COORDINATOR_BASE_URL=http://localhost:8140
 ECOMMERCE_BASE_URL=http://localhost:18130
 LOGIN_URL=http://localhost:18000/login
 LOGOUT_URL=http://localhost:18000/login

.env.test

@@ -3,6 +3,7 @@ NODE_ENV=development
 BASE_URL=localhost:1996
 LMS_BASE_URL=http://localhost:18000
 CREDENTIALS_BASE_URL=http://localhost:18150
+COMMERCE_COORDINATOR_BASE_URL=http://localhost:8140
 ECOMMERCE_BASE_URL=http://localhost:18130
 LOGIN_URL=http://localhost:18000/login
 LOGOUT_URL=http://localhost:18000/login

src/order-history/service.js

@@ -1,23 +1,50 @@
 import { getAuthenticatedHttpClient, getAuthenticatedUser } from '@edx/frontend-platform/auth';
-import { getConfig } from '@edx/frontend-platform';
+import { getConfig, initialize, mergeConfig } from '@edx/frontend-platform';
 
 const { ECOMMERCE_BASE_URL } = getConfig();
+
+// Temporary: until we add COMMERCE_COORDINATOR_BASE_URL in frontend-platform, use mergeConfig to join it
+// with the rest of the config items (so we don't need to get it separately from process.env). After we add
+// it to frontend-platform, we'll be able to set it like ECOMMERCE_BASE_URL above, and block below can go away.
+initialize({
+  handlers: {
+    config: () => {
+      mergeConfig({
+        COMMERCE_COORDINATOR_BASE_URL: process.env.COMMERCE_COORDINATOR_BASE_URL || null,
+      });
+    },
+  },
+});
+
 const ECOMMERCE_API_BASE_URL = `${ECOMMERCE_BASE_URL}/api/v2`;
 const ECOMMERCE_RECEIPT_BASE_URL = `${ECOMMERCE_BASE_URL}/checkout/receipt/`;
 
 // eslint-disable-next-line import/prefer-default-export
 export async function getOrders(page = 1, pageSize = 20) {
   const httpClient = getAuthenticatedHttpClient();
   const { username } = getAuthenticatedUser();
+  const { COMMERCE_COORDINATOR_BASE_URL } = getConfig();
 
-  const { data } = await httpClient.get(`${ECOMMERCE_API_BASE_URL}/orders/`, {
+  // @@TODO: replace with real waffle when that's available
+  const callEcommerce = true; // <- false makes it call commerce-coordinator
+  let orderFetchingUrl = null;
+  if (callEcommerce) {
+    orderFetchingUrl = `${ECOMMERCE_API_BASE_URL}/orders/`;
+  } else {
+    orderFetchingUrl = `${COMMERCE_COORDINATOR_BASE_URL}/orders/ecommerce/`;
+  }
+
+  const { data } = await httpClient.get(orderFetchingUrl, {
     params: {
       username,
       page,
       page_size: pageSize,
     },
   });
 
+  console.log('DKTEMP: orderFetchingUrl', orderFetchingUrl);
+  console.log('DKTEMP: data service.js', data);
+
   const transformedResults = data.results.map(({
     total_excl_tax, // eslint-disable-line camelcase
     lines,