13541491615

Committed 26 Feb 2025 10:12AM UTC coverage: 90.703% (-0.09%) from 90.792%

Build # 13541491615

Build Type

Pull #2378

github

Committed by

hectorgomezv

Commit Message

Remove test:e2e:cov temporarily

Pull Request Pull Request #2378: refactor: confirmation refinement

Run Details

3050 of 3701 branches covered (82.41%)

Branch coverage included in aggregate %.

59 of 73 new or added lines in 2 files covered. (80.82%)

24 existing lines in 11 files now uncovered.

10423 of 11153 relevant lines covered (93.45%)

503.58 hits per line

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

95.0

/src/datasources/network/network.module.ts

import { Global, Module } from '@nestjs/common';
import { IConfigurationService } from '@/config/configuration.service.interface';
import { FetchNetworkService } from '@/datasources/network/fetch.network.service';
import { NetworkService } from '@/datasources/network/network.service.interface';
import { NetworkResponse } from '@/datasources/network/entities/network.response.entity';
import {
  NetworkRequestError,
  NetworkResponseError,
} from '@/datasources/network/entities/network.error.entity';
import type { Raw } from '@/validation/entities/raw.entity';

export type FetchClient = <T>(
  url: string,
  options: RequestInit,
) => Promise<NetworkResponse<T>>;

/**
 * Use this factory to create a {@link FetchClient} instance
 * that can be used to make HTTP requests.
 */
function fetchClientFactory(
  configurationService: IConfigurationService,
): FetchClient {
  const requestTimeout = configurationService.getOrThrow<number>(
    'httpClient.requestTimeout',
  );

  return async <T>(
    url: string,
    options: RequestInit,
  ): Promise<NetworkResponse<T>> => {
    let urlObject: URL | null = null;
    let response: Response | null = null;

    try {
      urlObject = new URL(url);
      response = await fetch(url, {
        ...options,
        signal: AbortSignal.timeout(requestTimeout),
        keepalive: true,
      });
    } catch (error) {
      throw new NetworkRequestError(urlObject, error);
    }

    // We validate data so don't need worry about casting `null` response
    const data = (await response.json().catch(() => null)) as Raw<T>;

    if (!response.ok) {
      throw new NetworkResponseError(urlObject, response, data);
    }

    return {
      status: response.status,
      data,
    };
  };
}

/**
 * A {@link Global} Module which provides HTTP support via {@link NetworkService}
 * Feature Modules don't need to import this module directly in order to inject
 * the {@link NetworkService}.
 *
 * This module should be included in the "root" application module
 */
@Global()
@Module({
  providers: [
    {
      provide: 'FetchClient',
      useFactory: fetchClientFactory,
      inject: [IConfigurationService],
    },
    { provide: NetworkService, useClass: FetchNetworkService },
  ],
  exports: [NetworkService],
})
export class NetworkModule {}

1	import { Global, Module } from '@nestjs/common';	106✔
2	import { IConfigurationService } from '@/config/configuration.service.interface';	106✔
3	import { FetchNetworkService } from '@/datasources/network/fetch.network.service';	106✔
4	import { NetworkService } from '@/datasources/network/network.service.interface';	106✔
5	import { NetworkResponse } from '@/datasources/network/entities/network.response.entity';
6	import {	106✔
7	NetworkRequestError,
8	NetworkResponseError,
9	} from '@/datasources/network/entities/network.error.entity';
10	import type { Raw } from '@/validation/entities/raw.entity';
11
12	export type FetchClient = <T>(
13	url: string,
14	options: RequestInit,
15	) => Promise<NetworkResponse<T>>;
16
17	/**
18	* Use this factory to create a {@link FetchClient} instance
19	* that can be used to make HTTP requests.
20	*/
21	function fetchClientFactory(
22	configurationService: IConfigurationService,
23	): FetchClient {
24	const requestTimeout = configurationService.getOrThrow<number>(	8✔
25	'httpClient.requestTimeout',
26	);
27
28	return async <T>(	8✔
29	url: string,
30	options: RequestInit,
31	): Promise<NetworkResponse<T>> => {
32	let urlObject: URL \| null = null;	8✔
33	let response: Response \| null = null;	8✔
34
35	try {	8✔
36	urlObject = new URL(url);	8✔
37	response = await fetch(url, {	6✔
38	...options,
39	signal: AbortSignal.timeout(requestTimeout),
40	keepalive: true,
41	});
42	} catch (error) {
43	throw new NetworkRequestError(urlObject, error);	4✔
44	}
45
46	// We validate data so don't need worry about casting `null` response
47	const data = (await response.json().catch(() => null)) as Raw<T>;	4✔
48
49	if (!response.ok) {	2✔
50	throw new NetworkResponseError(urlObject, response, data);	2✔
51	}
52
UNCOV 53	return {	×
54	status: response.status,
55	data,
56	};
57	};
58	}
59
60	/**
61	* A {@link Global} Module which provides HTTP support via {@link NetworkService}
62	* Feature Modules don't need to import this module directly in order to inject
63	* the {@link NetworkService}.
64	*
65	* This module should be included in the "root" application module
66	*/
67	@Global()
68	@Module({
69	providers: [
70	{
71	provide: 'FetchClient',
72	useFactory: fetchClientFactory,
73	inject: [IConfigurationService],
74	},
75	{ provide: NetworkService, useClass: FetchNetworkService },
76	],
77	exports: [NetworkService],
78	})
79	export class NetworkModule {}	106✔

safe-global / safe-client-gateway / 13541491615

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