8833776871

Committed 25 Apr 2024 01:47PM UTC coverage: 91.904% (-0.5%) from 92.38%

Build # 8833776871

Build Type

push

github

Committed by

web-flow

Commit Message

Rename `JwtRepository` to `AuthRepository` (#1456)

Renames `JwtRepository` to `AuthRepository` as the logic is customised to our auth implementation:

- Rename `JwtRepository` and interface to `AuthRepository`
- Rename `JwtAccessTokenPayload` to `AuthPayload`, as well as associated builder
- Rename `JwtAccessTokenPayloadSchema` to `AuthPayload`
- Amend associated tests with appropriate naming

Run Details

1915 of 2348 branches covered (81.56%)

Branch coverage included in aggregate %.

20 of 36 new or added lines in 8 files covered. (55.56%)

1 existing line in 1 file now uncovered.

6883 of 7225 relevant lines covered (95.27%)

306.41 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

0.0

/src/domain/auth/entities/__tests__/auth-payload.entity.ts

NEW 1	import { AuthPayloadSchema } from '@/domain/auth/entities/auth-payload.entity';	×
NEW 2	import { authPayloadBuilder } from '@/domain/auth/entities/__tests__/auth-payload.entity.builder';	×
3	import { faker } from '@faker-js/faker';	×
4	import { getAddress } from 'viem';	×
5
NEW 6	describe('AuthPayloadSchema', () => {	×
NEW 7	it('should parse a valid AuthPayloadSchema', () => {	×
NEW 8	const authPayload = authPayloadBuilder().build();	×
9
NEW 10	const result = AuthPayloadSchema.safeParse(authPayload);	×
11
12	expect(result.success).toBe(true);	×
13	// Address did not checksum as it already way
NEW 14	expect(result.success && result.data).toStrictEqual(authPayload);	×
15	});
16
17	it('should checksum the signer_address', () => {	×
18	const nonChecksummedAddress = faker.finance.ethereumAddress().toLowerCase();	×
NEW 19	const authPayload = authPayloadBuilder()	×
20	.with('signer_address', nonChecksummedAddress as `0x${string}`)
21	.build();
22
NEW 23	const result = AuthPayloadSchema.safeParse(authPayload);	×
24
25	expect(result.success && result.data.signer_address).toBe(	×
26	getAddress(nonChecksummedAddress),
27	);
28	});
29
30	it('should not allow a non-numeric chain_id', () => {	×
NEW 31	const authPayload = authPayloadBuilder()	×
32	.with('chain_id', faker.lorem.word())
33	.build();
34
NEW 35	const result = AuthPayloadSchema.safeParse(authPayload);	×
36
37	expect(result.success).toBe(false);	×
38	expect(!result.success && result.error.issues).toStrictEqual([	×
39	{
40	code: 'custom',
41	message: 'Invalid base-10 numeric string',
42	path: ['chain_id'],
43	},
44	]);
45	});
46
47	it('should not allow a non-address signer_address', () => {	×
NEW 48	const authPayload = authPayloadBuilder()	×
49	.with('signer_address', faker.lorem.word() as `0x${string}`)
50	.build();
51
NEW 52	const result = AuthPayloadSchema.safeParse(authPayload);	×
53
54	expect(result.success).toBe(false);	×
55	expect(!result.success && result.error.issues).toStrictEqual([	×
56	{
57	code: 'custom',
58	message: 'Invalid address',
59	path: ['signer_address'],
60	},
61	]);
62	});
63
NEW 64	it('should not parse an invalid AuthPayloadSchema', () => {	×
NEW 65	const authPayload = {	×
66	unknown: 'payload',
67	};
68
NEW 69	const result = AuthPayloadSchema.safeParse(authPayload);	×
70
71	expect(result.success).toBe(false);	×
72	expect(!result.success && result.error.issues).toStrictEqual([	×
73	{
74	code: 'invalid_type',
75	expected: 'string',
76	message: 'Required',
77	path: ['chain_id'],
78	received: 'undefined',
79	},
80	{
81	code: 'invalid_type',
82	expected: 'string',
83	message: 'Required',
84	path: ['signer_address'],
85	received: 'undefined',
86	},
87	]);
88	});
89	});

safe-global / safe-client-gateway / 8833776871

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous