Skip to content
Browse files

Merge pull request #13736 from chadhietala/glimmer-teardown

[Glimmer2] Cleanup teardown
  • Loading branch information...
2 parents b83b166 + ffd3d20 commit 22892c685cbc4ab6b4578c7175e54d8cfe7e13fd @krisselden krisselden committed on GitHub Jun 30, 2016
Showing with 14 additions and 0 deletions.
  1. +14 −0 packages/ember-glimmer/tests/integration/components/life-cycle-test.js
View
14 packages/ember-glimmer/tests/integration/components/life-cycle-test.js
@@ -10,6 +10,16 @@ class LifeCycleHooksTest extends RenderingTest {
this.components = {};
}
+ teardown() {
+ super();
+ this.assertHooks(
+ 'destroy',
+ ['the-top', 'willDestroyElement'],
+ ['the-middle', 'willDestroyElement'],
+ ['the-bottom', 'willDestroyElement']
+ );
+ }
+
/* abstract */
get ComponentClass() {
throw new Error('Not implemented: `ComponentClass`');
@@ -80,6 +90,10 @@ class LifeCycleHooksTest extends RenderingTest {
didUpdate(options) {
pushHook('didUpdate', options);
+ },
+
+ willDestroyElement() {
+ pushHook('willDestroyElement');
}
});

0 comments on commit 22892c6

Please sign in to comment.
Something went wrong with that request. Please try again.