chrome-devtools-frontend 1.0.1473514 → 1.0.1510180

package/scripts/eslint_rules/tests/no-importing-images-from-src.test.ts

@@ -1,40 +0,0 @@
-// Copyright 2022 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-import rule from '../lib/no-importing-images-from-src.ts';
-
-import {RuleTester} from './utils/RuleTester.ts';
-
-new RuleTester().run('no-importing-images-from-src', rule, {
-  valid: [
-    {
-      code: 'const someIcon = new URL(\'../../../Images/test_icon.svg\', import.meta.url).toString()',
-      filename: 'front_end/ui/components/component/file.ts',
-    },
-  ],
-
-  invalid: [
-    {
-      code: 'const someIcon = new URL(\'../../../Images/src/test_icon.svg\', import.meta.url).toString()',
-      filename: 'front_end/ui/components/component/file.ts',
-      output: 'const someIcon = new URL(\'../../../Images/test_icon.svg\', import.meta.url).toString()',
-      errors: [
-        {
-          messageId: 'imageImportUsingSrc',
-        },
-      ],
-    },
-    {
-      code:
-          'const someIcon = new URL(\'../../../devtools-frontend/front_end/Images/src/test_icon.svg\', import.meta.url).toString()',
-      filename: 'front_end/ui/components/component/file.ts',
-      output:
-          'const someIcon = new URL(\'../../../devtools-frontend/front_end/Images/test_icon.svg\', import.meta.url).toString()',
-      errors: [
-        {
-          messageId: 'imageImportUsingSrc',
-        },
-      ],
-    },
-  ],
-});

package/scripts/eslint_rules/tests/no-imports-in-directory.test.ts

@@ -1,89 +0,0 @@
-// Copyright 2022 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-import path from 'path';
-
-import rule from '../lib/no-imports-in-directory.ts';
-
-import {RuleTester} from './utils/RuleTester.ts';
-
-const NOT_SDK_PATH = path.join(
-    // @ts-expect-error
-    import.meta.dirname,
-    '..',
-    '..',
-    '..',
-    'front_end',
-    'core',
-    'not-sdk',
-    'sdk.js',
-);
-
-const SDK_PATH = path.join(
-    // @ts-expect-error
-    import.meta.dirname,
-    '..',
-    '..',
-    '..',
-    'front_end',
-    'core',
-    'sdk',
-    'sdk.js',
-);
-
-new RuleTester().run('no-imports-in-directory', rule, {
-  valid: [
-    {
-      code: 'import * as SDK from \'../../../core/sdk/sdk.js\';',
-      filename: 'front_end/models/trace/handlers/TestHandler.ts',
-      options: [
-        {
-          bannedImportPaths: [NOT_SDK_PATH],
-        },
-      ],
-    },
-  ],
-  invalid: [
-    {
-      code: 'import * as SDK from \'../../../core/sdk/sdk.js\';',
-      filename: 'front_end/models/trace/handlers/TestHandler.ts',
-      options: [
-        {
-          bannedImportPaths: [SDK_PATH],
-        },
-      ],
-      errors: [{messageId: 'invalidImport'}],
-    },
-    {
-      code: 'import type * as SDK from \'../../../core/sdk/sdk.js\';',
-      filename: 'front_end/models/trace/handlers/TestHandler.ts',
-      options: [
-        {
-          bannedImportPaths: [SDK_PATH],
-        },
-      ],
-      errors: [{messageId: 'invalidImport'}],
-    },
-    {
-      code: 'import \'../../../core/sdk/sdk.js\';',
-      filename: 'front_end/models/trace/handlers/TestHandler.ts',
-      options: [
-        {
-          bannedImportPaths: [SDK_PATH],
-        },
-      ],
-      errors: [{messageId: 'invalidImport'}],
-    },
-    {
-      code: 'import {Foo} from \'../../../core/sdk/sdk.js\';',
-      filename: 'front_end/models/trace/handlers/TestHandler.ts',
-      options: [
-        {
-          bannedImportPaths: [SDK_PATH],
-        },
-      ],
-      errors: [{messageId: 'invalidImport'}],
-    },
-  ],
-});

package/scripts/eslint_rules/tests/no-it-screenshot-only-or-repeat.test.ts

