ayo/lion
1
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(core): no registration of same class twice w/o scoped-registries polyfill

Browse source
This commit is contained in:
Thijs Louisse 2024-11-10 11:04:01 +01:00 committed by Thijs Louisse
parent 8dc51c1066
commit 510f8b931f
2 changed files with 36 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
packages/ui/components/core/src/ScopedElementsMixin.js
View file

@ -89,8 +89,8 @@ const ScopedElementsMixinImplementation = superclass =>
*/
defineScopedElement(tagName, classToBeRegistered) {
const registeredClass = this.registry.get(tagName);
const isAlreadyRegistered = registeredClass && registeredClass === classToBeRegistered;
if (isAlreadyRegistered && !supportsScopedRegistry()) {
const isNewClassWithSameName = registeredClass && registeredClass !== classToBeRegistered;
if (!supportsScopedRegistry() && isNewClassWithSameName) {
// eslint-disable-next-line no-console
console.error(
[

35
packages/ui/components/core/test/ScopedElementsMixin.test.js
View file

@ -6,9 +6,9 @@ import {
} from '@lit-labs/testing/fixtures.js';
import { LitElement, html } from 'lit';
import sinon from 'sinon';
import { browserDetection } from '../src/browserDetection.js';
import { ScopedElementsMixin, supportsScopedRegistry } from '../src/ScopedElementsMixin.js';
import { browserDetection } from '../src/browserDetection.js';
const hasRealScopedRegistrySupport = supportsScopedRegistry();
const originalShadowRootProps = {
@ -78,6 +78,8 @@ describe('ScopedElementsMixin', () => {
describe('When scoped registries are supported', () => {
it('registers elements on local registry', async () => {
if (!hasRealScopedRegistrySupport) return;
const ceDefineSpy = sinon.spy(customElements, 'define');
const el = /** @type {ScopedElementsHost} */ (
@ -127,5 +129,36 @@ describe('ScopedElementsMixin', () => {
expect(ceDefineSpy.calledWith('scoped-elements-child-no-reg')).to.be.true;
ceDefineSpy.restore();
});
it('fails when different classes are registered under different name', async () => {
class ScopedElementsHostNoReg2 extends ScopedElementsMixin(LitElement) {
static scopedElements = { 'scoped-elements-child-no-reg': class extends HTMLElement {} };
render() {
return html`<scoped-elements-child-no-reg></scoped-elements-child-no-reg>`;
}
}
customElements.define('scoped-elements-host-no-reg-2', ScopedElementsHostNoReg2);
const errorSpy = sinon.spy(console, 'error');
/** @type {ScopedElementsHostNoReg2} */ (
await fixture(html`<scoped-elements-host-no-reg></scoped-elements-host-no-reg>`)
);
/** @type {ScopedElementsHostNoReg2} */ (
await fixture(html`<scoped-elements-host-no-reg-2></scoped-elements-host-no-reg-2>`)
);
expect(errorSpy.args[0][0]).to.equal(
[
'You are trying to re-register the "scoped-elements-child-no-reg" custom element with a different class via ScopedElementsMixin.',
'This is only possible with a CustomElementRegistry.',
'Your browser does not support this feature so you will need to load a polyfill for it.',
'Load "@webcomponents/scoped-custom-element-registry" before you register ANY web component to the global customElements registry.',
'e.g. add "<script src="/node_modules/@webcomponents/scoped-custom-element-registry/scoped-custom-element-registry.min.js"></script>" as your first script tag.',
'For more details you can visit https://open-wc.org/docs/development/scoped-elements/',
].join('\n'),
);
errorSpy.restore();
});
});
});