Skip to content

Commit f371457

Browse files
test(validationDocument): add regression tests for null displayName
- Test acceptance of requested_by.displayName as null (missing profile info) - Test type validation still works (rejects non-string and missing) - Covers edge case where requester profile information is unavailable Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
1 parent 235fc39 commit f371457

1 file changed

Lines changed: 32 additions & 0 deletions

File tree

src/tests/services/validationDocument.spec.ts

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -246,4 +246,36 @@ describe('validationDocument', () => {
246246
expect(isLoadedValidationEnvelopeDocument(null)).toBe(false)
247247
expect(isLoadedValidationFileDocument(null)).toBe(false)
248248
})
249+
250+
it('accepts requested_by.displayName as null (missing profile info)', () => {
251+
// OpenAPI contract allows requested_by.displayName to be null
252+
// when requester profile info is not available
253+
const payload = createValidationPayload({
254+
requested_by: { userId: 'creator-user', displayName: null },
255+
})
256+
257+
const normalized = toValidationDocument(payload)
258+
259+
expect(normalized).not.toBeNull()
260+
expect(normalized?.requested_by).toEqual({
261+
userId: 'creator-user',
262+
displayName: null,
263+
})
264+
})
265+
266+
it('rejects requested_by when displayName is neither string nor null', () => {
267+
const payload = createValidationPayload({
268+
requested_by: { userId: 'creator-user', displayName: 123 },
269+
})
270+
271+
expect(toValidationDocument(payload)).toBeNull()
272+
})
273+
274+
it('rejects requested_by when displayName is missing', () => {
275+
const payload = createValidationPayload({
276+
requested_by: { userId: 'creator-user' },
277+
})
278+
279+
expect(toValidationDocument(payload)).toBeNull()
280+
})
249281
})

0 commit comments

Comments
 (0)