From 1f4111d6f279ea351711fe3360d66af882ec59a7 Mon Sep 17 00:00:00 2001 From: Jake Barnby Date: Wed, 6 May 2026 18:34:04 +1200 Subject: [PATCH] test(insights): drop summary update from testUpdate MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit User Update only accepts severity + status now — analyzer-controlled fields like summary flow through the manager Create endpoint. The previous testUpdate sent `summary: 'Updated summary.'` and asserted it changed; Utopia silently dropped the unknown param so the assertion would have failed. Trim the call to severity-only and verify the analyzer fields (including summary) are preserved on the response. Co-Authored-By: Claude Opus 4.7 (1M context) --- tests/e2e/Services/Insights/InsightsBase.php | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/tests/e2e/Services/Insights/InsightsBase.php b/tests/e2e/Services/Insights/InsightsBase.php index d0ddaac89d..efc00641df 100644 --- a/tests/e2e/Services/Insights/InsightsBase.php +++ b/tests/e2e/Services/Insights/InsightsBase.php @@ -686,15 +686,16 @@ trait InsightsBase $updated = $this->updateInsight($data['insightId'], [ 'severity' => 'critical', - 'summary' => 'Updated summary.', ]); $this->assertSame(200, $updated['headers']['status-code']); $this->assertSame('critical', $updated['body']['severity']); - $this->assertSame('Updated summary.', $updated['body']['summary']); - // Untouched fields preserved (regression for partial-document overwrite) + // Analyzer-controlled fields preserved (regression for partial-document + // overwrite). User Update only takes `severity` and `status`; everything + // else flows through the manager Create endpoint. $this->assertSame($original['title'], $updated['body']['title']); + $this->assertSame($original['summary'], $updated['body']['summary']); $this->assertSame($original['type'], $updated['body']['type']); $this->assertSame($original['resourceType'], $updated['body']['resourceType']); $this->assertSame($original['resourceId'], $updated['body']['resourceId']);