fix missing props in tests

components/sidebar/sidebar.jsx

@@ -76,12 +76,12 @@ export default class Sidebar extends React.PureComponent {
  /**
  * Current team object
  */
- currentTeam: PropTypes.object.isRequired,
+ currentTeam: PropTypes.object,
 
  /**
  * Current user object
  */
- currentUser: PropTypes.object.isRequired,
+ currentUser: PropTypes.object,
 
  /**
  * Number of unread mentions/messages

tests/components/__snapshots__/channel_notifications_modal.test.jsx.snap

@@ -68,6 +68,7 @@ exports[`components/channel_notifications_modal/ChannelNotificationsModal should
  />
  <NotificationSection
  expand={false}
+ memberNotificationLevel="all"
  onChange={[Function]}
  onSubmit={[Function]}
  onUpdateSection={[Function]}

tests/components/channel_notifications_modal.test.jsx

@@ -10,8 +10,18 @@ describe('components/channel_notifications_modal/ChannelNotificationsModal', ()
  show: true,
  onHide: () => {}, //eslint-disable-line no-empty-function
  channel: {id: 'channel_id', display_name: 'channel_display_name'},
- channelMember: {notify_props: {desktop: NotificationLevels.ALL}},
- currentUser: {id: 'current_user_id', notify_props: {desktop: NotificationLevels.ALL}},
+ channelMember: {
+ notify_props: {
+ desktop: NotificationLevels.ALL,
+ mark_unread: NotificationLevels.ALL,
+ },
+ },
+ currentUser: {
+ id: 'current_user_id',
+ notify_props: {
+ desktop: NotificationLevels.ALL,
+ },
+ },
  sendPushNotifications: true,
  actions: {
  updateChannelNotifyProps: () => {}, //eslint-disable-line no-empty-function
@@ -99,7 +109,12 @@ describe('components/channel_notifications_modal/ChannelNotificationsModal', ()
  });
 
  test('should match state on handleSubmitMarkUnreadLevel', () => {
- const channelMember = {notify_props: {mark_unread: NotificationLevels.ALL}};
+ const channelMember = {
+ notify_props: {
+ desktop: NotificationLevels.NONE,
+ mark_unread: NotificationLevels.ALL,
+ },
+ };
  const props = {...baseProps, channelMember};
  const wrapper = shallow(
  <ChannelNotificationsModal {...props}/>
@@ -120,7 +135,12 @@ describe('components/channel_notifications_modal/ChannelNotificationsModal', ()
  });
 
  test('should match state on handleUpdateMarkUnreadLevel', () => {
- const channelMember = {notify_props: {mark_unread: NotificationLevels.ALL}};
+ const channelMember = {
+ notify_props: {
+ desktop: NotificationLevels.NONE,
+ mark_unread: NotificationLevels.ALL,
+ },
+ };
  const props = {...baseProps, channelMember};
  const wrapper = shallow(
  <ChannelNotificationsModal {...props}/>
@@ -132,7 +152,12 @@ describe('components/channel_notifications_modal/ChannelNotificationsModal', ()
  });
 
  test('should match state on handleUpdateMarkUnreadSection', () => {
- const channelMember = {notify_props: {mark_unread: NotificationLevels.ALL}};
+ const channelMember = {
+ notify_props: {
+ desktop: NotificationLevels.NONE,
+ mark_unread: NotificationLevels.ALL,
+ },
+ };
  const props = {...baseProps, channelMember};
  const wrapper = shallow(
  <ChannelNotificationsModal {...props}/>
@@ -154,7 +179,13 @@ describe('components/channel_notifications_modal/ChannelNotificationsModal', ()
  });
 
  test('should match state on handleSubmitPushNotificationLevel', () => {
- const channelMember = {notify_props: {push: NotificationLevels.ALL}};
+ const channelMember = {
+ notify_props: {
+ desktop: NotificationLevels.NONE,
+ mark_unread: NotificationLevels.NONE,
+ push: NotificationLevels.ALL,
+ },
+ };
  const props = {...baseProps, channelMember};
  const wrapper = shallow(
  <ChannelNotificationsModal {...props}/>
@@ -175,7 +206,13 @@ describe('components/channel_notifications_modal/ChannelNotificationsModal', ()
  });
 
  test('should match state on handleUpdatePushNotificationLevel', () => {
- const channelMember = {notify_props: {push: NotificationLevels.ALL}};
+ const channelMember = {
+ notify_props: {
+ desktop: NotificationLevels.NONE,
+ mark_unread: NotificationLevels.NONE,
+ push: NotificationLevels.ALL,
+ },
+ };
  const props = {...baseProps, channelMember};
  const wrapper = shallow(
  <ChannelNotificationsModal {...props}/>
@@ -187,7 +224,14 @@ describe('components/channel_notifications_modal/ChannelNotificationsModal', ()
  });
 
  test('should match state on handleUpdatePushSection', () => {
- const channelMember = {notify_props: {push: NotificationLevels.ALL}};
+ const channelMember = {
+ notify_props: {
+ desktop: NotificationLevels.NONE,
+ mark_unread: NotificationLevels.NONE,
+ push: NotificationLevels.ALL,
+ },
+ };
+
  const props = {...baseProps, channelMember};
  const wrapper = shallow(
  <ChannelNotificationsModal {...props}/>

tests/components/create_comment/create_comment.test.jsx

@@ -18,6 +18,7 @@ describe('components/CreateComment', () => {
  const baseProps = {
  channelId,
  rootId,
+ rootDeleted: false,
  channelMembersCount: 3,
  draft: {
  message: 'Test message',

tests/components/dot_menu/dot_menu_empty.test.jsx

@@ -32,6 +32,7 @@ describe('components/dot_menu/DotMenu returning empty ("")', () => {
  setEditingPost: jest.fn(),
  pinPost: jest.fn(),
  unpinPost: jest.fn(),
+ openModal: jest.fn(),
  },
  };
 
@@ -52,6 +53,7 @@ describe('components/dot_menu/DotMenu returning empty ("")', () => {
  setEditingPost: jest.fn(),
  pinPost: jest.fn(),
  unpinPost: jest.fn(),
+ openModal: jest.fn(),
  },
  };
 
@@ -72,6 +74,7 @@ describe('components/dot_menu/DotMenu returning empty ("")', () => {
  setEditingPost: jest.fn(),
  pinPost: jest.fn(),
  unpinPost: jest.fn(),
+ openModal: jest.fn(),
  },
  };
 

tests/components/dot_menu/dot_menu_mobile.test.jsx

@@ -32,6 +32,7 @@ describe('components/dot_menu/DotMenu on mobile view', () => {
  setEditingPost: jest.fn(),
  pinPost: jest.fn(),
  unpinPost: jest.fn(),
+ openModal: jest.fn(),
  },
  };
 

tests/components/navbar/navbar.test.jsx

@@ -12,6 +12,11 @@ describe('components/navbar/Navbar', () => {
  isPinnedPosts: true,
  actions: {
  showEditChannelHeaderModal: jest.fn(),
+ toggleLhs: jest.fn(),
+ closeLhs: jest.fn(),
+ closeRhs: jest.fn(),
+ toggleRhsMenu: jest.fn(),
+ closeRhsMenu: jest.fn(),
  },
  isLicensed: true,
  enableWebrtc: true,

tests/components/sidebar/sidebar.test.jsx

@@ -60,6 +60,7 @@ describe('component/sidebar/sidebar_channel/SidebarChannel', () => {
  EnableXToLeaveChannelsFromLHS: 'false',
  SiteName: 'Test site',
  },
+ isOpen: false,
  showUnreadSection: false,
  publicChannelIds: ['c1', 'c2'],
  privateChannelIds: ['c3', 'c4'],
@@ -108,6 +109,7 @@ describe('component/sidebar/sidebar_channel/SidebarChannel', () => {
  },
  actions: {
  goToChannelById: jest.fn(),
+ close: jest.fn(),
  },
  };
 

tests/components/sidebar/sidebar_channel.test.jsx

@@ -61,6 +61,7 @@ describe('component/sidebar/sidebar_channel/SidebarChannel', () => {
  offTopicDisplayName: 'Off-Topic',
  membersCount: 8,
  showUnreadForMsgs: true,
+ shouldHideChannel: false,
  actions: {
  savePreferences: jest.fn(),
  leaveChannel: jest.fn(),