diff options
author | Joas Schilling <coding@schilljs.com> | 2022-05-03 11:45:12 +0300 |
---|---|---|
committer | Joas Schilling <coding@schilljs.com> | 2022-05-03 11:45:12 +0300 |
commit | baedde903cc93c97201ed15cfb5829525dddd931 (patch) | |
tree | 77cf3465f69b6b251577cd6d40d9d96948acbe8d | |
parent | 84220d5b486c4e9f4ffde014e1ca8014ce77e8e2 (diff) |
Fix eslint warnings
Signed-off-by: Joas Schilling <coding@schilljs.com>
-rw-r--r-- | src/components/CallView/shared/Video.spec.js | 24 | ||||
-rw-r--r-- | src/maps.js | 8 | ||||
-rw-r--r-- | src/utils/webrtc/RemoteVideoBlocker.spec.js | 2 |
3 files changed, 28 insertions, 6 deletions
diff --git a/src/components/CallView/shared/Video.spec.js b/src/components/CallView/shared/Video.spec.js index 224a96521..9ad534aa6 100644 --- a/src/components/CallView/shared/Video.spec.js +++ b/src/components/CallView/shared/Video.spec.js @@ -36,6 +36,9 @@ describe('Video.vue', () => { let callParticipantModel + /** + * Constructor + */ function PeerMock() { this._superEmitterMixin() @@ -104,11 +107,13 @@ describe('Video.vue', () => { let wrapper - // "setupWrapper()" needs to be called right before checking the wrapper - // to ensure that the component state is updated. If the wrapper is - // created at the beginning of each test "await Vue.nextTick()" would - // need to be called instead (and for that the tests would need to be - // async). + /** + * "setupWrapper()" needs to be called right before checking the wrapper + * to ensure that the component state is updated. If the wrapper is + * created at the beginning of each test "await Vue.nextTick()" would + * need to be called instead (and for that the tests would need to be + * async). + */ function setupWrapper() { wrapper = shallowMount(Video, { localVue, @@ -126,6 +131,9 @@ describe('Video.vue', () => { }) } + /** + * @param {string} expectedText Expected connection label + */ function assertConnectionMessageLabel(expectedText) { const connectionMessageLabel = wrapper.find('.connection-message') if (expectedText) { @@ -136,11 +144,17 @@ describe('Video.vue', () => { } } + /** + * @param {boolean} expected Whether the loading icon is shown + */ function assertLoadingIconIsShown(expected) { const loadingIcon = wrapper.find('.icon-loading') expect(loadingIcon.exists()).toBe(expected) } + /** + * @param {boolean} expected Whether the connection is not connected + */ function assertNotConnected(expected) { const notConnected = wrapper.find('.not-connected') expect(notConnected.exists()).toBe(expected) diff --git a/src/maps.js b/src/maps.js index 16f902cda..d226c80e1 100644 --- a/src/maps.js +++ b/src/maps.js @@ -29,6 +29,11 @@ import { postRichObjectToConversation } from './services/messagesService' import RoomSelector from './views/RoomSelector' (function(OC, OCA, t, n) { + /** + * @param {object} location Geo location object + * @param {string} token Conversation token to be posted to + * @return {Promise<void>} + */ async function postLocationToRoom(location, token) { try { const response = await postRichObjectToConversation(token, { @@ -53,6 +58,9 @@ import RoomSelector from './views/RoomSelector' } } + /** + * Initialise the maps action + */ function init() { if (!OCA.Maps?.registerMapsAction) { return diff --git a/src/utils/webrtc/RemoteVideoBlocker.spec.js b/src/utils/webrtc/RemoteVideoBlocker.spec.js index 700b2918b..af9e0701c 100644 --- a/src/utils/webrtc/RemoteVideoBlocker.spec.js +++ b/src/utils/webrtc/RemoteVideoBlocker.spec.js @@ -29,7 +29,7 @@ describe('RemoteVideoBlocker', () => { jest.useFakeTimers() callParticipantModel = { - setVideoBlocked: jest.fn() + setVideoBlocked: jest.fn(), } remoteVideoBlocker = new RemoteVideoBlocker(callParticipantModel) |