#299

Committed 01 Apr 2024 08:41PM UTC coverage: 90.321% (-0.1%) from 90.465%

Build # #299

Build Type

push

Committed by

Commit Message

Implemented `roPath` component and fixed Interpreter Comparisons (#50)

* Added option to disable colors and fixed unit tests to not fail with colors

* Implemented `roPath` and fixed comparison to match Roku behavior

* Added unit tests and fixed prettier issues

* Properly compare Number with Boolean

* Added support to AND and OR between Boolean and Numbers

* Updated unit tests and fixed Int64

* Improved URL regex for CLI colorizer

Run Details

1881 of 2239 branches covered (84.01%)

Branch coverage included in aggregate %.

167 of 181 new or added lines in 11 files covered. (92.27%)

5 existing lines in 3 files now uncovered.

5519 of 5954 relevant lines covered (92.69%)

8637.25 hits per line

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

88.89

/src/brsTypes/components/BrsComponent.ts

import { BrsType } from "..";
import { BrsInvalid, ValueKind, BrsValue } from "../BrsType";
import { Callable } from "../Callable";
import { BrsInterface } from "../BrsInterface";

export class BrsComponent {
    private methods: Map<string, Callable> = new Map<string, Callable>();
    private readonly componentName: string;

    readonly interfaces = new Map<string, BrsInterface>();

    constructor(name: string) {
        this.componentName = name;
    }

    /**
     * Returns the name of the component, used to create instances via `createObject`.
     * @returns the name of this component.
     */
    getComponentName(): string {
        return this.componentName;
    }

    hasInterface(interfaceName: string) {
        return this.interfaces.has(interfaceName.toLowerCase());
    }

    protected registerMethods(interfaces: Record<string, Callable[]>) {
        Object.entries(interfaces).forEach(([interfaceName, methods]) => {
            let interfaceKey = interfaceName.toLowerCase();
            if (!this.interfaces.has(interfaceKey)) {
                this.interfaces.set(interfaceKey, new BrsInterface(interfaceName, methods));
            }

            this.appendMethods(methods);
        });
    }

    /** Appends a method to the component. */
    appendMethod(index: string, method: Callable) {
        this.methods.set(index.toLowerCase(), method);
    }

    /** Given a list of methods, appends all of them to the component. */
    appendMethods(methods: Callable[]) {
        methods.forEach((m) => this.methods.set((m.name || "").toLowerCase(), m));
    }

    getMethod(index: string): Callable | undefined {
        return this.methods.get(index.toLowerCase());
    }
}

/** Represents a BrightScript component that has elements that can be iterated across. */
export interface BrsIterable {
    /**
     * Returns the set of iterable elements contained in this component.
     * @returns an array of elements contained in this component.
     */
    getElements(): ReadonlyArray<BrsType>;

    /**
     * Retrieves an element from this component at the provided `index`.
     * @param index the index in this component from which to retrieve the desired element.
     * @param isCaseSensitive determinate whether operation of getting should be case sensitive or not.
     * @returns the element at `index` if one exists, otherwise throws an Error.
     */
    get(index: BrsType, isCaseSensitive?: boolean): BrsType;

    set(index: BrsType, value: BrsType, isCaseSensitive?: boolean): BrsInvalid;
}

1	import { BrsType } from "..";
2	import { BrsInvalid, ValueKind, BrsValue } from "../BrsType";
3	import { Callable } from "../Callable";
4	import { BrsInterface } from "../BrsInterface";	141✔
5
6	export class BrsComponent {	141✔
7	private methods: Map<string, Callable> = new Map<string, Callable>();	15,477✔
8	private readonly componentName: string;
9
10	readonly interfaces = new Map<string, BrsInterface>();	15,477✔
11
12	constructor(name: string) {
13	this.componentName = name;	15,477✔
14	}
15
16	/**
17	* Returns the name of the component, used to create instances via `createObject`.
18	* @returns the name of this component.
19	*/
20	getComponentName(): string {
21	return this.componentName;	169✔
22	}
23
24	hasInterface(interfaceName: string) {
NEW 25	return this.interfaces.has(interfaceName.toLowerCase());	×
26	}
27
28	protected registerMethods(interfaces: Record<string, Callable[]>) {
29	Object.entries(interfaces).forEach(([interfaceName, methods]) => {	15,410✔
30	let interfaceKey = interfaceName.toLowerCase();	41,086✔
31	if (!this.interfaces.has(interfaceKey)) {	41,086✔
32	this.interfaces.set(interfaceKey, new BrsInterface(interfaceName, methods));	41,026✔
33	}
34
35	this.appendMethods(methods);	41,086✔
36	});
37	}
38
39	/** Appends a method to the component. */
40	appendMethod(index: string, method: Callable) {
41	this.methods.set(index.toLowerCase(), method);	6✔
42	}
43
44	/** Given a list of methods, appends all of them to the component. */
45	appendMethods(methods: Callable[]) {
46	methods.forEach((m) => this.methods.set((m.name \|\| "").toLowerCase(), m));	269,927!
47	}
48
49	getMethod(index: string): Callable \| undefined {
50	return this.methods.get(index.toLowerCase());	1,965✔
51	}
52	}
53
54	/** Represents a BrightScript component that has elements that can be iterated across. */
55	export interface BrsIterable {
56	/**
57	* Returns the set of iterable elements contained in this component.
58	* @returns an array of elements contained in this component.
59	*/
60	getElements(): ReadonlyArray<BrsType>;
61
62	/**
63	* Retrieves an element from this component at the provided `index`.
64	* @param index the index in this component from which to retrieve the desired element.
65	* @param isCaseSensitive determinate whether operation of getting should be case sensitive or not.
66	* @returns the element at `index` if one exists, otherwise throws an Error.
67	*/
68	get(index: BrsType, isCaseSensitive?: boolean): BrsType;
69
70	set(index: BrsType, value: BrsType, isCaseSensitive?: boolean): BrsInvalid;
71	}

rokucommunity / brs / #299

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