Merge pull request #1412 from pikasTech/fix/1400-dashboard-trigger-status
fix(web-sentinel): pass dashboard trigger without screenshot
This commit is contained in:
@@ -578,7 +578,7 @@ export function probeSentinelDashboardBrowser(state: SentinelCicdState, options:
|
||||
const artifacts = Array.isArray(transport.artifacts) ? transport.artifacts.map(record).map(compactDashboardArtifact) : [];
|
||||
const screenshot = artifacts.find((artifact) => typeof artifact.localPath === "string" && String(artifact.localPath).endsWith(".png")) ?? null;
|
||||
const browserOk = page?.ok === true;
|
||||
const screenshotOk = options.action === "verify" || screenshot !== null && screenshot.verified === true;
|
||||
const screenshotOk = options.action !== "screenshot" || screenshot !== null && screenshot.verified === true;
|
||||
const ok = result.exitCode === 0 && transport.ok === true && browserOk && screenshotOk;
|
||||
return {
|
||||
ok,
|
||||
|
||||
Reference in New Issue
Block a user