From 0a78bfb836f520d3709a32c8a978d08d33198d83 Mon Sep 17 00:00:00 2001 From: "Kevin (Kun) \"Kassimo\" Qian" Date: Thu, 16 Jan 2020 16:42:58 -0800 Subject: Add Deno.symbols and move internal fields for test (#3693) --- cli/js/mixins/dom_iterable.ts | 4 +++- cli/js/mixins/dom_iterable_test.ts | 5 ++++- 2 files changed, 7 insertions(+), 2 deletions(-) (limited to 'cli/js/mixins') diff --git a/cli/js/mixins/dom_iterable.ts b/cli/js/mixins/dom_iterable.ts index bbd1905ce..dcbd5150e 100644 --- a/cli/js/mixins/dom_iterable.ts +++ b/cli/js/mixins/dom_iterable.ts @@ -3,6 +3,7 @@ import { DomIterable } from "../dom_types.ts"; import { window } from "../window.ts"; import { requiredArguments } from "../util.ts"; +import { exposeForTest } from "../internals.ts"; // eslint-disable-next-line @typescript-eslint/no-explicit-any type Constructor = new (...args: any[]) => T; @@ -10,7 +11,6 @@ type Constructor = new (...args: any[]) => T; /** Mixes in a DOM iterable methods into a base class, assumes that there is * a private data iterable that is part of the base class, located at * `[dataSymbol]`. - * TODO Don't expose DomIterableMixin from "deno" namespace. */ export function DomIterableMixin( Base: TBase, @@ -80,3 +80,5 @@ export function DomIterableMixin( return DomIterable; } + +exposeForTest("DomIterableMixin", DomIterableMixin); diff --git a/cli/js/mixins/dom_iterable_test.ts b/cli/js/mixins/dom_iterable_test.ts index d3147b061..57e655989 100644 --- a/cli/js/mixins/dom_iterable_test.ts +++ b/cli/js/mixins/dom_iterable_test.ts @@ -21,7 +21,10 @@ function setup() { // This is using an internal API we don't want published as types, so having // to cast to any to "trick" TypeScript // eslint-disable-next-line @typescript-eslint/no-explicit-any - DomIterable: (Deno as any).DomIterableMixin(Base, dataSymbol) + DomIterable: (Deno[Deno.symbols.internal] as any).DomIterableMixin( + Base, + dataSymbol + ) }; } -- cgit v1.2.3