@@ -1,34 +0,0 @@
-// Copyright 2023 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-import rule from '../lib/no-it-screenshot-only-or-repeat.ts';
-
-import {RuleTester} from './utils/RuleTester.ts';
-
-new RuleTester().run('no-it-screenshot-only-or-repeat', rule, {
-  valid: [
-    {
-      code: `itScreenshot('does a thing', () => {
-        assertElementScreenshotUnchanged(element, 'foo.png');
-      })`,
-      filename: 'front_end/components/test.ts',
-    },
-  ],
-  invalid: [
-    {
-      code: `itScreenshot.only('does a thing', () => {
-        assertElementScreenshotUnchanged(element, 'foo.png');
-      })`,
-      filename: 'front_end/components/test.ts',
-      errors: [{messageId: 'itScreenshot-only', column: 14, endColumn: 18}],
-    },
-    {
-      code: `itScreenshot.repeat(20, 'does a thing', () => {
-        assertElementScreenshotUnchanged(element, 'foo.png');
-      })`,
-      filename: 'front_end/components/test.ts',
-      errors: [{messageId: 'itScreenshot-repeat', column: 14, endColumn: 20}],
-    },
-  ],
-});

package/scripts/eslint_rules/tests/no-lit-render-outside-of-view.test.ts

@@ -1,80 +0,0 @@
-// Copyright 2025 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-import rule from '../lib/no-lit-render-outside-of-view.ts';
-
-import {RuleTester} from './utils/RuleTester.ts';
-
-new RuleTester().run('no-lit-render-outside-of-view', rule, {
-  valid: [
-    {
-      code: `const DEFAULT_VIEW = (input, output, target) => {
-        render(html\`<div>Hello world</div>\`, target, {host: input});
-      }`,
-      filename: 'front_end/components/test.ts',
-    },
-    {
-      code: `function defaultView(viewInput, viewOutput, target) {
-        render(html\`<div>Hello world</div>\`, target, {host: viewInput});
-      }`,
-      filename: 'front_end/panels/recorder/test.ts',
-    },
-    {
-      code: `class SomeWidget extends UI.Widget.Widget {
-          constructor(view = (input, _output, target) => {
-            render(html\`<div>Hello world</div>\`, target, {host: input});
-          }) {
-          super(view);
-        }
-      }`,
-      filename: 'front_end/panels/recorder/test.ts',
-    },
-  ],
-  invalid: [
-    {
-      code: `class SomeWidget extends UI.Widget.Widget {
-        constructor() {
-          super();
-          render(html\`<div>Hello world</div>\`, target, {host: this});
-        }
-      }`,
-      filename: 'front_end/components/test.ts',
-      errors: [{messageId: 'litRenderShouldBeInsideOfView'}],
-    },
-    {
-      code: `class SomeWidget extends UI.Widget.Widget {
-        constructor() {
-          super();
-          this.render();
-        }
-
-        render() {
-          render(html\`<div>Hello world</div>\`, target, {host: this});
-        }
-      }`,
-      filename: 'front_end/components/test.ts',
-      errors: [{messageId: 'litRenderShouldBeInsideOfView'}],
-    },
-    {
-      code: `const DEFAULT_VIEW = (input, output, target) => {
-        render(html\`<div>Hello world</div>\`, target);
-      }`,
-      filename: 'front_end/components/test.ts',
-      errors: [{messageId: 'litRenderShouldBeInsideOfView'}],
-    },
-    {
-      code: `const DEFAULT_VIEW = (input, output, target) => {
-        render(html\`<div>Hello world</div>\`, target, {host: this});
-      }`,
-      filename: 'front_end/components/test.ts',
-      errors: [{messageId: 'litRenderShouldBeInsideOfView'}],
-    },
-    {
-      code: `const DEFAULT_VIEW = (input, output, target) => {
-        render(html\`<div>Hello world</div>\`, this.contentElement, {host: input});
-      }`,
-      filename: 'front_end/components/test.ts',
-      errors: [{messageId: 'litRenderShouldBeInsideOfView'}],
-    },
-  ],
-});

package/scripts/eslint_rules/tests/no-new-lit-element-components.test.ts

@@ -1,27 +0,0 @@
-// Copyright 2023 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-import rule from '../lib/no-new-lit-element-components.ts';
-
-import {RuleTester} from './utils/RuleTester.ts';
-
-new RuleTester().run('no-new-lit-element-components', rule, {
-  valid: [
-    {
-      code: 'class A extends B {}',
-      filename: 'front_end/components/test.ts',
-    },
-    {
-      code: 'class A extends LitElement {}',
-      filename: 'front_end/panels/recorder/test.ts',
-    },
-  ],
-  invalid: [
-    {
-      code: 'class A extends LitElement {}',
-      filename: 'front_end/components/test.ts',
-      errors: [{messageId: 'noNewLitElementComponents'}],
-    },
-  ],
-});

