#15903

Committed 11 May 2026 06:41PM UTC coverage: 86.896% (-2.2%) from 89.094%

Build Type

push

Committed by

Commit Message

Merge 70dfd6181 into ce68f5cb7

Coverage Stats

15597 of 18958 branches covered (82.27%)

Branch coverage included in aggregate %.

9 of 9 new or added lines in 3 files covered. (100.0%)

955 existing lines in 53 files now uncovered.

16351 of 17808 relevant lines covered (91.82%)

27326.16 hits per line

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

59.38

/src/bscPlugin/codeActions/codeActionHelpers.ts

import type { XmlFile } from '../../files/XmlFile';
import type { BsDiagnostic } from '../../interfaces';
import { util } from '../../util';

/**
 * Returns the position at which an `extends` attribute should be inserted for
 * a component that is missing one: after the last existing attribute, or after
 * the `<component` tag itself if there are no attributes yet.
 */
export function getMissingExtendsInsertPosition(file: XmlFile) {
    const componentElement = file.parser.ast.componentElement;
    const lastAttribute = componentElement.attributes[componentElement.attributes.length - 1];
    const range = lastAttribute?.location?.range ?? componentElement.tokens.startTagName?.location?.range;
    return range?.end;
}

/**
 * Returns the delete change that removes the return value from a `return <expr>`
 * statement flagged by `voidFunctionMayNotReturnValue`. Leaves the bare `return`
 * keyword in place.
 */
export function getRemoveReturnValueChange(diagnostic: BsDiagnostic, filePath: string) {
    return {
        type: 'delete' as const,
        filePath: filePath,
        range: util.createRange(
            diagnostic.location.range.start.line,
            diagnostic.location.range.start.character + 'return'.length,
            diagnostic.location.range.end.line,
            diagnostic.location.range.end.character
        )
    };
}

1	import type { XmlFile } from '../../files/XmlFile';
2	import type { BsDiagnostic } from '../../interfaces';
3	import { util } from '../../util';	1✔
4
5	/**
6	* Returns the position at which an `extends` attribute should be inserted for
7	* a component that is missing one: after the last existing attribute, or after
8	* the `<component` tag itself if there are no attributes yet.
9	*/
10	export function getMissingExtendsInsertPosition(file: XmlFile) {	1✔
11	const componentElement = file.parser.ast.componentElement;	2✔
12	const lastAttribute = componentElement.attributes[componentElement.attributes.length - 1];	2✔
13	const range = lastAttribute?.location?.range ?? componentElement.tokens.startTagName?.location?.range;	2!
14	return range?.end;	2!
15	}
16
17	/**
18	* Returns the delete change that removes the return value from a `return <expr>`
19	* statement flagged by `voidFunctionMayNotReturnValue`. Leaves the bare `return`
20	* keyword in place.
21	*/
22	export function getRemoveReturnValueChange(diagnostic: BsDiagnostic, filePath: string) {	1✔
UNCOV 23	return {	×
24	type: 'delete' as const,
25	filePath: filePath,
26	range: util.createRange(
27	diagnostic.location.range.start.line,
28	diagnostic.location.range.start.character + 'return'.length,
29	diagnostic.location.range.end.line,
30	diagnostic.location.range.end.character
31	)
32	};
33	}

rokucommunity / brighterscript / #15903

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