Skip to content

test(persistence): Scan.contentSha256 in Test-Fixture#5

Merged
sebolber merged 1 commit into
mainfrom
claude/fix-scan-content-sha
May 13, 2026
Merged

test(persistence): Scan.contentSha256 in Test-Fixture#5
sebolber merged 1 commit into
mainfrom
claude/fix-scan-content-sha

Conversation

@sebolber

Copy link
Copy Markdown
Owner

Warum

Im main-Lauf nach PR #4 ist cvm-persistence weiter rot, aber an einer anderen Stelle: AssessmentImmutableTest.bereiteAssessmentVor:94 scheitert am NOT-NULL-Constraint Scan.contentSha256.

ProductRepositoryIntegrationsTest und AiEmbeddingPgvectorTest sind grün, also wirkt die tenantId-Korrektur. ProductRepositoryIntegrationsTest-Hibernate-Inserts sind im Log sichtbar.

Fix

Ein zusaetzliches Feld auf dem Scan-Fixture-Builder in AssessmentImmutableTest:

.contentSha256("sha256:imm-content")

NOT-NULL-Audit der restlichen Fixture-Entities ergab keine weiteren Luecken: Environment, Component, ComponentOccurrence, Finding, Assessment, ProductVersion sind bereits vollstaendig. Cve.kevListed ist NOT NULL, aber die Entity hat ein @PrePersist, das auf FALSE faellt, wenn null - kein Eingriff noetig.

Erwartung

Nach Merge: cvm-persistence komplett gruen, dann laufen cvm-application, cvm-integration, cvm-ai-services, cvm-api, cvm-app und cvm-architecture-tests endlich durch. Mit --fail-at-end sehen wir in einem Lauf alle ggf. noch vorhandenen Failures.

https://claude.ai/code/session_014NNjRKw5bDqRffqCUGe8ud


Generated by Claude Code

ProductRepositoryIntegrationsTest und AiEmbeddingPgvectorTest sind jetzt
gruen, aber AssessmentImmutableTest scheitert am naechsten NOT-NULL
auf Scan.contentSha256 (Pflichtfeld seit der Content-Hash-Migration).

Cve.kevListed bleibt unberuehrt: dort setzt @PrePersist
implizit FALSE, wenn null.

CVM-62

https://claude.ai/code/session_014NNjRKw5bDqRffqCUGe8ud
@sebolber sebolber merged commit 1d3f1ed into main May 13, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants