Skip to content

Commit c2ebf7b

Browse files
authored
feat: update order number href for entitlements (#451)
1 parent 3e2761a commit c2ebf7b

File tree

2 files changed

+16
-1
lines changed

2 files changed

+16
-1
lines changed

src/users/entitlements/Entitlements.jsx

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,11 @@ export default function Entitlements({
9292
modified: entitlement.modified,
9393
orderNumber: (
9494
<a
95-
href={`${getConfig().ECOMMERCE_BASE_URL}/dashboard/orders/${entitlement.orderNumber}/`}
95+
href={
96+
getConfig().COMMERCE_COORDINATOR_ORDER_DETAILS_URL
97+
? `${getConfig().COMMERCE_COORDINATOR_ORDER_DETAILS_URL}/?order_number=${entitlement.orderNumber}`
98+
: `${getConfig().ECOMMERCE_BASE_URL}/dashboard/orders/${entitlement.orderNumber}`
99+
}
96100
rel="noopener noreferrer"
97101
target="_blank"
98102
>

src/users/entitlements/Entitlements.test.jsx

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,13 @@ const EntitlementsPageWrapper = (props) => (
1414
</UserMessagesProvider>
1515
);
1616

17+
jest.mock('@edx/frontend-platform', () => ({
18+
...jest.requireActual('@edx/frontend-platform'),
19+
getConfig: jest.fn(() => ({
20+
ECOMMERCE_BASE_URL: 'http://example.com',
21+
})),
22+
}));
23+
1724
describe('Entitlements Listing', () => {
1825
let apiMock;
1926
let wrapper;
@@ -108,6 +115,10 @@ describe('Entitlements Listing', () => {
108115
waitFor(() => expect(componentHeader.text()).toEqual('Entitlements (1)'));
109116
});
110117

118+
it('Renders correct href for Order Number', async () => {
119+
expect(/http:\/\/example.com\/dashboard\/orders\/123edX456789/.test(wrapper.html())).toBeTruthy();
120+
});
121+
111122
describe('Expire Entitlement button', () => {
112123
it('Disabled Expire entitlement button', async () => {
113124
// We're only checking row 0 of the table since it has the button Expire Button disabled

0 commit comments

Comments
 (0)