test-helpers: Add explicit return types

without this our `test:types` job fails for some reason
glimmerjs · Dec 24, 2018 · e5a4b68 · e5a4b68
1 parent 4560212
commit e5a4b68
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/packages/@glimmer/test-helpers/lib/environment.ts b/packages/@glimmer/test-helpers/lib/environment.ts
@@ -26,7 +26,7 @@ export class TestDynamicScope implements DynamicScope {
     return this.bucket[key];
   }
 
-  set(key: string, reference: PathReference<Opaque>) {
+  set(key: string, reference: PathReference<Opaque>): PathReference<Opaque> {
     return (this.bucket[key] = reference);
   }
 

diff --git a/packages/@glimmer/test-helpers/lib/environment/modes/eager/render-delegate.ts b/packages/@glimmer/test-helpers/lib/environment/modes/eager/render-delegate.ts
@@ -157,7 +157,7 @@ export default class EagerRenderDelegate implements RenderDelegate {
     };
   }
 
-  getSelf(context: Opaque) {
+  getSelf(context: Opaque): UpdatableReference<Opaque> {
     return new UpdatableReference(context);
   }
 

diff --git a/packages/@glimmer/test-helpers/lib/render-test.ts b/packages/@glimmer/test-helpers/lib/render-test.ts
@@ -649,7 +649,7 @@ export class RehydrationDelegate implements RenderDelegate {
     return this.serialize(element);
   }
 
-  getSelf(context: Opaque) {
+  getSelf(context: Opaque): UpdatableReference<Opaque> {
     return new UpdatableReference(context);
   }
 
@@ -993,13 +993,13 @@ export function renderTemplate(
   env: LazyTestEnvironment,
   self: PathReference<Opaque>,
   builder: ElementBuilder
-) {
+): RenderResult<Opaque> {
   let template = env.preprocess(src);
   let iterator = env.renderMain(template, self, builder);
   return renderSync(env, iterator);
 }
 
-export function renderSync(env: Environment, iterator: TemplateIterator) {
+export function renderSync(env: Environment, iterator: TemplateIterator): RenderResult<Opaque> {
   env.begin();
 
   let iteratorResult: IteratorResult<RenderResult>;
-Original file line number
+Diff line change
@@ Expand Up @@
         };
       }
-      getSelf(context: Opaque) {
+      getSelf(context: Opaque): UpdatableReference<Opaque> {
         return new UpdatableReference(context);
       }
@@ Expand Down @@