luulam/AFFiNE
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

test(editor): revert timeout condition in turbo renderer (#12159)

Browse Source
This commit is contained in:
Yifeng Wang
2025-05-07 09:29:12 +08:00
committed by GitHub
parent b3b78335ad
commit ad7868633d
1 changed files with 8 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

26
blocksuite/integration-test/src/__tests__/edgeless/turbo-renderer.spec.ts
View File

@@ -8,7 +8,6 @@ import {
ViewportTurboRendererExtension, ViewportTurboRendererExtension,
} from '@blocksuite/affine-gfx-turbo-renderer'; } from '@blocksuite/affine-gfx-turbo-renderer';
import { firstValueFrom } from 'rxjs'; import { firstValueFrom } from 'rxjs';
import { filter } from 'rxjs/operators';
import { afterEach, beforeEach, describe, expect, test } from 'vitest'; import { afterEach, beforeEach, describe, expect, test } from 'vitest';
import { wait } from '../utils/common.js'; import { wait } from '../utils/common.js';
@@ -39,10 +38,7 @@ describe('viewport turbo renderer', () => {
test('should render 6 notes in viewport', async () => { test('should render 6 notes in viewport', async () => {
addSampleNotes(doc, 6); addSampleNotes(doc, 6);
const renderer = getRenderer();
await renderer.refresh();
await wait(FRAME); await wait(FRAME);
await firstValueFrom(renderer.state$.pipe(filter(s => s === 'ready')));
const notes = document.querySelectorAll('affine-edgeless-note'); const notes = document.querySelectorAll('affine-edgeless-note');
expect(notes.length).toBe(6); expect(notes.length).toBe(6);
@@ -63,13 +59,12 @@ describe('viewport turbo renderer', () => {
test('zooming should change internal state and populate optimized block ids', async () => { test('zooming should change internal state and populate optimized block ids', async () => {
const renderer = getRenderer(); const renderer = getRenderer();
addSampleNotes(doc, 1); addSampleNotes(doc, 1);
await renderer.refresh();
await wait(FRAME); await wait(FRAME);
await firstValueFrom(renderer.state$.pipe(filter(s => s === 'ready')));
expect(renderer.optimizedBlockIds.length).toBe(0); expect(renderer.optimizedBlockIds.length).toBe(0);
renderer.viewport.zooming$.next(true); renderer.viewport.zooming$.next(true);
await firstValueFrom(renderer.state$.pipe(filter(s => s === 'zooming'))); const nextState = await firstValueFrom(renderer.state$);
expect(nextState).toBe('zooming');
const canUseCache = renderer.canUseBitmapCache(); const canUseCache = renderer.canUseBitmapCache();
expect(canUseCache).toBe(false); expect(canUseCache).toBe(false);
@@ -79,9 +74,9 @@ describe('viewport turbo renderer', () => {
expect(renderer.optimizedBlockIds.length).toBe(1); expect(renderer.optimizedBlockIds.length).toBe(1);
renderer.viewport.zooming$.next(false); renderer.viewport.zooming$.next(false);
await firstValueFrom(renderer.state$.pipe(filter(s => s === 'ready'))); await wait(renderer.options.debounceTime + 100);
expect(renderer.state$.value).toBe('ready'); expect(renderer.state$.value).not.toBe('zooming');
expect(renderer.optimizedBlockIds.length).toBe(0); expect(renderer.optimizedBlockIds.length).toBe(0);
}); });
@@ -89,16 +84,12 @@ describe('viewport turbo renderer', () => {
const renderer = getRenderer(); const renderer = getRenderer();
addSampleNotes(doc, 1); addSampleNotes(doc, 1);
await renderer.refresh();
await wait(FRAME); await wait(FRAME);
await firstValueFrom(renderer.state$.pipe(filter(s => s === 'pending')));
expect(renderer.state$.value).toBe('pending'); expect(renderer.state$.value).toBe('pending');
// Ensure zooming is off and wait for debounce + buffer
renderer.viewport.zooming$.next(false); renderer.viewport.zooming$.next(false);
await renderer.refresh(); await wait(renderer.options.debounceTime + 500);
await wait(FRAME);
await firstValueFrom(renderer.state$.pipe(filter(s => s === 'ready')));
expect(renderer.state$.value).toBe('ready'); expect(renderer.state$.value).toBe('ready');
}); });
@@ -112,14 +103,14 @@ describe('viewport turbo renderer', () => {
addSampleNotes(doc, 1); addSampleNotes(doc, 1);
await wait(100); await wait(100);
// Access getter to populate cache
const _cache = renderer.layoutCache; const _cache = renderer.layoutCache;
noop(_cache); noop(_cache);
expect(renderer.layoutCacheData).not.toBeNull(); expect(renderer.layoutCacheData).not.toBeNull();
// Invalidate
addSampleNotes(doc, 1); addSampleNotes(doc, 1);
await wait(100); await wait(100);
await renderer.refresh();
await wait(FRAME);
expect(renderer.layoutCacheData).toBeNull(); expect(renderer.layoutCacheData).toBeNull();
}); });
@@ -127,7 +118,6 @@ describe('viewport turbo renderer', () => {
test('accessing layoutCache getter should populate cache data', async () => { test('accessing layoutCache getter should populate cache data', async () => {
const renderer = getRenderer(); const renderer = getRenderer();
addSampleNotes(doc, 1); addSampleNotes(doc, 1);
await renderer.refresh();
await wait(FRAME); await wait(FRAME);
expect(renderer.layoutCacheData).toBeNull(); expect(renderer.layoutCacheData).toBeNull();