Remove addStrategy function

Author: koddsson

src/lazy-define.ts

@@ -55,13 +55,6 @@ const strategies: Record<string, Strategy> = {
   visible
 }
 
-export function addStrategy(name: string, strategy: Strategy) {
-  if (name in strategies) {
-    throw new Error(`Strategy ${name} already exists!`)
-  }
-  strategies[name] = strategy
-}
-
 const timers = new WeakMap<Element, number>()
 function scan(node: Element = document.body) {
   cancelAnimationFrame(timers.get(node) || 0)

test/lazy-define.ts

@@ -1,33 +1,6 @@
 import {expect, fixture, html} from '@open-wc/testing'
 import {spy} from 'sinon'
-import {addStrategy, lazyDefine} from '../src/lazy-define.js'
-
-describe('addStrategy', () => {
-  let strategy: ReturnType<typeof spy>
-  let promise: Promise<void>
-  let resolve: () => void
-
-  beforeEach(() => {
-    strategy = spy(() => (promise = new Promise(res => (resolve = res))))
-  })
-
-  it('adds a new strategy', async () => {
-    const onDefine = spy()
-    lazyDefine('foo-bar', onDefine)
-    addStrategy('test', strategy)
-    await fixture(html`<foo-bar data-load-on="test"></foo-bar>`)
-
-    expect(strategy).to.be.calledOnceWith('foo-bar')
-    expect(onDefine).to.have.callCount(0)
-    resolve()
-    await promise
-    expect(onDefine).to.be.callCount(1)
-  })
-
-  it("doesn't overwrite a existing strategy", () => {
-    expect(() => addStrategy('ready', spy())).to.throw(/already exists/)
-  })
-})
+import {lazyDefine} from '../src/lazy-define.js'
 
 describe('lazyDefine', () => {
   describe('ready strategy', () => {