test: updated tests

Author: eemaanamir

src/notification-preferences/NotificationPreferences.test.jsx

@@ -5,7 +5,7 @@ import { BrowserRouter as Router } from 'react-router-dom';
 
 import * as auth from '@edx/frontend-platform/auth';
 import { IntlProvider } from '@edx/frontend-platform/i18n';
-import { fireEvent, render, screen } from '@testing-library/react';
+import { render, screen } from '@testing-library/react';
 
 import { defaultState } from './data/reducers';
 import NotificationPreferences from './NotificationPreferences';
@@ -77,7 +77,7 @@ const setupStore = (override = {}) => {
   storeState.courses = {
     status: SUCCESS_STATUS,
     courses: [
-      { id: '', name: 'Account' },
+      { id: '', name: 'Select Course' },
       { id: 'selected-course-id', name: 'Selected Course' },
     ],
   };

src/notification-preferences/data/reducers.test.js

@@ -44,7 +44,7 @@ describe('notification-preferences reducer', () => {
     );
     expect(result.courses).toEqual({
       status: SUCCESS_STATUS,
-      courses: [{ id: '', name: 'Account' }],
+      courses: [{ id: '', name: 'Select Course' }],
       pagination: data.pagination,
     });
   });
@@ -61,7 +61,7 @@ describe('notification-preferences reducer', () => {
       status,
       courses: [{
         id: '',
-        name: 'Account',
+        name: 'Select Course',
       }],
       pagination: {},
     });

src/notification-preferences/data/utils.js

@@ -1,6 +1,6 @@
 import { getConfig } from '@edx/frontend-platform';
 
-export const notificationChannels = () => ((getConfig().SHOW_EMAIL_CHANNEL === 'true' && { EMAIL: 'email' }));
+export const notificationChannels = () => ({ WEB: 'web', ...(getConfig().SHOW_EMAIL_CHANNEL === 'true' && { EMAIL: 'email' }) });
 
 export const shouldHideAppPreferences = (preferences, appId) => {
   const appPreferences = preferences.filter(pref => pref.appId === appId);