wip: no @glimmer/interfaces ?????

glimmerjs · Nov 2, 2021 · d899490 · d899490
1 parent bfcc8a9
commit d899490
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/packages/@glimmer/manager/lib/internal/defaults.ts b/packages/@glimmer/manager/lib/internal/defaults.ts
@@ -1,4 +1,6 @@
-import { CAPABILITIES, CapturedArguments as Arguments } from '@glimmer/interfaces';
+import { CAPABILITIES } from '@glimmer/interfaces';
+
+import type { CapturedArguments as Arguments } from '@glimmer/interfaces';
 
 type FnArgs<Args extends Arguments = Arguments> =
   | [...Args['positional'], Args['named']]

diff --git a/packages/@glimmer/manager/test/managers-test.ts b/packages/@glimmer/manager/test/managers-test.ts
@@ -183,11 +183,18 @@ module('Managers', () => {
       assert.ok(typeof instance === 'object', 'manager is an internal manager');
       assert.ok(
         typeof instance.getHelper({}) === 'function',
-        'manager can generatew helper function'
+        'manager can generate helper function'
       );
       assert.equal(instance['factory'], factory, 'manager has correct delegate factory');
     });
 
+    test('it determines the default manager', (assert) => {
+      let helper = () => 0;
+      let instance = getInternalHelperManager(helper);
+
+      assert.ok(instance, helper as any);
+    });
+
     test('it works with internal helpers', (assert) => {
       let helper = () => {
         return UNDEFINED_REFERENCE;