mirror of
https://github.com/mudler/LocalAI.git
synced 2026-10-28 17:18:49 -04:00
* fix(ci): capture and surface ginkgo errors in run-coverage.sh
The coverage script ran ginkgo per test root with `|| fail=1`, which
swallowed the exit code. Combined with `make --output-sync=target`,
which buffers all output and flushes at the end, the actual ginkgo
error was invisible in interleaved CI logs. Every test suite passed
but the script still exited 1, and the error message was buried in
22 000 lines of buffered output.
Each ginkgo invocation now captures combined stdout+stderr to a
per-root log file. On failure the last 80 lines are printed to
stderr so the error is visible in the CI log. On success the log is
replayed to stdout so test output remains visible.
The build flags (--tags, --coverpkg) are quoted correctly so values
containing spaces (e.g. --tags="debug auth") are preserved through
eval.
Signed-off-by: Ettore Di Giacinto <mudler@localai.io>
* fix(ci): create output dir before writing ginkgo logs
The previous commit added per-root log files but forgot to create
the output directory before the first ginkgo invocation. On CI the
coverage/ dir did not exist yet, so the shell redirect to
cover-pkg.out.log failed with "No such file or directory" before
ginkgo even ran.
Signed-off-by: Ettore Di Giacinto <mudler@localai.io>
* test(core/http): add canvas preview route to auth coverage allowlist
Commit 9ad78381d8 added GET /api/canvas/preview/:id as a public route
(no auth — the random id is the capability, an iframe cannot send API
key headers) but did not add it to expectedPublicRoutes in the route
auth coverage test. The test flagged the route as an anonymous-access
leak, failing the entire ./core ginkgo suite on every CI run since.
The POST /api/canvas/preview endpoint is auth-gated (returns 401 to
anonymous requests) and is intentionally absent from the allowlist.
Signed-off-by: Ettore Di Giacinto <mudler@localai.io>
* fix(ui): restore confirmed per-node installed backend removal
Verify exact node inventory and protection before offering and confirming removal. Capture the node/backend target, guard duplicate requests, and preserve installed data when refresh fails. Reuse the flow for catalog target removal and document scoped retry behavior.
Assisted-by: nib:gpt-6.1-sol
Signed-off-by: Ettore Di Giacinto <mudler@localai.io>
* fix(ui): guard targeted backend mutations and refresh feedback
Distinguish ordinary refresh failures from successful deletion with stale data. Lock target installs synchronously and block conflicting install/removal actions through completion-driven refresh. Cover retry wording, deferred mutation races, obsolete inventory responses, and final installation selection cleanup.
Assisted-by: nib:gpt-6.1-sol
Signed-off-by: Ettore Di Giacinto <mudler@localai.io>
* fix(ui): block conflicting generic backend deletion
Keep generic deletion blocked during scoped removal and targeted installs.
Recheck conflicts after dependency lookup and before confirmation so
navigation cannot expose a conflicting delete action.
Assisted-by: nib:gpt-6.1-sol
Signed-off-by: Ettore Di Giacinto <mudler@localai.io>
---------
Signed-off-by: Ettore Di Giacinto <mudler@localai.io>
Co-authored-by: Ettore Di Giacinto <mudler@localai.io>
432 lines
23 KiB
JavaScript
432 lines
23 KiB
JavaScript
// SPDX-License-Identifier: MIT
|
|
import { test, expect } from './coverage-fixtures.js'
|
|
|
|
const BUILD = 'cpu-example-engine-development'
|
|
const SIBLING = 'cpu-example-engine'
|
|
const ids = ['worker-alpha', 'worker-beta']
|
|
const remove = (page, id) => page.getByRole('button', { name: `Remove from ${id}…`, exact: true })
|
|
|
|
async function setup(page, options = {}) {
|
|
const state = { deletes: [], installs: [], inventoryCalls: {}, installedCalls: 0, failRefresh: false, ...options }
|
|
const nodes = ids.map(id => ({ id, name: id, node_type: 'backend', status: 'healthy', address: `${id}.example:50051` }))
|
|
const inventories = Object.fromEntries(ids.map(id => [id, [{ name: BUILD, is_system: false, is_meta: false }, { name: SIBLING, is_system: false, is_meta: false }]]))
|
|
state.nodes = nodes
|
|
state.inventories = inventories
|
|
await page.route('**/api/nodes', route => route.fulfill({ json: state.nodes }))
|
|
await page.route('**/api/backends?*', route => state.failRefresh
|
|
? route.fulfill({ status: 500, json: { error: 'catalog unavailable' } })
|
|
: route.fulfill({ json: { backends: [BUILD, SIBLING].map(name => ({ id: name, name, installed: true, isAlias: true, isDevelopment: name === BUILD, nodes: ids.map(id => ({ node_id: id, node_name: id })) })) } }))
|
|
await page.route('**/api/backends/upgrades', route => route.fulfill({ json: {} }))
|
|
await page.route('**/backends', async route => {
|
|
state.installedCalls++
|
|
if (state.refreshWait && state.deletes.length) await state.refreshWait
|
|
return state.failRefresh ? route.fulfill({ status: 500, json: { error: 'installed unavailable' } }) : route.fulfill({ json: [BUILD, SIBLING].filter(Name => ids.some(id => inventories[id].some(b => b.name === Name))).map(Name => ({ Name, IsSystem: !!state.aggregateSystem, Nodes: ids.filter(id => inventories[id].some(b => b.name === Name)).map(id => ({ node_id: id, node_name: id, node_status: 'healthy' })) })) })
|
|
})
|
|
for (const id of ids) {
|
|
await page.route(`**/api/nodes/${id}/backends/install`, async route => {
|
|
state.installs.push({ id, method: route.request().method(), body: route.request().postDataJSON() })
|
|
if (state.installWait) await state.installWait
|
|
return route.fulfill({ json: { message: 'queued' } })
|
|
})
|
|
await page.route(`**/api/nodes/${id}/backends`, async route => {
|
|
state.inventoryCalls[id] = (state.inventoryCalls[id] || 0) + 1
|
|
if (state.inventoryWait) await state.inventoryWait
|
|
return state.inventoryError ? route.fulfill({ status: 500, json: { error: 'inventory unavailable' } }) : route.fulfill({ json: inventories[id] })
|
|
})
|
|
await page.route(`**/api/nodes/${id}/backends/delete`, async route => {
|
|
state.deletes.push({ id, method: route.request().method(), body: route.request().postDataJSON() })
|
|
if (state.deleteWait) await state.deleteWait
|
|
if (state.deleteError) return route.fulfill({ status: 500, json: { error: 'removal rejected' } })
|
|
inventories[id] = inventories[id].filter(b => b.name !== BUILD)
|
|
if (state.refreshError) state.failRefresh = true
|
|
return route.fulfill({ json: { message: 'removed' } })
|
|
})
|
|
}
|
|
await page.route('**/backends/delete/**', () => { throw new Error('Global deletion is forbidden') })
|
|
return state
|
|
}
|
|
async function open(page, extra = '') {
|
|
await page.goto(`/app/backends?view=installed&show_all=1&development=1${extra}`)
|
|
await expect(page.locator(`[data-entity="${BUILD}"]`)).toBeVisible()
|
|
await page.locator(`[data-entity="${BUILD}"]`).click()
|
|
}
|
|
const confirm = page => page.getByRole('alertdialog').locator('button.btn-danger')
|
|
|
|
test('Installed concrete build has independent lazy per-node actions and scoped deletion', async ({ page }) => {
|
|
const s = await setup(page)
|
|
await page.goto('/app/backends?view=installed&show_all=1&development=1')
|
|
await expect(page.locator(`[data-entity="${BUILD}"]`)).toBeVisible()
|
|
expect(s.inventoryCalls).toEqual({})
|
|
await page.locator(`[data-entity="${BUILD}"]`).click()
|
|
await expect(remove(page, ids[0])).toBeEnabled()
|
|
await expect(remove(page, ids[1])).toBeEnabled()
|
|
await remove(page, ids[0]).click()
|
|
await expect(page.getByRole('alertdialog')).toContainText(BUILD)
|
|
await expect(page.getByRole('alertdialog')).toContainText(ids[0])
|
|
expect(s.deletes).toEqual([])
|
|
await confirm(page).click()
|
|
await expect(page.getByRole('alertdialog')).toBeHidden()
|
|
expect(s.deletes).toEqual([{ id: ids[0], method: 'POST', body: { backend: BUILD } }])
|
|
await expect(remove(page, ids[1])).toBeEnabled()
|
|
await expect(page.locator(`[data-entity="${SIBLING}"]`)).toBeVisible()
|
|
expect(s.installedCalls).toBeGreaterThan(1)
|
|
})
|
|
|
|
for (const cancel of ['Cancel', 'Escape', 'backdrop']) test(`cancellation ${cancel} sends no request`, async ({ page }) => {
|
|
const s = await setup(page); await open(page)
|
|
await remove(page, ids[0]).click()
|
|
if (cancel === 'Escape') await page.keyboard.press('Escape')
|
|
else if (cancel === 'backdrop') await page.locator('.confirm-dialog-backdrop').click({ position: { x: 2, y: 2 } })
|
|
else await page.getByRole('alertdialog').getByRole('button', { name: 'Cancel', exact: true }).click()
|
|
await expect(page.getByRole('alertdialog')).toBeHidden()
|
|
expect(s.deletes).toEqual([])
|
|
await expect(remove(page, ids[0])).toBeEnabled()
|
|
})
|
|
|
|
for (const aggregateSystem of [true, false]) test(`actual mixed system inventory overrides aggregate ${aggregateSystem}`, async ({ page }) => {
|
|
const s = await setup(page, { aggregateSystem }); s.inventories[ids[0]][0].is_system = true
|
|
await open(page)
|
|
await expect(remove(page, ids[0])).toBeDisabled()
|
|
await expect(remove(page, ids[1])).toBeEnabled()
|
|
})
|
|
|
|
for (const mode of ['system', 'missing', 'offline', 'inventory error']) test(`confirmation revalidation blocks ${mode}`, async ({ page }) => {
|
|
const s = await setup(page); await open(page); await remove(page, ids[0]).click()
|
|
if (mode === 'system') s.inventories[ids[0]][0].is_system = true
|
|
if (mode === 'missing') s.inventories[ids[0]] = []
|
|
if (mode === 'offline') s.nodes[0].status = 'offline'
|
|
if (mode === 'inventory error') s.inventoryError = true
|
|
await confirm(page).click()
|
|
await expect(page.getByRole('alertdialog')).toBeHidden()
|
|
await expect(page.getByRole('alert').filter({ hasText: ids[0] })).toBeVisible()
|
|
expect(s.deletes).toEqual([])
|
|
})
|
|
|
|
for (const mode of ['unknown', 'meta', 'alias', 'offline', 'draining', 'missing target']) test(`fail closed for ${mode}`, async ({ page }) => {
|
|
const s = await setup(page)
|
|
if (mode === 'unknown') delete s.inventories[ids[0]][0].is_system
|
|
if (mode === 'meta') s.inventories[ids[0]][0].is_meta = true
|
|
if (mode === 'alias') s.inventories[ids[0]] = []
|
|
if (mode === 'offline' || mode === 'draining') s.nodes[0].status = mode
|
|
if (mode === 'missing target') s.nodes.shift()
|
|
await open(page, `&target=${ids[0]}`)
|
|
if (mode === 'alias') await expect(remove(page, ids[0])).toHaveCount(0)
|
|
else await expect(remove(page, ids[0])).toBeDisabled()
|
|
await expect(remove(page, ids[1])).toHaveCount(0)
|
|
expect(s.deletes).toEqual([])
|
|
})
|
|
|
|
test('inventory error retry and deferred loading fail closed', async ({ page }) => {
|
|
let release; const wait = new Promise(resolve => { release = resolve })
|
|
const s = await setup(page, { inventoryWait: wait, inventoryError: true }); await open(page)
|
|
await expect(remove(page, ids[0])).toBeDisabled(); release()
|
|
await expect(page.getByRole('button', { name: `Retry inventory for ${ids[0]}` })).toBeVisible()
|
|
s.inventoryError = false
|
|
await page.getByRole('button', { name: `Retry inventory for ${ids[0]}` }).click()
|
|
await expect(remove(page, ids[0])).toBeEnabled()
|
|
})
|
|
|
|
test('immutable URL target and duplicate confirmation guard', async ({ page }) => {
|
|
let release; const wait = new Promise(resolve => { release = resolve })
|
|
const s = await setup(page, { deleteWait: wait }); await open(page, `&target=${ids[0]}`)
|
|
await remove(page, ids[0]).click()
|
|
await page.evaluate(id => { const url = new URL(location.href); url.searchParams.set('target', id); history.pushState({}, '', url); dispatchEvent(new PopStateEvent('popstate')) }, ids[1])
|
|
await confirm(page).evaluate(el => { el.click(); el.click() })
|
|
await expect.poll(() => s.deletes.length).toBe(1)
|
|
await expect(confirm(page)).toBeDisabled()
|
|
release(); await expect(page.getByRole('alertdialog')).toBeHidden()
|
|
expect(s.deletes).toEqual([{ id: ids[0], method: 'POST', body: { backend: BUILD } }])
|
|
})
|
|
|
|
test('API failure preserves rows and permits explicit confirmed retry', async ({ page }) => {
|
|
const s = await setup(page, { deleteError: true }); await open(page)
|
|
await remove(page, ids[0]).click(); await confirm(page).click()
|
|
await expect(page.getByRole('alertdialog')).toBeHidden()
|
|
await expect(page.getByRole('alert').filter({ hasText: ids[0] })).toBeVisible()
|
|
await expect(remove(page, ids[0])).toBeEnabled()
|
|
expect(s.deletes).toHaveLength(1)
|
|
s.deleteError = false
|
|
await remove(page, ids[0]).click(); expect(s.deletes).toHaveLength(1)
|
|
await confirm(page).click(); await expect(page.getByRole('alertdialog')).toBeHidden()
|
|
expect(s.deletes).toHaveLength(2)
|
|
})
|
|
|
|
test('successful deletion with refresh failure retains stale rows and offers refresh only', async ({ page }) => {
|
|
const s = await setup(page, { refreshError: true }); await open(page)
|
|
await remove(page, ids[0]).click(); await confirm(page).click()
|
|
await expect(page.getByRole('alertdialog')).toBeHidden()
|
|
await expect(page.getByRole('button', { name: 'Retry backend refresh' })).toBeVisible()
|
|
await expect(page.locator(`[data-entity="${BUILD}"]`)).toBeVisible()
|
|
await expect(page.locator(`[data-entity="${SIBLING}"]`)).toBeVisible()
|
|
await expect(page.getByText(/Deletion succeeded/i)).toBeVisible()
|
|
const before = s.installedCalls
|
|
await page.getByRole('button', { name: 'Retry backend refresh' }).click()
|
|
await expect.poll(() => s.installedCalls).toBeGreaterThan(before)
|
|
await expect(page.getByText(/Deletion succeeded/i)).toBeVisible()
|
|
s.failRefresh = false
|
|
await page.getByRole('button', { name: 'Retry backend refresh' }).click()
|
|
await expect(remove(page, ids[1])).toBeEnabled()
|
|
expect(s.deletes).toHaveLength(1)
|
|
})
|
|
|
|
|
|
test('catalog target uses the same confirmed exact-node flow', async ({ page }) => {
|
|
const s = await setup(page)
|
|
await page.goto(`/app/backends?view=catalog&show_all=1&development=1&target=${ids[0]}&backend=${BUILD}`)
|
|
await expect(remove(page, ids[0])).toBeEnabled()
|
|
await remove(page, ids[0]).click()
|
|
expect(s.deletes).toHaveLength(0)
|
|
await expect(page.getByRole('alertdialog')).toContainText(BUILD)
|
|
await confirm(page).click()
|
|
await expect(page.getByRole('alertdialog')).toBeHidden()
|
|
expect(s.deletes).toEqual([{ id: ids[0], method: 'POST', body: { backend: BUILD } }])
|
|
})
|
|
|
|
test('cold installed failure is not empty success, and retry recovers', async ({ page }) => {
|
|
const s = await setup(page, { failRefresh: true })
|
|
await page.goto('/app/backends?view=installed&show_all=1&development=1')
|
|
await expect(page.getByRole('button', { name: 'Retry backend refresh' })).toBeVisible()
|
|
await expect(page.getByText('No backends installed', { exact: true })).toHaveCount(0)
|
|
s.failRefresh = false
|
|
await page.getByRole('button', { name: 'Retry backend refresh' }).click()
|
|
await expect(page.locator(`[data-entity="${BUILD}"]`)).toBeVisible()
|
|
})
|
|
|
|
test('background installed refresh retains rows on failure without claiming deletion success', async ({ page }) => {
|
|
const s = await setup(page); await open(page)
|
|
s.failRefresh = true
|
|
// An operations change triggers the existing background list refresh.
|
|
await page.route('**/api/operations', route => route.fulfill({ json: { operations: [{ name: SIBLING, jobID: 'synthetic-job', isQueued: true }] } }))
|
|
await expect(page.getByRole('button', { name: 'Retry backend refresh' })).toBeVisible({ timeout: 15000 })
|
|
await expect(page.locator(`[data-entity="${BUILD}"]`)).toBeVisible()
|
|
expect(s.deletes).toHaveLength(0)
|
|
})
|
|
|
|
test('conflicting queued operation disables removal', async ({ page }) => {
|
|
const s = await setup(page)
|
|
await page.route('**/api/operations', route => route.fulfill({ json: { operations: [{ name: BUILD, jobID: 'synthetic-job', isQueued: true }] } }))
|
|
await open(page)
|
|
await expect(remove(page, ids[0])).toBeDisabled()
|
|
expect(s.deletes).toHaveLength(0)
|
|
})
|
|
|
|
test('standalone system protection has no scoped removal', async ({ page }) => {
|
|
const s = await setup(page, { aggregateSystem: true })
|
|
await page.route('**/api/nodes', route => route.fulfill({ status: 503, json: { error: 'distributed disabled' } }))
|
|
await open(page)
|
|
await expect(remove(page, ids[0])).toHaveCount(0)
|
|
await expect(page.locator(`[data-entity="${BUILD}"]`)).toContainText('Protected')
|
|
expect(s.deletes).toHaveLength(0)
|
|
})
|
|
|
|
for (const initial of [true, false]) test(`unsuccessful ${initial ? 'initial' : 'background'} refresh retry never claims deletion`, async ({ page }) => {
|
|
const s = await setup(page, { failRefresh: initial })
|
|
if (initial) await page.goto('/app/backends?view=installed&show_all=1&development=1')
|
|
else {
|
|
await open(page)
|
|
s.failRefresh = true
|
|
await page.route('**/api/operations', route => route.fulfill({ json: { operations: [{ name: SIBLING, jobID: 'synthetic-job', isQueued: true }] } }))
|
|
}
|
|
const retry = page.getByRole('button', { name: 'Retry backend refresh' })
|
|
await expect(retry).toBeVisible({ timeout: 15000 })
|
|
const before = s.installedCalls
|
|
await retry.click()
|
|
await expect.poll(() => s.installedCalls).toBeGreaterThan(before)
|
|
await expect(page.getByRole('alert').filter({ hasText: /refresh failed|could not|unable|failed to/i }).first()).toBeVisible()
|
|
await expect(page.getByText(/Deletion succeeded/i)).toHaveCount(0)
|
|
expect(s.deletes).toEqual([])
|
|
})
|
|
|
|
const catalogTarget = page => page.goto(`/app/backends?view=catalog&show_all=1&development=1&target=${ids[0]}&backend=${BUILD}`)
|
|
const reinstall = page => page.getByRole('button', { name: 'Reinstall', exact: true })
|
|
|
|
test('deferred target reinstall synchronously blocks duplicate installs and removal', async ({ page }) => {
|
|
let release; const wait = new Promise(resolve => { release = resolve })
|
|
const s = await setup(page, { installWait: wait }); await catalogTarget(page)
|
|
await expect(remove(page, ids[0])).toBeEnabled()
|
|
await reinstall(page).evaluate(el => { el.click(); el.click() })
|
|
await expect.poll(() => s.installs.length).toBeGreaterThan(0)
|
|
await expect(reinstall(page)).toBeDisabled()
|
|
await expect(remove(page, ids[0])).toBeDisabled()
|
|
// Programmatic clicks still honor disabled state; no confirmation/mutation.
|
|
await remove(page, ids[0]).evaluate(el => el.click())
|
|
await expect(page.getByRole('alertdialog')).toBeHidden()
|
|
expect(s.deletes).toEqual([])
|
|
expect(s.installs).toEqual([{ id: ids[0], method: 'POST', body: { backend: BUILD } }])
|
|
release()
|
|
await expect(reinstall(page)).toBeEnabled()
|
|
})
|
|
|
|
test('target reinstall stays blocked throughout deletion and deferred postdelete refresh', async ({ page }) => {
|
|
let releaseDelete, releaseRefresh
|
|
const s = await setup(page, {
|
|
deleteWait: new Promise(resolve => { releaseDelete = resolve }),
|
|
refreshWait: new Promise(resolve => { releaseRefresh = resolve }),
|
|
})
|
|
await catalogTarget(page); await remove(page, ids[0]).click(); await confirm(page).click()
|
|
await expect.poll(() => s.deletes.length).toBe(1)
|
|
await expect(reinstall(page)).toBeDisabled()
|
|
releaseDelete()
|
|
await expect(page.getByRole('alertdialog')).toBeHidden()
|
|
await expect(reinstall(page)).toBeDisabled()
|
|
await expect(remove(page, ids[0])).toBeDisabled()
|
|
await reinstall(page).evaluate(el => el.click())
|
|
expect(s.installs).toEqual([])
|
|
expect(s.deletes).toHaveLength(1)
|
|
releaseRefresh()
|
|
await expect(reinstall(page)).toBeEnabled()
|
|
})
|
|
|
|
test('out-of-order inventory from previous row cannot overwrite current inventory', async ({ page }) => {
|
|
const s = await setup(page)
|
|
let release; const wait = new Promise(resolve => { release = resolve })
|
|
let first = true
|
|
await page.route(`**/api/nodes/${ids[0]}/backends`, async route => {
|
|
if (first) {
|
|
first = false
|
|
await wait
|
|
return route.fulfill({ json: [{ name: SIBLING, is_system: false, is_meta: false }] })
|
|
}
|
|
return route.fulfill({ json: [{ name: SIBLING, is_system: true, is_meta: false }] })
|
|
})
|
|
await open(page, `&target=${ids[0]}`)
|
|
await expect(remove(page, ids[0])).toBeDisabled()
|
|
await expect.poll(() => first).toBe(false)
|
|
await page.locator(`[data-entity="${SIBLING}"]`).click()
|
|
await expect(remove(page, ids[0])).toBeDisabled()
|
|
// Wait for the new protected response before releasing the obsolete managed one.
|
|
await expect(page.locator('.text-note').filter({ hasText: /protected|system/i })).toBeVisible()
|
|
const obsoleteResponse = page.waitForResponse(response => response.url().endsWith(`/api/nodes/${ids[0]}/backends`))
|
|
release()
|
|
await obsoleteResponse
|
|
await expect(remove(page, ids[0])).toBeDisabled()
|
|
expect(s.deletes).toEqual([])
|
|
})
|
|
|
|
test('last installation disappearance clears only selection and preserves URL refinements', async ({ page }) => {
|
|
const s = await setup(page)
|
|
s.inventories[ids[1]] = s.inventories[ids[1]].filter(b => b.name !== BUILD)
|
|
await open(page, `&target=${ids[0]}&q=example`)
|
|
await remove(page, ids[0]).click(); await confirm(page).click()
|
|
await expect(page.locator(`[data-entity="${BUILD}"]`)).toHaveCount(0)
|
|
await expect.poll(() => new URL(page.url()).searchParams.has('backend')).toBe(false)
|
|
const params = new URL(page.url()).searchParams
|
|
expect(Object.fromEntries(params)).toEqual({ view: 'installed', show_all: '1', development: '1', target: ids[0], q: 'example' })
|
|
await expect(page.locator(`[data-entity="${SIBLING}"]`)).toBeVisible()
|
|
expect(s.deletes).toEqual([{ id: ids[0], method: 'POST', body: { backend: BUILD } }])
|
|
})
|
|
|
|
const clearTarget = page => page.evaluate(() => {
|
|
const url = new URL(location.href)
|
|
url.searchParams.delete('target')
|
|
history.pushState({}, '', url)
|
|
dispatchEvent(new PopStateEvent('popstate'))
|
|
})
|
|
const genericRemove = page => page.locator('.bk-remove')
|
|
|
|
// A synthetic click must not bypass the disabled control.
|
|
const dispatchClick = locator => locator.evaluate(el => el.dispatchEvent(new MouseEvent('click', { bubbles: true })))
|
|
|
|
test('generic removal stays blocked during scoped postdelete refresh after target clears', async ({ page }) => {
|
|
let releaseRefresh
|
|
const s = await setup(page, { refreshWait: new Promise(resolve => { releaseRefresh = resolve }) })
|
|
const globalDeletes = []
|
|
await page.route('**/backends/delete/**', route => {
|
|
globalDeletes.push(route.request().url())
|
|
return route.fulfill({ json: { message: 'removed' } })
|
|
})
|
|
await catalogTarget(page); await remove(page, ids[0]).click(); await confirm(page).click()
|
|
await expect(page.getByRole('alertdialog')).toBeHidden()
|
|
await expect.poll(() => s.installedCalls).toBeGreaterThan(1)
|
|
await clearTarget(page)
|
|
await expect(genericRemove(page)).toBeDisabled()
|
|
await dispatchClick(genericRemove(page))
|
|
await expect(page.getByRole('alertdialog')).toBeHidden()
|
|
expect(globalDeletes).toEqual([])
|
|
expect(s.deletes).toEqual([{ id: ids[0], method: 'POST', body: { backend: BUILD } }])
|
|
releaseRefresh()
|
|
await expect(genericRemove(page)).toHaveCount(0)
|
|
})
|
|
|
|
test('generic removal rechecks scoped conflict after dependency lookup', async ({ page }) => {
|
|
let releaseCapabilities, releaseRefresh
|
|
const s = await setup(page, { refreshWait: new Promise(resolve => { releaseRefresh = resolve }) })
|
|
const capabilitiesWait = new Promise(resolve => { releaseCapabilities = resolve })
|
|
let capabilitiesStarted = false
|
|
await page.route('**/models/capabilities', async route => {
|
|
capabilitiesStarted = true
|
|
await capabilitiesWait
|
|
return route.fulfill({ json: { data: [] } })
|
|
})
|
|
await catalogTarget(page)
|
|
await clearTarget(page)
|
|
await genericRemove(page).click()
|
|
await expect.poll(() => capabilitiesStarted).toBe(true)
|
|
// The generic lookup is pending, so a scoped confirmation can start first.
|
|
await page.evaluate(id => {
|
|
const url = new URL(location.href); url.searchParams.set('target', id)
|
|
history.pushState({}, '', url); dispatchEvent(new PopStateEvent('popstate'))
|
|
}, ids[0])
|
|
await expect(remove(page, ids[0])).toBeEnabled()
|
|
await remove(page, ids[0]).click(); await confirm(page).click()
|
|
await expect(page.getByRole('alertdialog')).toBeHidden()
|
|
const capabilitiesResponse = page.waitForResponse(response => response.url().endsWith('/models/capabilities'))
|
|
releaseCapabilities(); await capabilitiesResponse
|
|
await expect(page.getByRole('alertdialog')).toBeHidden()
|
|
expect(s.deletes).toHaveLength(1)
|
|
releaseRefresh()
|
|
await expect(reinstall(page)).toBeEnabled()
|
|
})
|
|
|
|
test('generic confirmation rechecks current row conflict', async ({ page }) => {
|
|
const s = await setup(page)
|
|
const globalDeletes = []
|
|
await page.route('**/backends/delete/**', route => {
|
|
globalDeletes.push(route.request().url())
|
|
return route.fulfill({ json: { message: 'removed' } })
|
|
})
|
|
await catalogTarget(page); await clearTarget(page); await genericRemove(page).click()
|
|
await expect(page.getByRole('alertdialog')).toBeVisible()
|
|
await page.route('**/api/operations', route => route.fulfill({ json: { operations: [{ name: BUILD, jobID: 'synthetic-job', isQueued: true }] } }))
|
|
await expect(page.locator(`[data-entity="${BUILD}"]`)).toContainText('Queued', { timeout: 15000 })
|
|
await confirm(page).click()
|
|
await expect(page.getByRole('alertdialog')).toBeHidden()
|
|
expect(globalDeletes).toEqual([])
|
|
expect(s.deletes).toEqual([])
|
|
})
|
|
|
|
for (const beforeConfirm of [false, true]) test(`generic deletion blocks targeted install ${beforeConfirm ? 'at confirmation' : 'after target clears'}`, async ({ page }) => {
|
|
let releaseInstall
|
|
const s = await setup(page, { installWait: new Promise(resolve => { releaseInstall = resolve }) })
|
|
const globalDeletes = []
|
|
await page.route('**/backends/delete/**', route => {
|
|
globalDeletes.push(route.request().url())
|
|
return route.fulfill({ json: { message: 'removed' } })
|
|
})
|
|
await catalogTarget(page)
|
|
if (beforeConfirm) {
|
|
await clearTarget(page); await genericRemove(page).click()
|
|
await expect(page.getByRole('alertdialog')).toBeVisible()
|
|
// Start the targeted request through its captured React handler while the
|
|
// existing generic confirmation remains open.
|
|
await page.evaluate(id => {
|
|
const url = new URL(location.href); url.searchParams.set('target', id)
|
|
history.pushState({}, '', url); dispatchEvent(new PopStateEvent('popstate'))
|
|
}, ids[0])
|
|
}
|
|
await reinstall(page).evaluate(el => el.click())
|
|
await expect.poll(() => s.installs.length).toBe(1)
|
|
if (beforeConfirm) {
|
|
await confirm(page).click()
|
|
await expect(page.getByRole('alertdialog')).toBeHidden()
|
|
} else {
|
|
await clearTarget(page)
|
|
await expect(genericRemove(page)).toBeDisabled()
|
|
await dispatchClick(genericRemove(page))
|
|
await expect(page.getByRole('alertdialog')).toBeHidden()
|
|
}
|
|
expect(globalDeletes).toEqual([])
|
|
expect(s.deletes).toEqual([])
|
|
releaseInstall()
|
|
})
|