Restore all tests

Author: backspace

packages/matrix/tests/forgot-password.spec.ts

@@ -108,7 +108,7 @@ test.describe('Forgot password', () => {
     await assertLoggedIn(resetPasswordPage);
   });
 
-  test.skip('It shows an error when email does not belong to any account', async ({
+  test('It shows an error when email does not belong to any account', async ({
     page,
   }) => {
     await gotoForgotPassword(page);
@@ -147,7 +147,7 @@ test.describe('Forgot password', () => {
     );
   });
 
-  test.skip('It shows an error when password does not meet the requirement', async ({
+  test('It shows an error when password does not meet the requirement', async ({
     page,
   }) => {
     await gotoForgotPassword(page);
@@ -217,7 +217,7 @@ test.describe('Forgot password', () => {
     await resetPasswordPage.locator('[data-test-reset-password-btn]').click();
   });
 
-  test.skip('it can resend email validation message', async ({ page }) => {
+  test('it can resend email validation message', async ({ page }) => {
     await gotoForgotPassword(page);
 
     await expect(

packages/matrix/tests/login.spec.ts

@@ -32,7 +32,7 @@ test.describe('Login', () => {
     await synapseStop(synapse.synapseId);
   });
 
-  test.skip('it can login', async ({ page }) => {
+  test('it can login', async ({ page }) => {
     await openRoot(page);
     await toggleOperatorMode(page);
 
@@ -91,7 +91,7 @@ test.describe('Login', () => {
     await assertLoggedOut(page);
   });
 
-  test.skip('it can logout using the profile popover', async ({ page }) => {
+  test('it can logout using the profile popover', async ({ page }) => {
     await login(page, 'user1', 'pass');
 
     await expect(
@@ -107,7 +107,7 @@ test.describe('Login', () => {
     await expect(page.locator('[data-test-login-btn]')).toBeVisible();
   });
 
-  test.skip('it shows an error when invalid credentials are provided', async ({
+  test('it shows an error when invalid credentials are provided', async ({
     page,
   }) => {
     await openRoot(page);
@@ -133,7 +133,7 @@ test.describe('Login', () => {
     await assertLoggedIn(page);
   });
 
-  test.skip('it reacts to enter keypresses', async ({ page }) => {
+  test('it reacts to enter keypresses', async ({ page }) => {
     await openRoot(page);
     await toggleOperatorMode(page);
 
@@ -145,7 +145,7 @@ test.describe('Login', () => {
     await assertLoggedIn(page);
   });
 
-  test.skip('it returns to login when auth is invalid', async ({ page }) => {
+  test('it returns to login when auth is invalid', async ({ page }) => {
     await page.addInitScript({
       content: `
         window.localStorage.setItem(

packages/matrix/tests/messages.spec.ts

@@ -70,7 +70,7 @@ test.describe('Room messages', () => {
     await assertMessages(page, [{ from: 'user1', message: 'Message 1' }]);
   });
 
-  test.skip(`it can load all events back to beginning of timeline for timelines that truncated`, async ({
+  test(`it can load all events back to beginning of timeline for timelines that truncated`, async ({
     page,
   }) => {
     // generally the matrix server paginates after 10 messages
@@ -92,7 +92,7 @@ test.describe('Room messages', () => {
     );
   });
 
-  test.skip(`it can send a markdown message`, async ({ page }) => {
+  test(`it can send a markdown message`, async ({ page }) => {
     await login(page, 'user1', 'pass');
     let room1 = await getRoomId(page);
     await sendMessage(page, room1, 'message with _style_');
@@ -107,9 +107,7 @@ test.describe('Room messages', () => {
     ).toContainText('style');
   });
 
-  test.skip(`it can create a room specific pending message`, async ({
-    page,
-  }) => {
+  test(`it can create a room specific pending message`, async ({ page }) => {
     await login(page, 'user1', 'pass');
     let room1 = await getRoomId(page);
     await sendMessage(page, room1, 'Hello');
@@ -143,7 +141,7 @@ test.describe('Room messages', () => {
     ).toHaveValue('room 2 message');
   });
 
-  test.skip('can add a card to a markdown message', async ({ page }) => {
+  test('can add a card to a markdown message', async ({ page }) => {
     const testCard = `${testHost}/hassan`;
     await login(page, 'user1', 'pass');
     await page.locator(`[data-test-room-settled]`).waitFor();
@@ -176,7 +174,7 @@ test.describe('Room messages', () => {
     ).toContainText('my');
   });
 
-  test.skip('can add a card that is over 65K to a message (i.e. split card into multiple matrix events)', async ({
+  test('can add a card that is over 65K to a message (i.e. split card into multiple matrix events)', async ({
     page,
   }) => {
     const testCard = `${testHost}/big-card`; // this is a 153KB card
@@ -219,7 +217,7 @@ test.describe('Room messages', () => {
     ]);
   });
 
-  test.skip('it can strip out base64 image fields from cards sent in messages', async ({
+  test('it can strip out base64 image fields from cards sent in messages', async ({
     page,
   }) => {
     const testCard = `${testHost}/mango-puppy`; // this is a 153KB card
@@ -253,7 +251,7 @@ test.describe('Room messages', () => {
     expect(serializeCard.data.attributes.picture).toBeUndefined();
   });
 
-  test.skip(`it does include patch function in message event when top-most card is writable and context is shared`, async ({
+  test(`it does include patch function in message event when top-most card is writable and context is shared`, async ({
     page,
   }) => {
     await login(page, 'user1', 'pass');
@@ -312,7 +310,7 @@ test.describe('Room messages', () => {
     ]);
   });
 
-  test.skip(`it does not include patch function in message event for an open card that is not attached`, async ({
+  test(`it does not include patch function in message event for an open card that is not attached`, async ({
     page,
   }) => {
     await login(page, 'user1', 'pass');
@@ -337,7 +335,7 @@ test.describe('Room messages', () => {
     expect(boxelMessageData.context.functions).toMatchObject([]);
   });
 
-  test.skip(`it does not include patch function in message event when top-most card is read-only`, async ({
+  test(`it does not include patch function in message event when top-most card is read-only`, async ({
     page,
   }) => {
     // the base realm is a read-only realm
@@ -360,7 +358,7 @@ test.describe('Room messages', () => {
     expect(boxelMessageData.context.functions).toMatchObject([]);
   });
 
-  test.skip('can send only a card as a message', async ({ page }) => {
+  test('can send only a card as a message', async ({ page }) => {
     const testCard = `${testHost}/hassan`;
     await login(page, 'user1', 'pass');
     let room1 = await getRoomId(page);
@@ -373,9 +371,7 @@ test.describe('Room messages', () => {
     ]);
   });
 
-  test.skip('can send cards with types unsupported by matrix', async ({
-    page,
-  }) => {
+  test('can send cards with types unsupported by matrix', async ({ page }) => {
     const testCard = `${testHost}/type-examples`;
     await login(page, 'user1', 'pass');
     let room1 = await getRoomId(page);
@@ -390,7 +386,7 @@ test.describe('Room messages', () => {
     ]);
   });
 
-  test.skip('can remove a card from a pending message', async ({ page }) => {
+  test('can remove a card from a pending message', async ({ page }) => {
     const testCard = `${testHost}/hassan`;
     const testCard2 = `${testHost}/mango`;
     await login(page, 'user1', 'pass');
@@ -446,7 +442,7 @@ test.describe('Room messages', () => {
     ]);
   });
 
-  test.skip('can render multiple cards in a room', async ({ page }) => {
+  test('can render multiple cards in a room', async ({ page }) => {
     // the loader deadlocking issue would otherwise prevent this
 
     const testCard1 = `${testHost}/hassan`;
@@ -481,7 +477,7 @@ test.describe('Room messages', () => {
     await assertMessages(page, [message1, message2]);
   });
 
-  test.skip('can send multiple cards in a message', async ({ page }) => {
+  test('can send multiple cards in a message', async ({ page }) => {
     const testCard1 = `${testHost}/hassan`;
     const testCard2 = `${testHost}/mango`;
     const message = {
@@ -511,7 +507,7 @@ test.describe('Room messages', () => {
     await assertMessages(page, [message]);
   });
 
-  test.skip('attached cards are not duplicated', async ({ page }) => {
+  test('attached cards are not duplicated', async ({ page }) => {
     const testCard1 = `${testHost}/hassan`;
     const testCard2 = `${testHost}/mango`;
     const testCard3 = `${testHost}/type-examples`;
@@ -538,7 +534,7 @@ test.describe('Room messages', () => {
     ]);
   });
 
-  test.skip('displays view all pill if attached card more than 4', async ({
+  test('displays view all pill if attached card more than 4', async ({
     page,
   }) => {
     const testCard1 = `${testHost}/hassan`;
@@ -568,7 +564,7 @@ test.describe('Room messages', () => {
     await expect(page.locator(`[data-test-attached-card]`)).toHaveCount(5);
   });
 
-  test.skip('displays auto-attached card', async ({ page }) => {
+  test('displays auto-attached card', async ({ page }) => {
     const testCard1 = `${testHost}/hassan`;
     const testCard2 = `${testHost}/mango`;
 
@@ -624,7 +620,7 @@ test.describe('Room messages', () => {
     ]);
   });
 
-  test.skip('does not auto attach index card', async ({ page }) => {
+  test('does not auto attach index card', async ({ page }) => {
     const testCard1 = `${testHost}/hassan`;
 
     await login(page, 'user1', 'pass');
@@ -670,7 +666,7 @@ test.describe('Room messages', () => {
     ]);
   });
 
-  test.skip('can remove auto-attached card', async ({ page }) => {
+  test('can remove auto-attached card', async ({ page }) => {
     const testCard1 = `${testHost}/hassan`;
     const testCard2 = `${testHost}/mango`;
     const testCard3 = `${testHost}/type-examples`;
@@ -734,7 +730,7 @@ test.describe('Room messages', () => {
     ]);
   });
 
-  test.skip('it can send the prompts on new-session room as chat message on click', async ({
+  test('it can send the prompts on new-session room as chat message on click', async ({
     page,
   }) => {
     const prompt = {
@@ -752,9 +748,7 @@ test.describe('Room messages', () => {
     await assertMessages(page, [prompt]);
   });
 
-  test.skip('sends message when no card open in the stack', async ({
-    page,
-  }) => {
+  test('sends message when no card open in the stack', async ({ page }) => {
     await login(page, 'user1', 'pass');
     await page
       .locator('[data-test-stack-card] [data-test-close-button]')
@@ -773,9 +767,7 @@ test.describe('Room messages', () => {
     ]);
   });
 
-  test.skip('attaches a card in a coversation multiple times', async ({
-    page,
-  }) => {
+  test('attaches a card in a coversation multiple times', async ({ page }) => {
     const testCard = `${testHost}/hassan`;
 
     await login(page, 'user1', 'pass');

packages/matrix/tests/profile.spec.ts

@@ -72,7 +72,7 @@ test.describe('Profile', () => {
     ).toBeDisabled();
   });
 
-  test.skip('it can change email in settings', async ({ page }) => {
+  test('it can change email in settings', async ({ page }) => {
     await gotoProfileSettings(page);
     await expect(page.locator('[data-test-current-email]')).toContainText(
       'user1@localhost',
@@ -154,7 +154,7 @@ test.describe('Profile', () => {
     await expect(page.locator('[data-test-new-email]')).toHaveCount(0);
   });
 
-  test.skip('it can handle incorrect password when changing email', async ({
+  test('it can handle incorrect password when changing email', async ({
     page,
   }) => {
     await gotoProfileSettings(page);
@@ -183,7 +183,7 @@ test.describe('Profile', () => {
     ).toHaveCount(0);
   });
 
-  test.skip('it can handle setting email to an already existing email when changing email', async ({
+  test('it can handle setting email to an already existing email when changing email', async ({
     page,
   }) => {
     await gotoProfileSettings(page);
@@ -211,7 +211,7 @@ test.describe('Profile', () => {
     ).toHaveCount(0);
   });
 
-  test.skip('it can resend email verification message', async ({ page }) => {
+  test('it can resend email verification message', async ({ page }) => {
     await gotoProfileSettings(page);
     await expect(page.locator('[data-test-current-email]')).toContainText(
       'user1@localhost',
@@ -238,7 +238,7 @@ test.describe('Profile', () => {
     });
   });
 
-  test.skip('it can cancel email verification', async ({ page }) => {
+  test('it can cancel email verification', async ({ page }) => {
     await gotoProfileSettings(page);
     await expect(page.locator('[data-test-current-email]')).toContainText(
       'user1@localhost',
@@ -262,7 +262,7 @@ test.describe('Profile', () => {
     await expect(page.locator('[data-test-new-email]')).toHaveCount(0);
   });
 
-  test.skip('it can cancel password confirmation when changing email', async ({
+  test('it can cancel password confirmation when changing email', async ({
     page,
   }) => {
     await gotoProfileSettings(page);
@@ -287,7 +287,7 @@ test.describe('Profile', () => {
     );
   });
 
-  test.skip('it can cancel changing an email', async ({ page }) => {
+  test('it can cancel changing an email', async ({ page }) => {
     await gotoProfileSettings(page);
     await expect(page.locator('[data-test-current-email]')).toContainText(
       'user1@localhost',
@@ -304,7 +304,7 @@ test.describe('Profile', () => {
     );
   });
 
-  test.skip('it can change password in settings', async ({ page }) => {
+  test('it can change password in settings', async ({ page }) => {
     await gotoProfileSettings(page);
     await expect(page.locator('[data-test-current-email]')).toContainText(
       'user1@localhost',
@@ -347,7 +347,7 @@ test.describe('Profile', () => {
     await assertLoggedIn(page);
   });
 
-  test.skip('It shows an error when new password does not meet the requirement', async ({
+  test('It shows an error when new password does not meet the requirement', async ({
     page,
   }) => {
     await gotoProfileSettings(page);
@@ -427,7 +427,7 @@ test.describe('Profile', () => {
     await assertLoggedIn(page);
   });
 
-  test.skip('It shows an error when current password is invalid', async ({
+  test('It shows an error when current password is invalid', async ({
     page,
   }) => {
     await gotoProfileSettings(page);