13906243876

Committed 17 Mar 2025 05:43PM UTC coverage: 94.719%. Remained the same

Build # 13906243876

Build Type

Pull #391

github

Committed by

web-flow

Commit Message

Merge dd2a4f8ee into b8693f39a

Pull Request Pull Request #391: Bump actions/setup-node from 4.2.0 to 4.3.0

Run Details

501 of 560 branches covered (89.46%)

Branch coverage included in aggregate %.

916 of 936 relevant lines covered (97.86%)

206.89 hits per line

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

97.22

/lib/plugin/variable-collector.js

"use strict";
const posixPath = require("path").posix;
const less = require("../thirdparty/less");
const backslashRegExp = /\\/g;

function toPosix(p) {
        return p.replace(backslashRegExp, "/");
}

const VariableCollector = module.exports = function(env) {
        /* eslint-disable new-cap */
        this.oVisitor = new less.tree.visitor(this);
        /* eslint-enable new-cap */
        this.env = env;
        this.mVariables = {};
        this.mGlobalVariables = {};
        this.mVarFile = {};
};

VariableCollector.prototype = {
        isPreVisitor: true,
        run: function(root) {
                this.mGlobalVariables = root.variables();
                return this.oVisitor.visit(root);
        },

        visitRule: function(node) {
                if (!node.variable) {
                        return;
                }
                if (!this.mGlobalVariables[node.name]) {
                        // Ignoring local variable
                        return;
                }
                try {
                        const value = node.value.toCSS(this.env);
                        this.mVariables[node.name.substr(1)] = {
                                value: value,
                                filename: node.currentFileInfo.filename,
                                rootFilename: node.currentFileInfo.rootFilename
                        };
                // eslint-disable-next-line no-unused-vars
                } catch (err) {
                        // Errors might occur within mixins.
                        // But as we only collect global variables, this doesn't matter...
                }
        },

        getVariables: function(aImports) {
                const mVariables = {};

                const aPosixImports = aImports.map(toPosix);

                const includeCache = {};

                function shouldIncludeVariable({rootFilename, filename}) {
                        // Simple case where variable is defined in root file
                        if (rootFilename === filename) {
                                return true;
                        }

                        const cacheKey = rootFilename + "|" + filename;
                        if (includeCache[cacheKey] !== undefined) {
                                return includeCache[cacheKey];
                        }

                        const dirname = posixPath.dirname(filename);
                        const baseFileName = posixPath.basename(rootFilename); // usually library.source.less

                        function check(currentDirname) {
                                const libraryBaseFile = posixPath.join(currentDirname, baseFileName);
                                if (libraryBaseFile === rootFilename || aPosixImports.includes(libraryBaseFile)) {
                                        return true;
                                } else {
                                        // Recursively check parent directories whether they contain an imported "base file"
                                        // This is only relevant when a theme contains sub-directories
                                        const parentDirname = posixPath.dirname(currentDirname);
                                        if (parentDirname === currentDirname) {
                                                return false; // We are at the root
                                        } else {
                                                return check(parentDirname);
                                        }
                                }
                        }

                        return includeCache[cacheKey] = check(dirname);
                }

                for (const name in this.mVariables) {
                        if (Object.prototype.hasOwnProperty.call(this.mVariables, name)) {
                                const oVar = this.mVariables[name];
                                // Ensure posix paths
                                const rootFilename = toPosix(oVar.rootFilename);
                                const filename = toPosix(oVar.filename);

                                // Only add variable if the corresponding library "base file" has been imported
                                if (shouldIncludeVariable({rootFilename, filename})) {
                                        mVariables[name] = oVar.value;
                                }
                        }
                }

                return mVariables;
        },

        getAllVariables: function(aImports) {
                const mVariables = {};
                for (const name in this.mVariables) {
                        if (Object.prototype.hasOwnProperty.call(this.mVariables, name)) {
                                const oVar = this.mVariables[name];
                                mVariables[name] = oVar.value;
                        }
                }
                return mVariables;
        }

};

