12376268926

Committed 17 Dec 2024 03:34PM UTC coverage: 90.16% (-0.02%) from 90.176%

Build # 12376268926

Build Type

push

github

Committed by

web-flow

Commit Message

Ensure chainId is returned correctly during notification errors (#2201)

Run Details

2824 of 3509 branches covered (80.48%)

Branch coverage included in aggregate %.

5 of 5 new or added lines in 1 file covered. (100.0%)

2 existing lines in 2 files now uncovered.

9628 of 10302 relevant lines covered (93.46%)

438.51 hits per line

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

93.75

/src/datasources/transaction-api/transaction-api.manager.ts

import { Inject, Injectable } from '@nestjs/common';
import { IConfigurationService } from '@/config/configuration.service.interface';
import { CacheFirstDataSource } from '@/datasources/cache/cache.first.data.source';
import {
  CacheService,
  ICacheService,
} from '@/datasources/cache/cache.service.interface';
import { HttpErrorFactory } from '@/datasources/errors/http-error-factory';
import {
  INetworkService,
  NetworkService,
} from '@/datasources/network/network.service.interface';
import { TransactionApi } from '@/datasources/transaction-api/transaction-api.service';
import { ChainSchema } from '@/domain/chains/entities/schemas/chain.schema';
import { IConfigApi } from '@/domain/interfaces/config-api.interface';
import { ITransactionApiManager } from '@/domain/interfaces/transaction-api.manager.interface';
import { ILoggingService, LoggingService } from '@/logging/logging.interface';

@Injectable()
export class TransactionApiManager implements ITransactionApiManager {
  private transactionApiMap: Record<string, TransactionApi> = {};

  private readonly useVpcUrl: boolean;

  constructor(
    @Inject(IConfigurationService)
    private readonly configurationService: IConfigurationService,
    @Inject(IConfigApi) private readonly configApi: IConfigApi,
    private readonly dataSource: CacheFirstDataSource,
    @Inject(CacheService) private readonly cacheService: ICacheService,
    private readonly httpErrorFactory: HttpErrorFactory,
    @Inject(NetworkService) private readonly networkService: INetworkService,
    @Inject(LoggingService) private readonly loggingService: ILoggingService,
  ) {
    this.useVpcUrl = this.configurationService.getOrThrow<boolean>(
      'safeTransaction.useVpcUrl',
    );
  }

  async getApi(chainId: string): Promise<TransactionApi> {
    const transactionApi = this.transactionApiMap[chainId];
    if (transactionApi !== undefined) return transactionApi;

    const chain = await this.configApi
      .getChain(chainId)
      .then(ChainSchema.parse);
    this.transactionApiMap[chainId] = new TransactionApi(
      chainId,
      this.useVpcUrl ? chain.vpcTransactionService : chain.transactionService,
      this.dataSource,
      this.cacheService,
      this.configurationService,
      this.httpErrorFactory,
      this.networkService,
      this.loggingService,
    );
    return this.transactionApiMap[chainId];
  }

  destroyApi(chainId: string): void {
    if (this.transactionApiMap[chainId] !== undefined) {
      delete this.transactionApiMap[chainId];
    }
  }
}

1	import { Inject, Injectable } from '@nestjs/common';	154✔
2	import { IConfigurationService } from '@/config/configuration.service.interface';	154✔
3	import { CacheFirstDataSource } from '@/datasources/cache/cache.first.data.source';	154✔
4	import {	154✔
5	CacheService,
6	ICacheService,
7	} from '@/datasources/cache/cache.service.interface';
8	import { HttpErrorFactory } from '@/datasources/errors/http-error-factory';	154✔
9	import {	154✔
10	INetworkService,
11	NetworkService,
12	} from '@/datasources/network/network.service.interface';
13	import { TransactionApi } from '@/datasources/transaction-api/transaction-api.service';	154✔
14	import { ChainSchema } from '@/domain/chains/entities/schemas/chain.schema';	154✔
15	import { IConfigApi } from '@/domain/interfaces/config-api.interface';	154✔
16	import { ITransactionApiManager } from '@/domain/interfaces/transaction-api.manager.interface';
17	import { ILoggingService, LoggingService } from '@/logging/logging.interface';	154✔
18
19	@Injectable()
20	export class TransactionApiManager implements ITransactionApiManager {	154✔
21	private transactionApiMap: Record<string, TransactionApi> = {};	1,274✔
22
23	private readonly useVpcUrl: boolean;
24
25	constructor(
26	@Inject(IConfigurationService)
27	private readonly configurationService: IConfigurationService,	1,274✔
28	@Inject(IConfigApi) private readonly configApi: IConfigApi,	1,274✔
29	private readonly dataSource: CacheFirstDataSource,	1,274✔
30	@Inject(CacheService) private readonly cacheService: ICacheService,	1,274✔
31	private readonly httpErrorFactory: HttpErrorFactory,	1,274✔
32	@Inject(NetworkService) private readonly networkService: INetworkService,	1,274✔
33	@Inject(LoggingService) private readonly loggingService: ILoggingService,	1,274✔
34	) {
35	this.useVpcUrl = this.configurationService.getOrThrow<boolean>(	1,274✔
36	'safeTransaction.useVpcUrl',
37	);
38	}
39
40	async getApi(chainId: string): Promise<TransactionApi> {
41	const transactionApi = this.transactionApiMap[chainId];	3,650✔
42	if (transactionApi !== undefined) return transactionApi;	3,650✔
43
44	const chain = await this.configApi	1,044✔
45	.getChain(chainId)
46	.then(ChainSchema.parse);
47	this.transactionApiMap[chainId] = new TransactionApi(	1,008✔
48	chainId,
49	this.useVpcUrl ? chain.vpcTransactionService : chain.transactionService,	504✔
50	this.dataSource,
51	this.cacheService,
52	this.configurationService,
53	this.httpErrorFactory,
54	this.networkService,
55	this.loggingService,
56	);
57	return this.transactionApiMap[chainId];	1,008✔
58	}
59
60	destroyApi(chainId: string): void {
61	if (this.transactionApiMap[chainId] !== undefined) {	2!
UNCOV 62	delete this.transactionApiMap[chainId];	×
63	}
64	}
65	}

safe-global / safe-client-gateway / 12376268926

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