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

chore(server): fix unstable test (#13037)

Browse Source 
#### PR Dependency Tree


* **PR #13037** 👈

This tree was auto-generated by
[Charcoal](https://github.com/danerwilliams/charcoal)

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Tests**
* Improved test reliability by generating unique user accounts and
prompt names for each test run.
  * Updated test setup to streamline database initialization.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->
This commit is contained in:
fengmk2
2025-07-04 18:46:07 +08:00
committed by GitHub
parent 831da01432
commit 5da56b5b04
1 changed files with 12 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
packages/backend/server/src/__tests__/copilot.e2e.ts
View File

@@ -134,14 +134,16 @@ test.before(async t => {
t.context.jobs = jobs; t.context.jobs = jobs;
}); });
const textPromptName = 'prompt'; let textPromptName = 'prompt';
const imagePromptName = 'prompt-image'; let imagePromptName = 'prompt-image';
test.beforeEach(async t => { test.beforeEach(async t => {
Sinon.restore(); Sinon.restore();
const { app, prompt } = t.context; const { app, prompt } = t.context;
await app.initTestingDB();
await prompt.onApplicationBootstrap(); await prompt.onApplicationBootstrap();
t.context.u1 = await app.signupV1('u1@affine.pro'); t.context.u1 = await app.signupV1();
textPromptName = randomUUID().replaceAll('-', '');
imagePromptName = randomUUID().replaceAll('-', '');
await prompt.set(textPromptName, 'test', [ await prompt.set(textPromptName, 'test', [
{ role: 'system', content: 'hello {{word}}' }, { role: 'system', content: 'hello {{word}}' },
@@ -189,7 +191,7 @@ test('should create session correctly', async t => {
} }
{ {
const u2 = await app.createUser('u2@affine.pro'); const u2 = await app.createUser();
const { id } = await createWorkspace(app); const { id } = await createWorkspace(app);
await app.login(u2); await app.login(u2);
await assertCreateSession(id, '', async x => { await assertCreateSession(id, '', async x => {
@@ -253,8 +255,8 @@ test('should update session correctly', async t => {
} }
{ {
await app.signupV1('test@affine.pro'); await app.signupV1();
const u2 = await app.createUser('u2@affine.pro'); const u2 = await app.createUser();
const { id: workspaceId } = await createWorkspace(app); const { id: workspaceId } = await createWorkspace(app);
const inviteId = await inviteUser(app, workspaceId, u2.email); const inviteId = await inviteUser(app, workspaceId, u2.email);
await app.login(u2); await app.login(u2);
@@ -356,7 +358,7 @@ test('should fork session correctly', async t => {
} }
{ {
const u2 = await app.signupV1('u2@affine.pro'); const u2 = await app.signupV1();
await assertForkSession(id, sessionId, randomUUID(), '', async x => { await assertForkSession(id, sessionId, randomUUID(), '', async x => {
await t.throwsAsync( await t.throwsAsync(
x, x,
@@ -712,7 +714,7 @@ test('should reject message from different session', async t => {
test('should reject request from different user', async t => { test('should reject request from different user', async t => {
const { app, u1 } = t.context; const { app, u1 } = t.context;
const u2 = await app.createUser('u2@affine.pro'); const u2 = await app.createUser();
const { id } = await createWorkspace(app); const { id } = await createWorkspace(app);
const sessionId = await createCopilotSession( const sessionId = await createCopilotSession(
app, app,
@@ -789,7 +791,7 @@ test('should be able to list history', async t => {
test('should reject request that user have not permission', async t => { test('should reject request that user have not permission', async t => {
const { app, u1 } = t.context; const { app, u1 } = t.context;
const u2 = await app.createUser('u2@affine.pro'); const u2 = await app.createUser();
const { id: workspaceId } = await createWorkspace(app); const { id: workspaceId } = await createWorkspace(app);
// should reject request that user have not permission // should reject request that user have not permission