fix(overlays): add default tag in OverlayMixin test-suite

ing-bank · Nov 10, 2021 · f69aec6 · f69aec6
1 parent 3c43184
commit f69aec6
Showing 1 changed file with 33 additions and 8 deletions.
diff --git a/packages/overlays/test-suites/OverlayMixin.suite.js b/packages/overlays/test-suites/OverlayMixin.suite.js
@@ -1,14 +1,22 @@
-import { expect, fixture, html, nextFrame, aTimeout } from '@open-wc/testing';
+import { LitElement } from '@lion/core';
+import {
+  aTimeout,
+  defineCE,
+  expect,
+  fixture,
+  html,
+  nextFrame,
+  unsafeStatic,
+} from '@open-wc/testing';
 import sinon from 'sinon';
-import { overlays } from '../src/overlays.js';
 import { OverlayController } from '../src/OverlayController.js';
+import { OverlayMixin } from '../src/OverlayMixin.js';
+import { overlays } from '../src/overlays.js';
 
 /**
  * @typedef {import('../types/OverlayConfig').OverlayConfig} OverlayConfig
  * @typedef {import('../types/OverlayMixinTypes').DefineOverlayConfig} DefineOverlayConfig
  * @typedef {import('../types/OverlayMixinTypes').OverlayHost} OverlayHost
- * @typedef {import('../types/OverlayMixinTypes').OverlayMixin} OverlayMixin
- * @typedef {import('@lion/core').LitElement} LitElement
  * @typedef {LitElement & OverlayHost & {_overlayCtrl:OverlayController}} OverlayEl
  */
 
@@ -19,10 +27,21 @@ function getGlobalOverlayNodes() {
 }
 
 /**
- * @param {{tagString:string, tag: object, suffix?:string}} config
+ * @param {{tagString:string, tag: object, suffix?:string}} [customConfig]
  */
-export function runOverlayMixinSuite({ tagString, tag, suffix = '' }) {
-  describe(`OverlayMixin${suffix}`, () => {
+export function runOverlayMixinSuite(customConfig) {
+  const cfg = {
+    tagString: null,
+    suffix: '',
+    ...customConfig,
+  };
+
+  describe(`OverlayMixin${cfg.suffix}`, () => {
+    class TestOverlay extends OverlayMixin(LitElement) {}
+
+    cfg.tagString = cfg.tagString ? cfg.tagString : defineCE(TestOverlay);
+    const tag = unsafeStatic(cfg.tagString);
+
     it('should not be opened by default', async () => {
       const el = /** @type {OverlayEl} */ (
         await fixture(html`
@@ -325,7 +344,13 @@ export function runOverlayMixinSuite({ tagString, tag, suffix = '' }) {
     });
   });
 
-  describe(`OverlayMixin${suffix} nested`, () => {
+  describe(`OverlayMixin${cfg.suffix} nested`, () => {
+    class TestOverlayNested extends OverlayMixin(LitElement) {}
+
+    cfg.tagString = cfg.tagString ? cfg.tagString : defineCE(TestOverlayNested);
+    const { tagString } = cfg;
+    const tag = unsafeStatic(cfg.tagString);
+
     // For some reason, globalRootNode is not cleared properly on disconnectedCallback from previous overlay test fixtures...
     // Not sure why this "bug" happens...
     beforeEach(() => {