5905542224

Committed 18 Aug 2023 05:45PM UTC coverage: 92.251% (-0.3%) from 92.551%

Build # 5905542224

Build Type

Pull #646

github

Committed by

web-flow

Commit Message

Merge 5b28a8b3c into 80a131dcb

Pull Request Pull Request #646: Coingecko prices migration poc

Run Details

971 of 1160 branches covered (83.71%)

Branch coverage included in aggregate %.

72 of 74 new or added lines in 9 files covered. (97.3%)

11 existing lines in 3 files now uncovered.

4422 of 4686 relevant lines covered (94.37%)

69.46 hits per line

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

86.36

/src/domain/prices/prices.repository.ts

import { Inject, Injectable } from '@nestjs/common';
import {
  ILoggingService,
  LoggingService,
} from '../../logging/logging.interface';
import { IPricesApi } from '../interfaces/prices-api.interface';
import { IPricesRepository } from './prices.repository.interface';

@Injectable()
export class PricesRepository implements IPricesRepository {
  constructor(
    @Inject(LoggingService) private readonly loggingService: ILoggingService,
    @Inject(IPricesApi) private readonly pricesApi: IPricesApi,
  ) {}

  async getNativeCoinPrice(args: {
    nativeCoinId: string;
    fiatCode: string;
  }): Promise<number | null> {
    const result = await this.pricesApi.getNativeCoinPrice(args);
    const { nativeCoinId, fiatCode } = args;
    const nativeCoinPrice = result?.[nativeCoinId]?.[fiatCode];

    if (!nativeCoinPrice) {
      this.loggingService.error(`Error getting ${nativeCoinId} price`);
    }

    return nativeCoinPrice ?? null;
  }

  async getTokenPrice(args: {
    nativeCoinId: string;
    tokenAddress: string;
    fiatCode: string;
  }): Promise<number | null> {
    const result = await this.pricesApi.getTokenPrice(args);
    const { tokenAddress, fiatCode } = args;
    const tokenPrice = result?.[tokenAddress.toLowerCase()]?.[fiatCode];

    if (!tokenPrice) {
      this.loggingService.warn(`Error getting ${tokenAddress} price`);
    }

    return tokenPrice ?? null;
  }
}

1	import { Inject, Injectable } from '@nestjs/common';	30✔
2	import {	30✔
3	ILoggingService,
4	LoggingService,
5	} from '../../logging/logging.interface';
6	import { IPricesApi } from '../interfaces/prices-api.interface';	30✔
7	import { IPricesRepository } from './prices.repository.interface';
8
9	@Injectable()
10	export class PricesRepository implements IPricesRepository {	30✔
11	constructor(
12	@Inject(LoggingService) private readonly loggingService: ILoggingService,	207✔
13	@Inject(IPricesApi) private readonly pricesApi: IPricesApi,	207✔
14	) {}
15
16	async getNativeCoinPrice(args: {
17	nativeCoinId: string;
18	fiatCode: string;
19	}): Promise<number \| null> {
20	const result = await this.pricesApi.getNativeCoinPrice(args);	2✔
21	const { nativeCoinId, fiatCode } = args;	2✔
22	const nativeCoinPrice = result?.[nativeCoinId]?.[fiatCode];	2!
23
24	if (!nativeCoinPrice) {	2!
NEW 25	this.loggingService.error(`Error getting ${nativeCoinId} price`);	×
26	}
27
28	return nativeCoinPrice ?? null;	2!
29	}
30
31	async getTokenPrice(args: {
32	nativeCoinId: string;
33	tokenAddress: string;
34	fiatCode: string;
35	}): Promise<number \| null> {
36	const result = await this.pricesApi.getTokenPrice(args);	5✔
37	const { tokenAddress, fiatCode } = args;	4✔
38	const tokenPrice = result?.[tokenAddress.toLowerCase()]?.[fiatCode];	4!
39
40	if (!tokenPrice) {	4✔
41	this.loggingService.warn(`Error getting ${tokenAddress} price`);	1✔
42	}
43
44	return tokenPrice ?? null;	4✔
45	}
46	}

safe-global / safe-client-gateway-nest / 5905542224

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