package/scripts/eslint_rules/tests/no-screenshot-test-outside-perf-panel.test.ts

@@ -1,99 +0,0 @@
-// Copyright 2023 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-import rule from '../lib/no-screenshot-test-outside-perf-panel.ts';
-
-import {RuleTester} from './utils/RuleTester.ts';
-
-const enabledTestCode = `describe('Performance panel', () => {
-    itScreenshot('renders the timeline correctly', async () => {
-      await loadComponentDocExample('performance_panel/basic.html?trace=animation');
-      await waitFor('#timeline-overview-panel');
-      const panel = await waitFor('body');
-      await assertElementScreenshotUnchanged(panel, 'performance/timeline.png');
-    });
-  });`;
-
-const disabledTestCode = `describe('Performance panel', () => {
-      // Disabled until screenshot tests infrastructure is available in DevTools bots.
-      itScreenshot.skip('[crbug.com/1407638] renders the timeline correctly', async () => {
-      await waitFor('#timeline-overview-panel');
-      const panel = await waitFor('body');
-      await assertElementScreenshotUnchanged(panel, 'performance/timeline.png');
-    });
-  });`;
-
-const notAScreenshotTestCode = `describe('Performance panel', () => {
-    it('renders the timeline correctly', async () => {
-    await waitFor('#timeline-overview-panel');
-    const panel = await waitFor('body');
-    await assertElementScreenshotUnchanged(panel, 'performance/timeline.png');
-  });
-  });`;
-
-const notAScreenshotTestDisabledCode = `describe('Performance panel', () => {
-    // This is disabled.
-    it.skip('[crbug.com/1407638] renders the timeline correctly', async () => {
-    await waitFor('#timeline-overview-panel');
-    const panel = await waitFor('body');
-    await assertElementScreenshotUnchanged(panel, 'performance/timeline.png');
-  });
-});`;
-
-const perfPanelInteractionTestsPath = 'test/interactions/panels/performance';
-const uiInteractionTestsPath = 'test/interactions/ui/components';
-const notPerfPanelTestPath = 'test/interactions/data_grid/data_grid_test.ts';
-
-new RuleTester().run('no-screenshot-test-outside-perf-panel', rule, {
-  valid: [
-    {
-      code: enabledTestCode,
-      filename: `${perfPanelInteractionTestsPath}/timeline/timeline_test.ts`,
-    },
-    {
-      code: enabledTestCode,
-      filename: `${uiInteractionTestsPath}/Dialog_test.ts`,
-    },
-    {
-      code: disabledTestCode,
-      filename: `${perfPanelInteractionTestsPath}/timeline/timeline_test.ts`,
-    },
-    {
-      code: enabledTestCode,
-      filename: `${perfPanelInteractionTestsPath}/user_timings/user_imings_test.ts`,
-    },
-    {
-      code: disabledTestCode,
-      filename: `${perfPanelInteractionTestsPath}/user_timings/user_imings_test.ts`,
-    },
-    {
-      code: notAScreenshotTestCode,
-      filename: notPerfPanelTestPath,
-    },
-    {
-      code: notAScreenshotTestDisabledCode,
-      filename: notPerfPanelTestPath,
-    },
-    {
-      code: notAScreenshotTestDisabledCode,
-      filename: `${perfPanelInteractionTestsPath}/timeline/timeline_test.ts`,
-    },
-  ],
-  invalid: [
-    {
-      code: enabledTestCode,
-      filename: notPerfPanelTestPath,
-      errors: [{messageId: 'invalidScreenshotTest'}],
-    },
-    {
-      code: disabledTestCode,
-      filename: notPerfPanelTestPath,
-      errors: [{messageId: 'invalidScreenshotTest'}],
-    },
-    {
-      code: enabledTestCode,
-      filename: 'test/unittests/front_end/panels/performance/timeline_test.ts',
-      errors: [{messageId: 'invalidScreenshotTest'}],
-    },
-  ],
-});

package/scripts/eslint_rules/tests/no-self-closing-custom-element-tagnames.test.ts

