Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Commit

Permalink
fix leaking registration test
Browse files Browse the repository at this point in the history
Signed-off-by: Kerry Archibald <kerrya@element.io>
  • Loading branch information
Kerry Archibald committed Dec 7, 2021
1 parent 3a85d68 commit 88189e5
Showing 1 changed file with 12 additions and 5 deletions.
17 changes: 12 additions & 5 deletions test/components/structures/auth/Registration-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,12 +14,18 @@ See the License for the specific language governing permissions and
limitations under the License.
*/

import sdk from '../../../skinned-sdk';

import React from 'react';
import ReactDOM from 'react-dom';
import ReactTestUtils from 'react-dom/test-utils';
import sdk from '../../../skinned-sdk';
import { createClient } from 'matrix-js-sdk/src/matrix';

import SdkConfig from '../../../../src/SdkConfig';
import { mkServerConfig } from "../../../test-utils";
import { createTestClient, mkServerConfig } from "../../../test-utils";

jest.mock('matrix-js-sdk/src/matrix');
jest.useFakeTimers();

const Registration = sdk.getComponent(
'structures.auth.Registration',
Expand All @@ -31,6 +37,7 @@ describe('Registration', function() {
beforeEach(function() {
parentDiv = document.createElement('div');
document.body.appendChild(parentDiv);
createClient.mockImplementation(() => createTestClient());
});

afterEach(function() {
Expand All @@ -48,13 +55,13 @@ describe('Registration', function() {
/>, parentDiv);
}

it('should show server picker', function() {
it('should show server picker', async function() {
const root = render();
const selector = ReactTestUtils.findRenderedDOMComponentWithClass(root, "mx_ServerPicker");
expect(selector).toBeTruthy();
});

it('should show form when custom URLs disabled', function() {
it('should show form when custom URLs disabled', async function() {
jest.spyOn(SdkConfig, "get").mockReturnValue({
disable_custom_urls: true,
});
Expand All @@ -77,7 +84,7 @@ describe('Registration', function() {
expect(form).toBeTruthy();
});

it("should show SSO options if those are available", () => {
it("should show SSO options if those are available", async () => {
jest.spyOn(SdkConfig, "get").mockReturnValue({
disable_custom_urls: true,
});
Expand Down

0 comments on commit 88189e5

Please sign in to comment.