@hero-design/snowflake-guard 1.2.3 → 1.2.4-alpha.1

package/README.md

@@ -10,6 +10,8 @@
 cp .env.example .env
 ```
 
+For React Native projects, include your repo name in the `MOBILE_REPO_NAMES` env.
+
 2. Set up [internal-tool-integrations service](https://github.com/Thinkei/internal-tool-integrations) for the bot to store report.
 
 - Replace `DB_HOST` with the host of the internal-tool-integrations service.

package/lib/src/index.js

@@ -11,12 +11,14 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
+const parseMobileSource_1 = __importDefault(require("./parseMobileSource"));
 const parseSource_1 = __importDefault(require("./parseSource"));
 const constants_1 = require("./reports/constants");
 const fetchGraphql_1 = __importDefault(require("./graphql/fetchGraphql"));
 const queryGenerators_1 = require("./graphql/queryGenerators");
 const getDiffLocs_1 = require("./utils/getDiffLocs");
 const TSX_REGEX = /\.tsx$/;
+const TSX_AND_JSX_REGEX = /\.(tsx|jsx|js)$/;
 const TEST_REGEX = /__tests__/;
 const SNOWFLAKE_COMMENTS = {
     style: 'Snowflake detected! A component is customized using inline styles. Make sure to not use [prohibited CSS properties](https://docs.google.com/spreadsheets/d/1Dj8vqLdFaf-CSaSVoYqyYZIkGqF6OoyP7K4G1_9L62U/edit?usp=sharing).',
@@ -24,6 +26,7 @@ const SNOWFLAKE_COMMENTS = {
     'styled-component': 'Please do not use styled-component to customize this component, use sx prop or inline style instead.',
     className: `Please make sure that this className is not used as a CSS classname for component customization purposes, use sx prop or inline style instead. In case this is none-css classname, please flag it with this comment \`${constants_1.APPROVED_CLASSNAME_COMMENT}\`.`,
 };
