Skip to content

Commit 268ccc8

Browse files
committed
fix: a couple of appId imports
1 parent 2045854 commit 268ccc8

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

site.config.test.tsx

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
import { EnvironmentTypes, SiteConfig } from '@openedx/frontend-base';
22

3+
import { appId } from './src/constants';
4+
35
const siteConfig: SiteConfig = {
46
siteId: 'learner-dashboard-test-site',
57
siteName: 'Learner Dashboard Test Site',
@@ -11,7 +13,7 @@ const siteConfig: SiteConfig = {
1113
environment: EnvironmentTypes.TEST,
1214
basename: '/learner-dashboard',
1315
apps: [{
14-
appId: 'org.openedx.frontend.app.learnerDashboard',
16+
appId,
1517
config: {
1618
ECOMMERCE_BASE_URL: 'http://localhost:18130',
1719
FAVICON_URL: 'https://edx-cdn.org/v3/default/favicon.ico',

src/data/services/lms/urls.test.js

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import { getAppConfig, getSiteConfig } from '@openedx/frontend-base';
2+
import { appId } from '../../../constants';
23
import * as urls from './urls';
34

45
describe('urls', () => {
@@ -25,7 +26,7 @@ describe('urls', () => {
2526
it('returns the url if it is relative', () => {
2627
const url = '/edx.org';
2728
expect(urls.learningMfeUrl(url)).toEqual(
28-
`${getAppConfig('org.openedx.frontend.app.learnerDashboard').LEARNING_BASE_URL}${url}`,
29+
`${getAppConfig(appId).LEARNING_BASE_URL}${url}`,
2930
);
3031
});
3132
it('return null if url is null', () => {

0 commit comments

Comments
 (0)