GLOO4 testfall 5.1 Signed och 5.2 Unsigned - felaktiga?

[Bergquist,Daniel]

Testfallen 5.1 Signed och 5.2 Unsigned för GLOO4 ser inte ut att fungera korrekt, asserts ser väldigt märkliga ut. Syftet med testfallen är väl att kontrollera att tjänsten kan producera en post som är signerad, respektive en post som är osignerad. Har satt upp en testpatient med samtliga poster signerade, och där failar 5.1 medan 5.2 går passed.

Kontrollen i asserts gör mot kombinationen header/signature/id + headerr/accessControlHeader/lockTime. Varför kontrolleras inte header/signature/timestamp också? Enligt dokumentation (JoL Header Fältregler_v1.2) för står lockTime att “Vid signering ska lockTime anges till signeringstidpunkt.” Detta bör väl också kontrolleras i testfallen som behandlar signering?

‌

[Jan Söderman]

Testfall “5.1 Signed” testar att det finns ett id för personen som signerat posten och att det saknas låsningstidpunkt, så det borde vara fel eftersom låsningstidpunkten ska sättas vid signering. Det borde räcka att kontrollera om det finns ett signature.id.

Testfall “5.2 Unsigned“ testar att det finns ett id för personen som signerat posten och att det finns en låsningstidpunkt. Detta är fel. Det borde räcka att kontrollera att signature.id saknas och att locktime saknas.

testfall “5.3 Locked” testar att det inte finns ett id för personen som signerat posten och att det finns en låsningstidpunkt. Detta borde vara korrekt.

Beskrivning av regelverket finns i JoL Header Fältregler_v1.2

[Bergquist,Daniel]

@Jan Söderman “5.1 Signed”, borde man inte ha med en kontroll även låsningstid är satt (dvs signatur.id + lockTime)? Om både signature.id och signature.timestamp är satta, men lockTime saknas så riskerar man väl att släppa igenom sådana poster ifall man inte har en explicit assert på lockTime? Sen kanske det inte hade varit fel att dessutom ha en assert även på signature.timestamp i testfallet, blir ju en extra tydlighet (även om det inte behövs egentligen, just det fallet täcks in av schemavalideringen).

Inera: När skulle en uppdaterad testsvit kunna vara tillgänglig?

[Jan Söderman]

Jag har konsulterat några kollegor i ärendet och vår samlade bedömning är att:

5.1 Signed bör testa att signature.id har ett värde

5.2 Unsigned bör testa att signature.timestamp saknar ett värde och att locktime saknar ett värde

5.3 Locked bör testa att locktime har ett värde och att signature.id saknar värde

[Jan Söderman]

Jag har checkat in en rättning i Develop med följande kontroller:

5.1 Att signature.id, signature.timestamp och locktime alla har ett värde

5.2 Att signature.id, signature.timestamp och locktime alla saknar värden

5.2 Att signature.id och signature.timestamp saknar värden och att locktime har ett värde

[Tobias Blomberg]

Ärendet löst, se tidigare kommentar