@@ -1,59 +0,0 @@
-// Copyright 2021 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-import rule from '../lib/no-self-closing-custom-element-tagnames.ts';
-
-import {RuleTester} from './utils/RuleTester.ts';
-
-new RuleTester().run('no-self-closing-custom-element-tagnames', rule, {
-  valid: [
-    {
-      code: 'Lit.html`<p></p>`',
-      filename: 'front_end/components/test.ts',
-    },
-    {
-      code: 'Lit.html`<input />`',
-      filename: 'front_end/components/test.ts',
-    },
-    {
-      code: 'Lit.html`<${DataGrid.litTagName}></${DataGrid.litTagName}>`',
-      filename: 'front_end/components/test.ts',
-    },
-    {
-      code: 'Lit.html`<p><${DataGrid.litTagName}></${DataGrid.litTagName}></p>`',
-      filename: 'front_end/components/test.ts',
-    },
-    {
-      code:
-          'Lit.html`<${DataGrid1.litTagName}><${DataGrid2.litTagName}></${DataGrid2.litTagName}></${DataGrid1.litTagName}>`',
-      filename: 'front_end/components/test.ts',
-    },
-    {
-      code: 'Lit.html`<${DataGrid1.litTagName}>\n</${DataGrid1.litTagName}>`',
-      filename: 'front_end/components/test.ts',
-    },
-  ],
-  invalid: [
-    {
-      code: 'Lit.html`<${DataGrid.litTagName} />`',
-      filename: 'front_end/components/test.ts',
-      errors: [{messageId: 'requiredEndTag'}],
-    },
-    {
-      code: 'Lit.html`<p><${DataGrid.litTagName} /></p>`',
-      filename: 'front_end/components/test.ts',
-      errors: [{messageId: 'requiredEndTag'}],
-    },
-    {
-      code: 'Lit.html`<${DataGrid1.litTagName}><${DataGrid2.litTagName} /></${DataGrid1.litTagName}>`',
-      filename: 'front_end/components/test.ts',
-      errors: [{messageId: 'requiredEndTag'}],
-    },
-    {
-      code: 'Lit.html`<${DataGrid.litTagName} .data=${{test: "Hello World"}}/>`',
-      filename: 'front_end/components/test.ts',
-      errors: [{messageId: 'requiredEndTag'}],
-    },
-  ],
-});

package/scripts/eslint_rules/tests/no-underscored-properties.test.ts

@@ -1,99 +0,0 @@
-// Copyright 2020 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-import rule from '../lib/no-underscored-properties.ts';
-
-import {RuleTester} from './utils/RuleTester.ts';
-
-new RuleTester().run('no-underscored-properties', rule, {
-  valid: [
-    {
-      filename: 'foo.ts',
-      code: `class X {
-        private foo: string = '';
-      }`,
-    },
-    {
-      filename: 'foo.ts',
-      code: `class X {
-        private foo() {
-        }
-      }`,
-    },
-    {
-      filename: 'foo.ts',
-      code: `class X {
-        public foo() {
-        }
-      }`,
-    },
-    {
-      filename: 'foo.ts',
-      code: `class X {
-        foo() {
-        }
-      }`,
-    },
-    {
-      filename: 'foo.ts',
-      code: `class X {
-        private _foo: string = '';
-        get foo() {
-          return this._foo;
-        }
-      }`,
-    },
-  ],
-
-  invalid: [
-    {
-      filename: 'foo.ts',
-      code: `class X {
-        private _foo: string = '';
-      }`,
-      errors: [
-        {messageId: 'underscorePrefix'},
-      ],
-    },
-    {
-      filename: 'foo.ts',
-      code: `class X {
-        public _foo: string = '';
-      }`,
-      errors: [
-        {messageId: 'underscorePrefix'},
-      ],
-    },
-    {
-      filename: 'foo.ts',
-      code: `class X {
-        _foo: string = '';
-      }`,
-      errors: [
-        {messageId: 'underscorePrefix'},
-      ],
-    },
-    {
-      filename: 'foo.ts',
-      code: `class X {
-        _foo() {
-        }
-      }`,
-      errors: [
-        {messageId: 'underscorePrefix'},
-      ],
-    },
-    {
-      filename: 'foo.ts',
-      code: `class X {
-        private _foo() {
-          console.log('I am private');
-        }
-      }`,
-      errors: [
-        {messageId: 'underscorePrefix'},
-      ],
-    },
-  ],
-});