Skip to content

Commit 1bf1f8b

Browse files
committed
test: align identify signer component spec
Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com> (cherry picked from commit 3431624)
1 parent 0a27289 commit 1bf1f8b

1 file changed

Lines changed: 5 additions & 7 deletions

File tree

src/tests/components/Request/IdentifySigner.spec.ts

Lines changed: 5 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,6 @@ type IdentifyMethodConfig = {
4242
type SignerToEdit = {
4343
displayName?: string
4444
description?: string
45-
identify?: string
4645
identifyMethods?: Array<{ method: string; value: string }>
4746
}
4847

@@ -160,7 +159,6 @@ describe('IdentifySigner rules', () => {
160159
it('treats signerToEdit as existing signer when populated', async () => {
161160
const signer = {
162161
displayName: 'John Doe',
163-
identify: 'john@example.com',
164162
identifyMethods: [{ method: 'email', value: 'john@example.com' }],
165163
}
166164

@@ -412,7 +410,7 @@ describe('IdentifySigner rules', () => {
412410

413411
it('sends signer list to save request', async () => {
414412
filesStore.getFile.mockReturnValueOnce({
415-
signers: [{ identify: { email: 'existing@example.com' } }],
413+
signers: [{ identifyMethods: [{ method: 'email', value: 'existing@example.com', mandatory: 0 }] }],
416414
})
417415
wrapper.vm.identifyMethod = 'email'
418416
wrapper.vm.displayName = 'John Doe'
@@ -423,11 +421,11 @@ describe('IdentifySigner rules', () => {
423421

424422
expect(filesStore.saveOrUpdateSignatureRequest).toHaveBeenCalledWith({
425423
signers: [
426-
{ identify: { email: 'existing@example.com' } },
424+
{ identifyMethods: [{ method: 'email', value: 'existing@example.com', mandatory: 0 }] },
427425
{
428426
displayName: 'John Doe',
429427
description: undefined,
430-
identify: 'john@example.com',
428+
email: 'john@example.com',
431429
status: 0,
432430
statusText: 'Draft',
433431
identifyMethods: [{
@@ -518,7 +516,6 @@ describe('IdentifySigner rules', () => {
518516
const signer = {
519517
displayName: 'Jane Doe',
520518
description: 'Please review',
521-
identify: 'jane@example.com',
522519
identifyMethods: [{ method: 'email', value: 'jane@example.com' }],
523520
}
524521

@@ -529,12 +526,13 @@ describe('IdentifySigner rules', () => {
529526

530527
expect(wrapper.vm.displayName).toBe('Jane Doe')
531528
expect(wrapper.vm.description).toBe('Please review')
529+
expect(wrapper.vm.identify).toBe('jane@example.com')
532530
})
533531

534532
it('ignores legacy signRequestId fallback when identify is missing', () => {
535533
wrapper = createWrapper({
536534
signerToEdit: {
537-
displayName: 'Legacy signer',
535+
displayName: 'Signer without methods',
538536
identifyMethods: [],
539537
...( { signRequestId: 123 } as unknown as SignerToEdit ),
540538
},

0 commit comments

Comments
 (0)