7019649399

Committed 28 Nov 2023 01:43PM UTC coverage: 89.705% (-1.5%) from 91.236%

Build # 7019649399

Build Type

push

github

Committed by

web-flow

Commit Message

Add a configurable webhook execution delay (#883)

Changes:
- Adds a configurable (via the environment variable `WEB_HOOK_EXECUTION_DELAY_MS`) delay before executing web hook. The default value for this is 5 seconds.
- The status code returned by the controller was changed to 202 as it is more accurate given the new behavior.

Run Details

1417 of 1711 branches covered (0.0%)

Branch coverage included in aggregate %.

6 of 7 new or added lines in 1 file covered. (85.71%)

98 existing lines in 10 files now uncovered.

5223 of 5691 relevant lines covered (91.78%)

151.71 hits per line

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

91.67

/src/domain/chains/chains.repository.ts

import { Inject, Injectable } from '@nestjs/common';
import { IChainsRepository } from '@/domain/chains/chains.repository.interface';
import { ChainsValidator } from '@/domain/chains/chains.validator';
import { Chain } from '@/domain/chains/entities/chain.entity';
import { MasterCopy } from '@/domain/chains/entities/master-copies.entity';
import { MasterCopyValidator } from '@/domain/chains/master-copy.validator';
import { Page } from '@/domain/entities/page.entity';
import { IConfigApi } from '@/domain/interfaces/config-api.interface';
import { ITransactionApiManager } from '@/domain/interfaces/transaction-api.manager.interface';

@Injectable()
export class ChainsRepository implements IChainsRepository {
  constructor(
    @Inject(IConfigApi) private readonly configApi: IConfigApi,
    @Inject(ITransactionApiManager)
    private readonly transactionApiManager: ITransactionApiManager,
    private readonly chainValidator: ChainsValidator,
    private readonly masterCopyValidator: MasterCopyValidator,
  ) {}

  async getChain(chainId: string): Promise<Chain> {
    const chain = await this.configApi.getChain(chainId);
    return this.chainValidator.validate(chain);
  }

  async clearChain(chainId: string): Promise<void> {
    return this.configApi.clearChain(chainId);
  }

  async getChains(limit?: number, offset?: number): Promise<Page<Chain>> {
    const page = await this.configApi.getChains({ limit, offset });
    page?.results.map((result) => this.chainValidator.validate(result));
    return page;
  }

  async getMasterCopies(chainId: string): Promise<MasterCopy[]> {
    const transactionApi =
      await this.transactionApiManager.getTransactionApi(chainId);
    const masterCopies = await transactionApi.getMasterCopies();
    return masterCopies.map((masterCopy) =>
      this.masterCopyValidator.validate(masterCopy),
    );
  }
}

1	import { Inject, Injectable } from '@nestjs/common';	70✔
2	import { IChainsRepository } from '@/domain/chains/chains.repository.interface';
3	import { ChainsValidator } from '@/domain/chains/chains.validator';	70✔
4	import { Chain } from '@/domain/chains/entities/chain.entity';
5	import { MasterCopy } from '@/domain/chains/entities/master-copies.entity';
6	import { MasterCopyValidator } from '@/domain/chains/master-copy.validator';	70✔
7	import { Page } from '@/domain/entities/page.entity';
8	import { IConfigApi } from '@/domain/interfaces/config-api.interface';	70✔
9	import { ITransactionApiManager } from '@/domain/interfaces/transaction-api.manager.interface';	70✔
10
11	@Injectable()
12	export class ChainsRepository implements IChainsRepository {	70✔
13	constructor(
14	@Inject(IConfigApi) private readonly configApi: IConfigApi,	516✔
15	@Inject(ITransactionApiManager)
16	private readonly transactionApiManager: ITransactionApiManager,	516✔
17	private readonly chainValidator: ChainsValidator,	516✔
18	private readonly masterCopyValidator: MasterCopyValidator,	516✔
19	) {}
20
21	async getChain(chainId: string): Promise<Chain> {
22	const chain = await this.configApi.getChain(chainId);	90✔
23	return this.chainValidator.validate(chain);	82✔
24	}
25
26	async clearChain(chainId: string): Promise<void> {
UNCOV 27	return this.configApi.clearChain(chainId);	×
28	}
29
30	async getChains(limit?: number, offset?: number): Promise<Page<Chain>> {
31	const page = await this.configApi.getChains({ limit, offset });	8✔
32	page?.results.map((result) => this.chainValidator.validate(result));	14!
33	return page;	4✔
34	}
35
36	async getMasterCopies(chainId: string): Promise<MasterCopy[]> {
37	const transactionApi =
38	await this.transactionApiManager.getTransactionApi(chainId);	62✔
39	const masterCopies = await transactionApi.getMasterCopies();	60✔
40	return masterCopies.map((masterCopy) =>	58✔
41	this.masterCopyValidator.validate(masterCopy),	60✔
42	);
43	}
44	}

safe-global / safe-client-gateway / 7019649399

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