11742433056

Committed 08 Nov 2024 12:50PM UTC coverage: 80.521% (+0.6%) from 79.898%

Build # 11742433056

Build Type

Pull #327

github

Committed by

web-flow

Commit Message

Merge d5289b43f into d2b97ac3d

Pull Request Pull Request #327: feat: Add ui5lint-disable directives

Run Details

1785 of 2343 branches covered (76.18%)

Branch coverage included in aggregate %.

113 of 117 new or added lines in 7 files covered. (96.58%)

34 existing lines in 3 files now uncovered.

2601 of 3104 relevant lines covered (83.8%)

1704.2 hits per line

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

1.01

/src/formatter/text.ts

import chalk from "chalk";
import path from "node:path";
import {LintResult, LintMessage} from "../linter/LinterContext.js";
import {LintMessageSeverity} from "../linter/messages.js";

const detailsHeader = chalk.white.bold("Details:");

function formatSeverity(severity: LintMessageSeverity) {
        if (severity === LintMessageSeverity.Error) {
                return chalk.red("error");
        } else if (severity === LintMessageSeverity.Warning) {
                return chalk.yellow("warning");
        } else {
                throw new Error(`Unknown severity: ${LintMessageSeverity[severity]}`);
        }
}

function formatLocation(line: LintMessage["line"], column: LintMessage["column"],
        lineInfoLength: number, columnInfoLength: number) {
        const lineStr = (line === undefined ? "0" : line.toString()).padStart(lineInfoLength, " ");
        const columnStr = (column === undefined ? "0" : column.toString()).padEnd(columnInfoLength, " ");

        return chalk.dim(`${lineStr}:${columnStr}`);
}

function formatMessageDetails(msg: LintMessage, showDetails: boolean) {
        if (!showDetails || !msg.messageDetails) {
                return "";
        }
        // Ensure that details are not containing line breaks as every message should be just a single line.
        // In addition, some integrations understand two whitespace chars (e.g. two spaces) as a separator
        // for a message code (e.g. $eslint-stylish problems matcher in VS Code).
        // Therefore, two or more whitespace chars are reduced to a single space.
        return `. ${detailsHeader} ${chalk.italic(msg.messageDetails.replace(/\s\s+|\n/g, " "))}`;
}

export class Text {
        #buffer = "";

        format(lintResults: LintResult[], showDetails: boolean) {
                this.#writeln(`UI5 linter report:`);
                this.#writeln("");
                let totalErrorCount = 0;
                let totalWarningCount = 0;
                let totalFatalErrorCount = 0;
                lintResults.forEach(({filePath, messages, errorCount, warningCount, fatalErrorCount}) => {
                        if (!errorCount && !warningCount) {
                                return;
                        }
                        totalErrorCount += errorCount;
                        totalWarningCount += warningCount;
                        totalFatalErrorCount += fatalErrorCount;

                        this.#writeln(chalk.inverse(path.resolve(process.cwd(), filePath)));

                        // Determine maximum line and column for position formatting
                        let maxLine = 0;
                        let maxColumn = 0;
                        messages.forEach((msg) => {
                                if (msg.line && msg.line > maxLine) {
                                        maxLine = msg.line;
                                }
                                if (msg.column && msg.column > maxColumn) {
                                        maxColumn = msg.column;
                                }
                        });

                        const lineInfoLength = maxLine.toString().length;
                        const columnInfoLength = maxColumn.toString().length;

                        // Sort by line, then by column. Use 0 if not set.
                        messages.sort((a, b) => (a.line ?? 0) - (b.line ?? 0) || (a.column ?? 0) - (b.column ?? 0));

                        messages.forEach((msg) => {
                                const formattedLocation =
                                        formatLocation(msg.line, msg.column, lineInfoLength, columnInfoLength);

                                this.#writeln(
                                        `  ${formattedLocation} ` +
                                        `${formatSeverity(msg.severity)} ` +
                                        `${msg.fatal ? "Fatal error: " : ""}` +
                                        `${msg.message}` +
                                        `${formatMessageDetails(msg, showDetails)}`);
                        });

                        this.#writeln("");
                });

                let summaryColor = chalk.green;
                if (totalErrorCount > 0) {
                        summaryColor = chalk.red;
                } else if (totalWarningCount > 0) {
                        summaryColor = chalk.yellow;
                }

                this.#writeln(
                        summaryColor(
                                `${totalErrorCount + totalWarningCount} problems ` +
                                `(${totalErrorCount} errors, ${totalWarningCount} warnings)`
                        )
                );
                if (totalFatalErrorCount) {
                        this.#writeln(summaryColor(`${totalFatalErrorCount} fatal errors`));
                }

                if (!showDetails && (totalErrorCount + totalWarningCount + totalFatalErrorCount) > 0) {
                        this.#writeln("");
                        this.#writeln(chalk.dim.bold("Note: ") +
                                chalk.dim(`Use "ui5lint --details" to show more information about the findings`));
                }

                return this.#buffer;
        }

        // eslint-disable-next-line no-unused-private-class-members
        #write(str: string) {
                this.#buffer += str;
        }

        #writeln(str: string) {
                this.#buffer += str + "\n";
        }
}

