fix: tests

toeverything · Jan 23, 2024 · 45d6126 · 45d6126
1 parent e6971a3
commit 45d6126
Show file tree

Hide file tree

Showing 3 changed files with 31 additions and 13 deletions.
diff --git a/packages/blocks/src/page-block/edgeless/controllers/clipboard.ts b/packages/blocks/src/page-block/edgeless/controllers/clipboard.ts
@@ -990,8 +990,10 @@ export class EdgelessClipboardController extends PageClipboard {
       block: TopLevelBlockModel,
       isInFrame = false
     ) => {
-      let blockElement = blockElementGetter(block, this.std.view)
-        ?.parentElement;
+      let blockElement = blockElementGetter(
+        block,
+        this.std.view
+      )?.parentElement;
       const blockPortalSelector = block.flavour.replace(
         'affine:',
         '.edgeless-block-portal-'

diff --git a/packages/blocks/src/page-block/edgeless/edgeless-page-block.ts b/packages/blocks/src/page-block/edgeless/edgeless-page-block.ts
@@ -291,9 +291,8 @@ export class EdgelessPageBlockComponent extends BlockElement<
     _disposables.add(this.tools);
     _disposables.add(this.service.selection);
     _disposables.add(
-      slots.zoomUpdated.on(
-        (action: ZoomAction) =>
-          this.components.zoomToolbar?.setZoomByAction(action)
+      slots.zoomUpdated.on((action: ZoomAction) =>
+        this.components.zoomToolbar?.setZoomByAction(action)
       )
     );
     _disposables.add(

diff --git a/tests/attachment.spec.ts b/tests/attachment.spec.ts
@@ -65,7 +65,7 @@ function getAttachment(page: Page) {
   };
 
   const getName = () =>
-    attachment.locator('.affine-attachment-name').innerText();
+    attachment.locator('.affine-attachment-content-title-text').innerText();
 
   return {
     // locators
@@ -82,7 +82,8 @@ function getAttachment(page: Page) {
      */
     waitLoading: () => loading.waitFor({ state: 'hidden' }),
     getName,
-    getSize: () => attachment.locator('.affine-attachment-desc').innerText(),
+    getSize: () =>
+      attachment.locator('.affine-attachment-content-info').innerText(),
 
     turnToEmbed: async () => {
       await expect(turnToEmbedBtn).toBeVisible();
@@ -151,9 +152,12 @@ test('can insert attachment from slash menu', async ({ page }) => {
 >
   <affine:attachment
     prop:embed={false}
+    prop:index="a0"
     prop:name="${FILE_NAME}"
+    prop:rotate={0}
     prop:size={${FILE_SIZE}}
     prop:sourceId="${FILE_ID}"
+    prop:style="horizontalThin"
     prop:type="image/png"
   />
 </affine:note>`,
@@ -193,9 +197,12 @@ test('should undo/redo works for attachment', async ({ page }) => {
 >
   <affine:attachment
     prop:embed={false}
+    prop:index="a0"
     prop:name="${FILE_NAME}"
+    prop:rotate={0}
     prop:size={${FILE_SIZE}}
     prop:sourceId="${FILE_ID}"
+    prop:style="horizontalThin"
     prop:type="image/png"
   />
 </affine:note>`,
@@ -252,9 +259,12 @@ test('should undo/redo works for attachment', async ({ page }) => {
 >
   <affine:attachment
     prop:embed={false}
+    prop:index="a0"
     prop:name="${FILE_NAME}"
+    prop:rotate={0}
     prop:size={${FILE_SIZE}}
     prop:sourceId="${FILE_ID}"
+    prop:style="horizontalThin"
     prop:type="image/png"
   />
 </affine:note>`,
@@ -368,9 +378,12 @@ test('should turn attachment to image works', async ({ page }) => {
   <affine:attachment
     prop:caption=""
     prop:embed={false}
+    prop:index="a0"
     prop:name="${FILE_NAME}"
+    prop:rotate={0}
     prop:size={${FILE_SIZE}}
     prop:sourceId="${FILE_ID}"
+    prop:style="horizontalThin"
     prop:type="image/png"
   />
 </affine:note>`,
@@ -453,9 +466,12 @@ test(`support dragging attachment block directly`, async ({ page }) => {
 >
   <affine:attachment
     prop:embed={false}
+    prop:index="a0"
     prop:name="${FILE_NAME}"
+    prop:rotate={0}
     prop:size={${FILE_SIZE}}
     prop:sourceId="${FILE_ID}"
+    prop:style="horizontalThin"
     prop:type="image/png"
   />
 </affine:note>`,
@@ -504,9 +520,12 @@ test(`support dragging attachment block directly`, async ({ page }) => {
   >
     <affine:attachment
       prop:embed={false}
+      prop:index="a0"
       prop:name="${FILE_NAME}"
+      prop:rotate={0}
       prop:size={${FILE_SIZE}}
       prop:sourceId="${FILE_ID}"
+      prop:style="horizontalThin"
       prop:type="image/png"
     />
     <affine:paragraph
@@ -528,13 +547,8 @@ test(`support dragging attachment block directly`, async ({ page }) => {
   // drag bookmark block
   await page.mouse.move(rect.x + 20, rect.y + 20);
   await page.mouse.down();
-  await page.waitForTimeout(200);
-
-  await page.mouse.move(rect.x + 40, rect.y + rect.height + 80);
-  await page.waitForTimeout(200);
-
+  await page.mouse.move(rect.x + 40, rect.y + rect.height + 80, { steps: 20 });
   await page.mouse.up();
-  await page.waitForTimeout(200);
 
   const rects = page.locator('affine-block-selection');
   await expect(rects).toHaveCount(1);
@@ -568,9 +582,12 @@ test(`support dragging attachment block directly`, async ({ page }) => {
     />
     <affine:attachment
       prop:embed={false}
+      prop:index="a0"
       prop:name="${FILE_NAME}"
+      prop:rotate={0}
       prop:size={${FILE_SIZE}}
       prop:sourceId="${FILE_ID}"
+      prop:style="horizontalThin"
       prop:type="image/png"
     />
     <affine:paragraph