Skip to content

Commit

Permalink
feat(MeetingsAdapter): addLocalMedia is removed from the interface
Browse files Browse the repository at this point in the history
  • Loading branch information
akoushke committed Jan 29, 2020
1 parent f640a11 commit 42777db
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 28 deletions.
14 changes: 0 additions & 14 deletions src/MeetingsAdapter.js
Original file line number Diff line number Diff line change
Expand Up @@ -146,18 +146,4 @@ export default class MeetingsAdapter extends WebexAdapter {
leaveMeeting(ID) {
throw new Error('leaveMeeting(ID) must be defined in MeetingsAdapter');
}

/**
* Retrieves the local device media (video/audio) and adds them to the meeting
* with the some default media settings.
* Adding local media is performed as a side-effect and this method does not
* return a value. Instead, adding local media to a meeting should trigger
* getMeeting to emit a new updated Meeting object.
*
* @param {string} ID ID of the meeting to add the local media to.
* @memberof MeetingsAdapter
*/
addLocalMedia(ID) {
throw new Error('addMediaLocal(ID) must be defined in MeetingsAdapter');
}
}
14 changes: 0 additions & 14 deletions src/MeetingsAdapter.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -64,18 +64,4 @@ describe('Meetings Adapter Interface', () => {
}
});
});

describe('addLocalMedia()', () => {
test('errors because it needs to be defined', () => {
try {
meetingsAdapter.addLocalMedia('meetingID');
} catch (error) {
expect(error.message).toBe('addMediaLocal(ID) must be defined in MeetingsAdapter');
}
});
});

test('meetingControls property exists', () => {
expect(meetingsAdapter).toHaveProperty('meetingControls');
});
});

0 comments on commit 42777db

Please sign in to comment.