From fa78a43402a7cd2553067da3ce4e4f11d9578819 Mon Sep 17 00:00:00 2001 From: Chris Thoburn Date: Wed, 7 Dec 2022 20:24:35 -0800 Subject: [PATCH] Remove excessive destroy call I noticed while migrating some things away from EmberObject (which debounces destroy) that destroy was being triggered more than once due to this. It appears the upstream change has long-since landed and we can remove this and speed up everyone's tests <3 --- addon-test-support/@ember/test-helpers/setup-context.ts | 6 ------ 1 file changed, 6 deletions(-) diff --git a/addon-test-support/@ember/test-helpers/setup-context.ts b/addon-test-support/@ember/test-helpers/setup-context.ts index cdcdf26a3..08bf715f1 100644 --- a/addon-test-support/@ember/test-helpers/setup-context.ts +++ b/addon-test-support/@ember/test-helpers/setup-context.ts @@ -200,12 +200,6 @@ function cleanup(context: BaseContext) { (Ember as any).testing = false; unsetContext(); - - if (isTestContext(context)) { - // this should not be required, but until https://github.com/emberjs/ember.js/pull/19106 - // lands in a 3.20 patch release - context.owner.destroy(); - } } /**