+const MOBILE_REPO_NAMES = JSON.parse(process.env.MOBILE_REPO_NAMES || '[]');
 const checkIfDetectedSnowflakesInDiff = (diffLocs, locToComment) => {
     const locIdx = diffLocs.findIndex(([start, end]) => {
         return locToComment >= start && locToComment <= end;
@@ -42,16 +45,19 @@ module.exports = (app) => {
         const prBranch = context.payload.pull_request.head.ref;
         // List all changed files
         const prFiles = yield context.octokit.pulls.listFiles(Object.assign(Object.assign({}, repoInfo), { pull_number: prNumber }));
-        const tsxFiles = prFiles.data.filter((file) => TSX_REGEX.test(file.filename) &&
+        const isMobile = MOBILE_REPO_NAMES.includes(context.payload.repository.name);
+        const parseSource = isMobile ? parseMobileSource_1.default : parseSource_1.default;
+        const SOURCE_REGEX = isMobile ? TSX_AND_JSX_REGEX : TSX_REGEX;
+        const sourceFiles = prFiles.data.filter((file) => SOURCE_REGEX.test(file.filename) &&
             !TEST_REGEX.test(file.filename) &&
             file.status !== 'removed');
         // Saving file patches to get diff locations
-        const prFilePatches = tsxFiles.reduce((acc, file) => {
+        const prFilePatches = sourceFiles.reduce((acc, file) => {
             acc[file.filename] = file.patch || '';
             return acc;
         }, {});
         // Get file contents
-        const prFileContentPromises = tsxFiles.map((file) => context.octokit.repos.getContent(Object.assign(Object.assign({}, repoInfo), { path: file.filename, ref: prBranch })));
+        const prFileContentPromises = sourceFiles.map((file) => context.octokit.repos.getContent(Object.assign(Object.assign({}, repoInfo), { path: file.filename, ref: prBranch })));
         const prFileContents = yield Promise.all(prFileContentPromises);
         const snowflakeComments = [];
         const approvedSnowflakeLocs = [];
@@ -65,7 +71,7 @@ module.exports = (app) => {
             // @ts-ignore
             file.data.content, 'base64').toString();
             // Parse file content to check for snowflakes
-            const snowflakeReport = (0, parseSource_1.default)(stringContent);
+            const snowflakeReport = parseSource(stringContent);
             snowflakeReport.styleLocs.forEach((loc) => {
                 if (checkIfDetectedSnowflakesInDiff(diffLocs, loc)) {
                     snowflakeComments.push({
@@ -75,15 +81,6 @@ module.exports = (app) => {
                     });
                 }
             });
-            snowflakeReport.sxLocs.forEach((loc) => {
-                if (checkIfDetectedSnowflakesInDiff(diffLocs, loc)) {
-                    snowflakeComments.push({
-                        path: filePath,
-                        body: SNOWFLAKE_COMMENTS['sx'],
-                        line: loc,
-                    });
-                }
-            });
             snowflakeReport.styledComponentLocs.forEach((loc) => {
                 if (checkIfDetectedSnowflakesInDiff(diffLocs, loc)) {
                     snowflakeComments.push({
@@ -93,20 +90,31 @@ module.exports = (app) => {
                     });
                 }
             });
-            snowflakeReport.classNameLocs.forEach((loc) => {
-                if (checkIfDetectedSnowflakesInDiff(diffLocs, loc)) {
-                    snowflakeComments.push({
-                        path: filePath,
-                        body: SNOWFLAKE_COMMENTS['className'],
-                        line: loc,
-                    });
-                }
-            });
             snowflakeReport.approvedLocs.forEach((loc) => {
                 if (checkIfDetectedSnowflakesInDiff(diffLocs, loc)) {
                     approvedSnowflakeLocs.push(loc);
                 }
             });
+            if (!isMobile) {
+                snowflakeReport.sxLocs.forEach((loc) => {
+                    if (checkIfDetectedSnowflakesInDiff(diffLocs, loc)) {
+                        snowflakeComments.push({
+                            path: filePath,
+                            body: SNOWFLAKE_COMMENTS['sx'],
+                            line: loc,
+                        });
+                    }
+                });
+                snowflakeReport.classNameLocs.forEach((loc) => {
+                    if (checkIfDetectedSnowflakesInDiff(diffLocs, loc)) {
+                        snowflakeComments.push({
+                            path: filePath,
+                            body: SNOWFLAKE_COMMENTS['className'],
+                            line: loc,
+                        });
+                    }
+                });
+            }
         }));
         // Saving report
         const snowflakeCount = snowflakeComments.length;

package/lib/src/parseMobileSource.d.ts

@@ -0,0 +1,15 @@
+import { InlineStyleProps } from './reports/mobile/reportInlineStyle';
+import type { CompoundMobileComponentName } from './reports/mobile/types';
+declare const parseMobileSource: (source: string) => {
+    styleLocs: number[];
+    styledComponentLocs: number[];
+    approvedLocs: number[];
+    violatingAttributes: {
+        attributeName: string;
+        attributeValue: string | null;
+        inlineStyleProps: InlineStyleProps;
+        componentName: CompoundMobileComponentName;
+        loc: number | undefined;
+    }[];
+};
+export default parseMobileSource;

package/lib/src/parseMobileSource.js

@@ -0,0 +1,118 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    __setModuleDefault(result, mod);
+    return result;
+};
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const recast = __importStar(require("recast"));
+const flowParser = __importStar(require("./parsers/flow"));
+const tsParser = __importStar(require("./parsers/typescript"));
+const constants_1 = require("./reports/mobile/constants");
+const constants_2 = require("./reports/constants");
+const reportCustomStyleProperties_1 = __importDefault(require("./reports/mobile/reportCustomStyleProperties"));
+const reportStyledComponents_1 = __importDefault(require("./reports/mobile/reportStyledComponents"));
+const parseMobileSource = (source) => {
+    let hasHeroDesignImport = false;
+    let hasStyledComponentsImport = false;
+    const componentList = {};
+    let styledAliasName = 'styled';
+    let styledComponentLocs = [];
+    let styleLocs = [];
+    let violatingAttributes = [];
+    const approvedInlineStyleCmts = [];
+    const approvedStyledComponentLocs = [];
+    const ast = recast.parse(source, {
+        parser: source.includes('@flow') ? flowParser : tsParser,
+    });
+    recast.visit(ast, {
+        visitImportDeclaration(path) {
+            this.traverse(path);
+            const importedFrom = path.value.source.value;
+            // Check if file imports components from '@hero-design/rn'
+            if (importedFrom === '@hero-design/rn') {
+                recast.visit(path.node, {
+                    visitImportSpecifier(importPath) {
+                        this.traverse(importPath);
+                        if (constants_1.HD_MOBILE_COMPONENTS.includes(importPath.value.imported.name)) {
+                            componentList[importPath.value.local.name] =
+                                importPath.value.imported.name;
+                            hasHeroDesignImport = true;
+                        }
+                    },
+                });
+            }
+            // Check if file imports from '@emotion/native'
+            if (importedFrom === '@emotion/native') {
+                recast.visit(path.node, {
+                    visitImportDefaultSpecifier(importPath) {
+                        this.traverse(importPath);
+                        styledAliasName = importPath.value.local.name;
+                        hasStyledComponentsImport = true;
+                    },
+                });
+            }
+        },
+        visitComment(path) {
+            this.traverse(path);
+            const comment = path.value.value;
+            if (comment
+                .toLowerCase()
+                .includes(constants_2.APPROVED_INLINE_STYLE_COMMENT.toLowerCase())) {
+                approvedInlineStyleCmts.push({
+                    loc: path.value.loc.start.line,
+                    comment,
+                });
+            }
+            if (comment
+                .toLowerCase()
+                .includes(constants_2.APPROVED_STYLED_COMPONENTS_COMMENT.toLowerCase())) {
+                approvedStyledComponentLocs.push(path.value.loc.start.line);
+            }
+        },
+    });
+    const isNotApprovedStyledComponentSnowflakes = (loc) => !approvedStyledComponentLocs.includes(loc - 1);
+    if (hasHeroDesignImport) {
+        // Case 1: Using style object to customise components
+        const customPropLocs = (0, reportCustomStyleProperties_1.default)(ast, componentList, {
+            styleCmts: approvedInlineStyleCmts,
+        });
+        styleLocs = customPropLocs.style;
+        // Case 2: Using styled-components to customise components
+        if (hasStyledComponentsImport) {
+            styledComponentLocs = (0, reportStyledComponents_1.default)(ast, componentList, styledAliasName).filter(isNotApprovedStyledComponentSnowflakes);
+        }
+        violatingAttributes = customPropLocs.violatingAttributes;
+    }
+    return {
+        styleLocs,
+        styledComponentLocs,
+        approvedLocs: [
+            ...approvedInlineStyleCmts.map((cmt) => cmt.loc),
+            ...approvedStyledComponentLocs,
+        ],
+        violatingAttributes,
+    };
+};
+exports.default = parseMobileSource;

package/lib/src/parsers/flow.d.ts

@@ -0,0 +1,3 @@
+import * as babelParser from '@babel/parser';
+import { Overrides } from 'recast/parsers/_babel_options';
+export declare const parse: (source: string, options?: Overrides) => babelParser.ParseResult<import("@babel/types").File>;

package/lib/src/parsers/flow.js

@@ -0,0 +1,37 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    __setModuleDefault(result, mod);
+    return result;
+};
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.parse = void 0;
+const babelParser = __importStar(require("@babel/parser"));
+const _babel_options_1 = __importDefault(require("recast/parsers/_babel_options"));
+const parse = (source, options) => {
+    const babelOptions = (0, _babel_options_1.default)(options);
+    babelOptions.plugins.push('jsx', 'flow');
+    return babelParser.parse(source, babelOptions);
+};
+exports.parse = parse;

package/lib/src/reports/mobile/__tests__/reportCustomStyleProperties.spec.d.ts

@@ -0,0 +1 @@
+export {};

package/lib/src/reports/mobile/__tests__/reportCustomStyleProperties.spec.js

@@ -0,0 +1,123 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const reportCustomStyleProperties_1 = __importDefault(require("../reportCustomStyleProperties"));
+const testUtils_1 = require("../testUtils");
+const componentList = {
+    Card: 'Card',
+    Button: 'Button',
+};
+const commentList = {
+    styleCmts: [],
+};
+describe('reportCustomStyleProperties', () => {
+    beforeEach(() => {
+        jest.clearAllMocks();
+    });
+    it('should detect component with inline styles', () => {
+        const source = `const styles = {
+        card: {
+          padding: 10,
+        },
+      };
+
+      const App = () => (
+        <Card style={styles.card}>Content</Card>
+      );
+    `;
+        const ast = (0, testUtils_1.parseTypeScript)(source);
+        const result = (0, reportCustomStyleProperties_1.default)(ast, componentList, commentList);
+        expect(result).toEqual({
+            style: [8],
+            violatingAttributes: [
+                {
+                    attributeName: 'padding',
+                    attributeValue: '10',
+                    componentName: 'Card',
+                    inlineStyleProps: 'style',
+                    loc: 8,
+                },
+            ],
+        });
+    });
+    it('should detect compound component with inline styles', () => {
+        const source = `const App = () => (
+        <Button.Icon style={{ padding: 10 }} />
+      );
+    `;
+        const ast = (0, testUtils_1.parseTypeScript)(source);
+        const result = (0, reportCustomStyleProperties_1.default)(ast, componentList, commentList);
+        expect(result).toEqual({
+            style: [2],
+            violatingAttributes: [
+                {
+                    attributeName: 'padding',
+                    attributeValue: '10',
+                    componentName: 'Button.Icon',
+                    inlineStyleProps: 'style',
+                    loc: 2,
+                },
+            ],
+        });
+    });
+    it('should detect compound component using spread operator with inline styles', () => {
+        const source = `const { Icon } = Button;
+      const App = () => (
+        <Icon style={{ padding: 10 }} />
+      );
+    `;
+        const ast = (0, testUtils_1.parseTypeScript)(source);
+        const result = (0, reportCustomStyleProperties_1.default)(ast, componentList, commentList);
+        expect(result).toEqual({
+            style: [3],
+            violatingAttributes: [
+                {
+                    attributeName: 'padding',
+                    attributeValue: '10',
+                    componentName: 'Button.Icon',
+                    inlineStyleProps: 'style',
+                    loc: 3,
+                },
+            ],
+        });
+    });
+    it('should not detect non-custom style properties', () => {
+        const source = `
+      const App = () => (
+        <Card>Content</Card>
+      );
+    `;
+        const ast = (0, testUtils_1.parseTypeScript)(source);
+        const result = (0, reportCustomStyleProperties_1.default)(ast, componentList, commentList);
+        expect(result).toEqual({ style: [], violatingAttributes: [] });
+    });
+    it('should ignore approved inline styles', () => {
+        const mockedCommentList = {
+            styleCmts: [
+                {
+                    loc: 10,
+                    comment: '@snowflake-guard/approved-inline-style attributes: padding',
+                },
+            ],
+        };
+        const source = `
+      const styles = {
+        button: {
+          padding: 10,
+        },
+      };
+
+      const App = () => (
+      <> 
+        {/* @snowflake-guard/approved-inline-style attributes: padding */}
+        <Button style={styles.button} />
+      </>
+      );
+    `;
+        const ast = (0, testUtils_1.parseTypeScript)(source);
+        const result = (0, reportCustomStyleProperties_1.default)(ast, componentList, mockedCommentList);
+        expect(result).toEqual({ style: [], violatingAttributes: [] });
+    });
+});

package/lib/src/reports/mobile/__tests__/reportInlineStyle.spec.d.ts

@@ -0,0 +1 @@
+export {};

package/lib/src/reports/mobile/__tests__/reportInlineStyle.spec.js

@@ -0,0 +1,103 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    __setModuleDefault(result, mod);
+    return result;
+};
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const recast = __importStar(require("recast"));
+const reportInlineStyle_1 = __importDefault(require("../reportInlineStyle"));
+const constants_1 = require("../constants");
+const tsParser = __importStar(require("../../../parsers/typescript"));
+describe('reportInlineStyle', () => {
+    it('should return line numbers and violating attributes for inline styles that violate the ruleset', () => {
+        const code = `
+    const MyComponent = () => (
+      <Card style={{
+              color: 'red',
+              backgroundColor: 'blue'
+            }}
+/>
+    );
+  `;
+        // Step 1: Parse the code to generate the AST
+        const ast = recast.parse(code, { parser: tsParser });
+        const attributes = [];
+        // Step 2: Traverse the AST to extract relevant JSXAttributes (style)
+        recast.visit(ast, {
+            visitJSXAttribute(path) {
+                const { node } = path;
+                attributes.push(node);
+                this.traverse(path);
+            },
+        });
+        const componentName = 'Card';
+        // Mock the RULESET_MAP for this test
+        constants_1.MOBILE_RULESET_MAP.Card = ['color', 'backgroundColor'];
+        // Step 3: Run the function with the dynamically extracted attributes
+        const result = (0, reportInlineStyle_1.default)(ast, attributes, componentName);
+        expect(result.locs.style).toEqual(3);
+        expect(result.violatingAttributes).toEqual([
+            {
+                attributeName: 'color',
+                attributeValue: "'red'",
+                componentName: 'Card',
+                inlineStyleProps: 'style',
+                loc: 3,
+            },
+            {
+                attributeName: 'backgroundColor',
+                attributeValue: "'blue'",
+                componentName: 'Card',
+                inlineStyleProps: 'style',
+                loc: 3,
+            },
+        ]);
+    });
+    it('should return empty arrays when no violations are found', () => {
+        const code = `
+      const MyComponent = () => (
+        <Card style={{ margin: 10 }} />
+      );
+    `;
+        // Step 1: Parse the code to generate the AST
+        const ast = recast.parse(code, { parser: tsParser });
+        const attributes = [];
+        // Step 2: Traverse the AST to extract relevant JSXAttributes (style)
+        recast.visit(ast, {
+            visitJSXAttribute(path) {
+                const { node } = path;
+                attributes.push(node);
+                this.traverse(path);
+            },
+        });
+        const componentName = 'Card';
+        // Mock the RULESET_MAP and SX_RULESET_MAP for this test
+        constants_1.MOBILE_RULESET_MAP.Card = ['color'];
+        const result = (0, reportInlineStyle_1.default)(ast, attributes, componentName);
+        expect(result.locs.style).toEqual(undefined);
+        expect(result.violatingAttributes).toEqual([]);
+        expect(result.violatingAttributes).toEqual([]);
+    });
+});

package/lib/src/reports/mobile/__tests__/reportStyledComponents.spec.d.ts

@@ -0,0 +1 @@
+export {};

package/lib/src/reports/mobile/__tests__/reportStyledComponents.spec.js

@@ -0,0 +1,54 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const reportStyledComponents_1 = __importDefault(require("../reportStyledComponents"));
+const testUtils_1 = require("../testUtils");
+const componentList = {
+    Button: 'Button',
+    Alert: 'Alert',
+};
+const styledAliasName = 'styled';
+describe('reportStyledComponents', () => {
+    it('should detect styled components with default component', () => {
+        const source = `
+      const StyledCard = styled(Button)\`
+        padding: 10px;
+      \`;
+    `;
+        const ast = (0, testUtils_1.parseTypeScript)(source);
+        const locs = (0, reportStyledComponents_1.default)(ast, componentList, styledAliasName);
+        expect(locs).toEqual([2]);
+    });
+    it('should detect styled components with compound component', () => {
+        const source = `
+      const StyledCardIcon = styled(Button.Icon)\`
+        padding: 10px;
+      \`;
+    `;
+        const ast = (0, testUtils_1.parseTypeScript)(source);
+        const locs = (0, reportStyledComponents_1.default)(ast, componentList, styledAliasName);
+        expect(locs).toEqual([2]);
+    });
+    it('should detect styled components with spread operator', () => {
+        const source = `
+      const { Icon } = Button;
+      const StyledIcon = styled(Icon)\`
+        color: red;
+      \`;
+      <StyledIcon />;
+    `;
+        const ast = (0, testUtils_1.parseTypeScript)(source);
+        const locs = (0, reportStyledComponents_1.default)(ast, componentList, styledAliasName);
+        expect(locs).toEqual([3]);
+    });
+    it('should not detect non-styled components', () => {
+        const source = `
+      const NotStyled = Button;
+    `;
+        const ast = (0, testUtils_1.parseTypeScript)(source);
+        const locs = (0, reportStyledComponents_1.default)(ast, componentList, styledAliasName);
+        expect(locs).toEqual([]);
+    });
+});