7695824089

Committed 29 Jan 2024 12:12PM UTC coverage: 93.281% (-0.1%) from 93.399%

Build # 7695824089

Build Type

Pull #1060

github

Committed by

iamacook

Commit Message

Don't log `typeof`

Pull Request Pull Request #1060: Prevent explicit use of `any`

Run Details

1460 of 1745 branches covered (0.0%)

Branch coverage included in aggregate %.

23 of 25 new or added lines in 13 files covered. (92.0%)

2 existing lines in 2 files now uncovered.

5579 of 5801 relevant lines covered (96.17%)

168.89 hits per line

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

47.37

/src/__tests__/test-app.provider.ts

import { INestApplication } from '@nestjs/common';
import { TestingModule } from '@nestjs/testing';
import {
  AppProvider,
  DEFAULT_CONFIGURATION,
  configureShutdownHooks,
} from '@/app.provider';

/**
 * A test {@link AppProvider}
 *
 * This provider provides an application given a {@link TestingModule}
 *
 * If the module provided is not a {@link TestingModule}, an error is thrown
 */
export class TestAppProvider<T> extends AppProvider<T> {
  // Disables shutdown hooks for tests (they are not required)
  // Enabling this in the tests might result in a MaxListenersExceededWarning
  // as the number of listeners that this adds exceed the default
  protected readonly configuration: ((app: INestApplication) => void)[] =
    DEFAULT_CONFIGURATION.filter((config) => config !== configureShutdownHooks);

  constructor() {
    super();
    if (process.env.NODE_ENV !== 'test') {
      throw Error('TestAppProvider used outside of a testing environment');
    }
  }

  protected getApp(module: T): Promise<INestApplication> {
    if (!(module instanceof TestingModule)) {
      const name =
        typeof module === 'function' ? module.constructor.name : module;
      return Promise.reject(`${name} is not a TestingModule`);
    }
    return Promise.resolve(module.createNestApplication());
  }
}

1	import { INestApplication } from '@nestjs/common';
2	import { TestingModule } from '@nestjs/testing';	82✔
3	import {	82✔
4	AppProvider,
5	DEFAULT_CONFIGURATION,
6	configureShutdownHooks,
7	} from '@/app.provider';
8
9	/**
10	* A test {@link AppProvider}
11	*
12	* This provider provides an application given a {@link TestingModule}
13	*
14	* If the module provided is not a {@link TestingModule}, an error is thrown
15	*/
16	export class TestAppProvider<T> extends AppProvider<T> {	82✔
17	// Disables shutdown hooks for tests (they are not required)
18	// Enabling this in the tests might result in a MaxListenersExceededWarning
19	// as the number of listeners that this adds exceed the default
20	protected readonly configuration: ((app: INestApplication) => void)[] =	550✔
21	DEFAULT_CONFIGURATION.filter((config) => config !== configureShutdownHooks);	2,200✔
22
23	constructor() {
24	super();	550✔
25	if (process.env.NODE_ENV !== 'test') {	550!
26	throw Error('TestAppProvider used outside of a testing environment');	×
27	}
28	}
29
30	protected getApp(module: T): Promise<INestApplication> {
31	if (!(module instanceof TestingModule)) {	550!
32	const name =
NEW 33	typeof module === 'function' ? module.constructor.name : module;	×
NEW 34	return Promise.reject(`${name} is not a TestingModule`);	×
35	}
36	return Promise.resolve(module.createNestApplication());	550✔
37	}
38	}

safe-global / safe-client-gateway / 7695824089

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