3247987602

Build Type

github

Committed by GitHub

Commit Message

Merge f7a306232 into 0ed9dcad2

Pull Request Pull Request #973: Improve test coverage

Run Details

1535 of 3546 branches covered (43.29%)

Branch coverage included in aggregate %.

273 of 285 new or added lines in 13 files covered. (95.79%)

1 existing line in 1 file now uncovered.

6566 of 10330 relevant lines covered (63.56%)

49.09 hits per line

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

88.24

/src/lib/util/promise-utils.ts

import { EventEmitter } from "events";

export function PromiseTimeout(timeout: number): Promise<void> {
  return new Promise<void>(resolve => {
    setTimeout(() => resolve(), timeout);
  });
}

export function awaitEventOnce<Obj extends EventEmitter, Event extends string, T>(element: Obj, event: Event, timeout?: number): Promise<T>;
export function awaitEventOnce<Obj extends EventEmitter, Event extends string>(element: Obj, event: Event, timeout?: number): Promise<void>;
// eslint-disable-next-line @typescript-eslint/ban-types
export function awaitEventOnce<Object extends EventEmitter, Event extends string, T>(element: Object, event: Event, timeout = 5000): Promise<void | T> {
  return new Promise<void | T>((resolve, reject) => {
    // eslint-disable-next-line prefer-const
    let timeoutId: NodeJS.Timeout;

    // eslint-disable-next-line @typescript-eslint/no-explicit-any
    const resolveListener = (...args: any) => {
      clearTimeout(timeoutId);

      resolve(args.length ? (args.length === 1 ? args[0] : args) : undefined);
    };

    timeoutId = setTimeout(() => {
      element.removeListener(event, resolveListener);
      reject(new Error(`awaitEvent for event ${event} timed out!`));
    }, timeout);

    element.once(event, resolveListener);
  });
}


1	import { EventEmitter } from "events";
2
3	export function PromiseTimeout(timeout: number): Promise<void> {	15✔
4	return new Promise<void>(resolve => {	72✔
5	setTimeout(() => resolve(), timeout);	72✔
6	});
7	}
8
9	export function awaitEventOnce<Obj extends EventEmitter, Event extends string, T>(element: Obj, event: Event, timeout?: number): Promise<T>;
10	export function awaitEventOnce<Obj extends EventEmitter, Event extends string>(element: Obj, event: Event, timeout?: number): Promise<void>;
11	// eslint-disable-next-line @typescript-eslint/ban-types
12	export function awaitEventOnce<Object extends EventEmitter, Event extends string, T>(element: Object, event: Event, timeout = 5000): Promise<void \| T> {	264✔
13	return new Promise<void \| T>((resolve, reject) => {	132✔
14	// eslint-disable-next-line prefer-const
15	let timeoutId: NodeJS.Timeout;
16
17	// eslint-disable-next-line @typescript-eslint/no-explicit-any
18	const resolveListener = (...args: any) => {	468✔
19	clearTimeout(timeoutId);	132✔
20
21	resolve(args.length ? (args.length === 1 ? args[0] : args) : undefined);	132✔
22	};
23
24	timeoutId = setTimeout(() => {	132✔
NEW 25	element.removeListener(event, resolveListener);	×
NEW 26	reject(new Error(`awaitEvent for event ${event} timed out!`));	×
27	}, timeout);
28
29	element.once(event, resolveListener);	132✔
30	});
31	}
32

homebridge / HAP-NodeJS / 3247987602

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