PrivateCoffee/element-web
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 3

Review clean

Browse source
This commit is contained in:
Florian Duros 2022-10-26 19:05:11 +02:00
parent d25cbb53b0
commit 92b4c8bf4e
No known key found for this signature in database
GPG key ID: 9700AA5870258A0B
1 changed files with 2 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
test/components/views/rooms/wysiwyg_composer/SendWysiwygComposer-test.tsx
View file

@ -24,7 +24,7 @@ import RoomContext from "../../../../../src/contexts/RoomContext";
import defaultDispatcher from "../../../../../src/dispatcher/dispatcher"; import defaultDispatcher from "../../../../../src/dispatcher/dispatcher";
import { Action } from "../../../../../src/dispatcher/actions"; import { Action } from "../../../../../src/dispatcher/actions";
import { IRoomState } from "../../../../../src/components/structures/RoomView"; import { IRoomState } from "../../../../../src/components/structures/RoomView";
import { createTestClient, getRoomContext, mkEvent, mkStubRoom } from "../../../../test-utils"; import { createTestClient, flushPromises, getRoomContext, mkEvent, mkStubRoom } from "../../../../test-utils";
import { SendWysiwygComposer } from "../../../../../src/components/views/rooms/wysiwyg_composer"; import { SendWysiwygComposer } from "../../../../../src/components/views/rooms/wysiwyg_composer";
import * as useComposerFunctions import * as useComposerFunctions
from "../../../../../src/components/views/rooms/wysiwyg_composer/hooks/useComposerFunctions"; from "../../../../../src/components/views/rooms/wysiwyg_composer/hooks/useComposerFunctions";
@ -110,9 +110,6 @@ describe('SendWysiwygComposer', () => {
it('Should focus when receiving an Action.FocusSendMessageComposer action', async () => { it('Should focus when receiving an Action.FocusSendMessageComposer action', async () => {
// Given we don't have focus // Given we don't have focus
customRender(jest.fn(), jest.fn(), false, isRichTextEnabled); customRender(jest.fn(), jest.fn(), false, isRichTextEnabled);
document.head.focus();
// screen.getByLabelText('Bold').focus();
// expect(screen.getByRole('textbox')).not.toHaveFocus();
// When we send the right action // When we send the right action
defaultDispatcher.dispatch({ defaultDispatcher.dispatch({
@ -129,8 +126,6 @@ describe('SendWysiwygComposer', () => {
const mock = jest.spyOn(useComposerFunctions, 'useComposerFunctions'); const mock = jest.spyOn(useComposerFunctions, 'useComposerFunctions');
mock.mockReturnValue({ clear: mockClear }); mock.mockReturnValue({ clear: mockClear });
customRender(jest.fn(), jest.fn(), false, isRichTextEnabled); customRender(jest.fn(), jest.fn(), false, isRichTextEnabled);
// screen.getByLabelText('Bold').focus();
// expect(screen.getByRole('textbox')).not.toHaveFocus();
// When we send the right action // When we send the right action
defaultDispatcher.dispatch({ defaultDispatcher.dispatch({
@ -148,8 +143,6 @@ describe('SendWysiwygComposer', () => {
it('Should focus when receiving a reply_to_event action', async () => { it('Should focus when receiving a reply_to_event action', async () => {
// Given we don't have focus // Given we don't have focus
customRender(jest.fn(), jest.fn(), false, isRichTextEnabled); customRender(jest.fn(), jest.fn(), false, isRichTextEnabled);
// screen.getByLabelText('Bold').focus();
// expect(screen.getByRole('textbox')).not.toHaveFocus();
// When we send the right action // When we send the right action
defaultDispatcher.dispatch({ defaultDispatcher.dispatch({
@ -178,7 +171,7 @@ describe('SendWysiwygComposer', () => {
}); });
// Wait for event dispatch to happen // Wait for event dispatch to happen
await new Promise((r) => setTimeout(r, 200)); await flushPromises();
// Then we don't get it because we are disabled // Then we don't get it because we are disabled
expect(screen.getByRole('textbox')).not.toHaveFocus(); expect(screen.getByRole('textbox')).not.toHaveFocus();