1	"use strict";
2	const posixPath = require("path").posix;	1✔
3	const less = require("../thirdparty/less");	1✔
4	const backslashRegExp = /\\/g;	1✔
5
6	function toPosix(p) {
7	return p.replace(backslashRegExp, "/");	639✔
8	}
9
10	const VariableCollector = module.exports = function(env) {	1✔
11	/* eslint-disable new-cap */
12	this.oVisitor = new less.tree.visitor(this);	198✔
13	/* eslint-enable new-cap */
14	this.env = env;	198✔
15	this.mVariables = {};	198✔
16	this.mGlobalVariables = {};	198✔
17	this.mVarFile = {};	198✔
18	};
19
20	VariableCollector.prototype = {	1✔
21	isPreVisitor: true,
22	run: function(root) {
23	this.mGlobalVariables = root.variables();	190✔
24	return this.oVisitor.visit(root);	190✔
25	},
26
27	visitRule: function(node) {
28	if (!node.variable) {	2,173✔
29	return;	1,891✔
30	}
31	if (!this.mGlobalVariables[node.name]) {	282✔
32	// Ignoring local variable
33	return;	16✔
34	}
35	try {	266✔
36	const value = node.value.toCSS(this.env);	266✔
37	this.mVariables[node.name.substr(1)] = {	265✔
38	value: value,
39	filename: node.currentFileInfo.filename,
40	rootFilename: node.currentFileInfo.rootFilename
41	};
42	// eslint-disable-next-line no-unused-vars
43	} catch (err) {
44	// Errors might occur within mixins.
45	// But as we only collect global variables, this doesn't matter...
46	}
47	},
48
49	getVariables: function(aImports) {
50	const mVariables = {};	188✔
51
52	const aPosixImports = aImports.map(toPosix);	188✔
53
54	const includeCache = {};	188✔
55
56	function shouldIncludeVariable({rootFilename, filename}) {
57	// Simple case where variable is defined in root file
58	if (rootFilename === filename) {	167✔
59	return true;	55✔
60	}
61
62	const cacheKey = rootFilename + "\|" + filename;	112✔
63	if (includeCache[cacheKey] !== undefined) {	112✔
64	return includeCache[cacheKey];	8✔
65	}
66
67	const dirname = posixPath.dirname(filename);	104✔
68	const baseFileName = posixPath.basename(rootFilename); // usually library.source.less	104✔
69
70	function check(currentDirname) {
71	const libraryBaseFile = posixPath.join(currentDirname, baseFileName);	166✔
72	if (libraryBaseFile === rootFilename \|\| aPosixImports.includes(libraryBaseFile)) {	166✔
73	return true;	93✔
74	} else {
75	// Recursively check parent directories whether they contain an imported "base file"
76	// This is only relevant when a theme contains sub-directories
77	const parentDirname = posixPath.dirname(currentDirname);	73✔
78	if (parentDirname === currentDirname) {	73✔
79	return false; // We are at the root	11✔
80	} else {
81	return check(parentDirname);	62✔
82	}
83	}
84	}
85
86	return includeCache[cacheKey] = check(dirname);	104✔
87	}
88
89	for (const name in this.mVariables) {	188✔
90	if (Object.prototype.hasOwnProperty.call(this.mVariables, name)) {	167!
91	const oVar = this.mVariables[name];	167✔
92	// Ensure posix paths
93	const rootFilename = toPosix(oVar.rootFilename);	167✔
94	const filename = toPosix(oVar.filename);	167✔
95
96	// Only add variable if the corresponding library "base file" has been imported
97	if (shouldIncludeVariable({rootFilename, filename})) {	167✔
98	mVariables[name] = oVar.value;	155✔
99	}
100	}
101	}
102
103	return mVariables;	188✔
104	},
105
106	getAllVariables: function(aImports) {
107	const mVariables = {};	190✔
108	for (const name in this.mVariables) {	190✔
109	if (Object.prototype.hasOwnProperty.call(this.mVariables, name)) {	176!
110	const oVar = this.mVariables[name];	176✔
111	mVariables[name] = oVar.value;	176✔
112	}
113	}
114	return mVariables;	190✔
115	}
116
117	};

SAP / less-openui5 / 13906243876

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