ros/automaker
1
0
Fork 0
mirror of https://github.com/AutoMaker-Org/automaker.git synced 2026-01-30 06:12:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #582 from stefandevo/fix/e2e-response-disposal-race

Browse Source 
fix: prevent response disposal race condition in E2E test
This commit is contained in:
Shirone
2026-01-19 00:02:04 +00:00
committed by GitHub
parent 09151aa3c8 eb30ef71f9
commit e73c92b031
1 changed files with 15 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

28
apps/ui/tests/projects/open-existing-project.spec.ts
View File

@@ -84,18 +84,24 @@ test.describe('Open Project', () => {
// Intercept settings API BEFORE any navigation to prevent restoring a currentProject
// AND inject our test project into the projects list
await page.route('**/api/settings/global', async (route) => {
const response = await route.fetch();
// Immediately consume the body to prevent disposal issues
const bodyPromise = response.body();
const status = response.status();
const headers = response.headers();
const body = await bodyPromise;
let response;
try {
response = await route.fetch();
} catch {
// If fetch fails, continue with original request
await route.continue();
return;
}
let json;
try {
json = JSON.parse(body.toString());
json = await response.json();
} catch {
json = {};
// If response is disposed, continue with original request
await route.continue();
return;
}
if (json.settings) {
// Remove currentProjectId to prevent restoring a project
json.settings.currentProjectId = null;
@@ -115,11 +121,7 @@ test.describe('Open Project', () => {
json.settings.projects = [testProject, ...existingProjects];
}
}
await route.fulfill({
status: status,
headers: headers,
json,
});
await route.fulfill({ response, json });
});
// Now navigate to the app