1	import chalk from "chalk";
2	import path from "node:path";
3	import {LintResult, LintMessage} from "../linter/LinterContext.js";
4	import {LintMessageSeverity} from "../linter/messages.js";
5
6	const detailsHeader = chalk.white.bold("Details:");	3✔
7
8	function formatSeverity(severity: LintMessageSeverity) {
9	if (severity === LintMessageSeverity.Error) {	×
10	return chalk.red("error");	×
11	} else if (severity === LintMessageSeverity.Warning) {	×
12	return chalk.yellow("warning");	×
13	} else {
14	throw new Error(`Unknown severity: ${LintMessageSeverity[severity]}`);	×
15	}
16	}
17
18	function formatLocation(line: LintMessage["line"], column: LintMessage["column"],
19	lineInfoLength: number, columnInfoLength: number) {
20	const lineStr = (line === undefined ? "0" : line.toString()).padStart(lineInfoLength, " ");	×
21	const columnStr = (column === undefined ? "0" : column.toString()).padEnd(columnInfoLength, " ");	×
22
23	return chalk.dim(`${lineStr}:${columnStr}`);	×
24	}
25
26	function formatMessageDetails(msg: LintMessage, showDetails: boolean) {
27	if (!showDetails \|\| !msg.messageDetails) {	×
28	return "";	×
29	}
30	// Ensure that details are not containing line breaks as every message should be just a single line.
31	// In addition, some integrations understand two whitespace chars (e.g. two spaces) as a separator
32	// for a message code (e.g. $eslint-stylish problems matcher in VS Code).
33	// Therefore, two or more whitespace chars are reduced to a single space.
34	return `. ${detailsHeader} ${chalk.italic(msg.messageDetails.replace(/\s\s+\|\n/g, " "))}`;	×
35	}
36
37	export class Text {
38	#buffer = "";	×
39
40	format(lintResults: LintResult[], showDetails: boolean) {
41	this.#writeln(`UI5 linter report:`);	×
42	this.#writeln("");	×
UNCOV 43	let totalErrorCount = 0;	×
44	let totalWarningCount = 0;	×
45	let totalFatalErrorCount = 0;	×
46	lintResults.forEach(({filePath, messages, errorCount, warningCount, fatalErrorCount}) => {	×
47	if (!errorCount && !warningCount) {	×
48	return;	×
49	}
50	totalErrorCount += errorCount;	×
51	totalWarningCount += warningCount;	×
52	totalFatalErrorCount += fatalErrorCount;	×
53
54	this.#writeln(chalk.inverse(path.resolve(process.cwd(), filePath)));	×
55
56	// Determine maximum line and column for position formatting
57	let maxLine = 0;	×
58	let maxColumn = 0;	×
59	messages.forEach((msg) => {	×
UNCOV 60	if (msg.line && msg.line > maxLine) {	×
61	maxLine = msg.line;	×
62	}
UNCOV 63	if (msg.column && msg.column > maxColumn) {	×
64	maxColumn = msg.column;	×
65	}
66	});
67
68	const lineInfoLength = maxLine.toString().length;	×
UNCOV 69	const columnInfoLength = maxColumn.toString().length;	×
70
71	// Sort by line, then by column. Use 0 if not set.
UNCOV 72	messages.sort((a, b) => (a.line ?? 0) - (b.line ?? 0) \|\| (a.column ?? 0) - (b.column ?? 0));	×
73
UNCOV 74	messages.forEach((msg) => {	×
75	const formattedLocation =
76	formatLocation(msg.line, msg.column, lineInfoLength, columnInfoLength);	×
77
UNCOV 78	this.#writeln(	×
79	` ${formattedLocation} ` +
80	`${formatSeverity(msg.severity)} ` +
81	`${msg.fatal ? "Fatal error: " : ""}` +	×
82	`${msg.message}` +
83	`${formatMessageDetails(msg, showDetails)}`);
84	});
85
UNCOV 86	this.#writeln("");	×
87	});
88
UNCOV 89	let summaryColor = chalk.green;	×
90	if (totalErrorCount > 0) {	×
91	summaryColor = chalk.red;	×
UNCOV 92	} else if (totalWarningCount > 0) {	×
UNCOV 93	summaryColor = chalk.yellow;	×
94	}
95
UNCOV 96	this.#writeln(	×
97	summaryColor(
98	`${totalErrorCount + totalWarningCount} problems ` +
99	`(${totalErrorCount} errors, ${totalWarningCount} warnings)`
100	)
101	);
UNCOV 102	if (totalFatalErrorCount) {	×
UNCOV 103	this.#writeln(summaryColor(`${totalFatalErrorCount} fatal errors`));	×
104	}
105
UNCOV 106	if (!showDetails && (totalErrorCount + totalWarningCount + totalFatalErrorCount) > 0) {	×
UNCOV 107	this.#writeln("");	×
UNCOV 108	this.#writeln(chalk.dim.bold("Note: ") +	×
109	chalk.dim(`Use "ui5lint --details" to show more information about the findings`));
110	}
111
UNCOV 112	return this.#buffer;	×
113	}
114
115	// eslint-disable-next-line no-unused-private-class-members
116	#write(str: string) {
UNCOV 117	this.#buffer += str;	×
118	}
119
120	#writeln(str: string) {
UNCOV 121	this.#buffer += str + "\n";	×
122	}
123	}

SAP / ui5-linter / 11742433056

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