See what happens when we delete ember-testing's test-helpers

broccoli/amd-compat-entrypoints/ember-testing.js

@@ -21,30 +21,6 @@ d('ember-testing/lib/ext/application', emberTestingLibExtApplication);
 import * as emberTestingLibExtRsvp from 'ember-testing/lib/ext/rsvp';
 d('ember-testing/lib/ext/rsvp', emberTestingLibExtRsvp);
 
-import * as emberTestingLibHelpers from 'ember-testing/lib/helpers';
-d('ember-testing/lib/helpers', emberTestingLibHelpers);
-
-import * as emberTestingLibHelpersAndThen from 'ember-testing/lib/helpers/and_then';
-d('ember-testing/lib/helpers/and_then', emberTestingLibHelpersAndThen);
-
-import * as emberTestingLibHelpersCurrentPath from 'ember-testing/lib/helpers/current_path';
-d('ember-testing/lib/helpers/current_path', emberTestingLibHelpersCurrentPath);
-
-import * as emberTestingLibHelpersCurrentRouteName from 'ember-testing/lib/helpers/current_route_name';
-d('ember-testing/lib/helpers/current_route_name', () => emberTestingLibHelpersCurrentRouteName);
-
-import * as emberTestingLibHelpersCurrentUrl from 'ember-testing/lib/helpers/current_url';
-d('ember-testing/lib/helpers/current_url', emberTestingLibHelpersCurrentUrl);
-
-import * as emberTestingLibHelpersPauseTest from 'ember-testing/lib/helpers/pause_test';
-d('ember-testing/lib/helpers/pause_test', emberTestingLibHelpersPauseTest);
-
-import * as emberTestingLibHelpersVisit from 'ember-testing/lib/helpers/visit';
-d('ember-testing/lib/helpers/visit', emberTestingLibHelpersVisit);
-
-import * as emberTestingLibHelpersWait from 'ember-testing/lib/helpers/wait';
-d('ember-testing/lib/helpers/wait', emberTestingLibHelpersWait);
-
 import * as emberTestingLibInitializers from 'ember-testing/lib/initializers';
 d('ember-testing/lib/initializers', emberTestingLibInitializers);
 
@@ -60,9 +36,6 @@ d('ember-testing/lib/test', emberTestingLibTest);
 import * as emberTestingLibTestAdapter from 'ember-testing/lib/test/adapter';
 d('ember-testing/lib/test/adapter', emberTestingLibTestAdapter);
 
-import * as emberTestingLibTestHelpers from 'ember-testing/lib/test/helpers';
-d('ember-testing/lib/test/helpers', emberTestingLibTestHelpers);
-
 import * as emberTestingLibTestOnInjectHelpers from 'ember-testing/lib/test/on_inject_helpers';
 d('ember-testing/lib/test/on_inject_helpers', emberTestingLibTestOnInjectHelpers);
 

lib/index.js

@@ -207,20 +207,11 @@ const shim = addonV1Shim(path.join(__dirname, '..'), {
             './dist/packages/ember-testing/lib/adapters/qunit.js',
             './dist/packages/ember-testing/lib/ext/application.js',
             './dist/packages/ember-testing/lib/ext/rsvp.js',
-            './dist/packages/ember-testing/lib/helpers.js',
-            './dist/packages/ember-testing/lib/helpers/and_then.js',
-            './dist/packages/ember-testing/lib/helpers/current_path.js',
-            './dist/packages/ember-testing/lib/helpers/current_route_name.js',
-            './dist/packages/ember-testing/lib/helpers/current_url.js',
-            './dist/packages/ember-testing/lib/helpers/pause_test.js',
-            './dist/packages/ember-testing/lib/helpers/visit.js',
-            './dist/packages/ember-testing/lib/helpers/wait.js',
             './dist/packages/ember-testing/lib/initializers.js',
             './dist/packages/ember-testing/lib/public-api.js',
             './dist/packages/ember-testing/lib/setup_for_testing.js',
             './dist/packages/ember-testing/lib/test.js',
             './dist/packages/ember-testing/lib/test/adapter.js',
-            './dist/packages/ember-testing/lib/test/helpers.js',
             './dist/packages/ember-testing/lib/test/on_inject_helpers.js',
             './dist/packages/ember-testing/lib/test/pending_requests.js',
             './dist/packages/ember-testing/lib/test/promise.js',

package.json

@@ -387,4 +387,4 @@
     }
   },
   "packageManager": "pnpm@10.5.0"
-}
+}

packages/ember-testing/lib/ext/application.ts

@@ -1,6 +1,5 @@
 import EmberApplication from '@ember/application';
 import setupForTesting from '../setup_for_testing';
-import { helpers } from '../test/helpers';
 import TestPromise, { resolve, getLastPromise } from '../test/promise';
 import run from '../test/run';
 import { invokeInjectHelpersCallbacks } from '../test/on_inject_helpers';
@@ -133,14 +132,6 @@
     });
 
     this.testHelpers = {};
-    for (let name in helpers) {
-      // SAFETY: It is safe to access a property on an object
-      this.originalMethods[name] = (this.helperContainer as any)[name];
-      // SAFETY: It is not quite as safe to do this, but it _seems_ to be ok.
-      this.testHelpers[name] = (this.helperContainer as any)[name] = helper(this, name);
-      // SAFETY: We checked that it exists
-      protoWrap(TestPromise.prototype, name, helper(this, name), helpers[name]!.meta.wait);
-    }
 
     invokeInjectHelpersCallbacks(this);
   },
@@ -163,7 +154,7 @@
       return;
     }
 
     for (let name in helpers) {
       this.helperContainer[name] = this.originalMethods[name];
       // SAFETY: This is a weird thing, but it's not technically unsafe here.
       delete (TestPromise.prototype as any)[name];
@@ -176,7 +167,7 @@
 // This method is no longer needed
 // But still here for backwards compatibility
 // of helper chaining
 function protoWrap(proto: TestPromise<unknown>, name: string, callback: AnyFn, isAsync: boolean) {
   // SAFETY: This isn't entirely safe, but it _seems_ to be ok.
   (proto as any)[name] = function (...args: unknown[]) {
     if (isAsync) {
@@ -190,8 +181,8 @@
   };
 }
 
 function helper(app: TestableApp, name: string) {
   let helper = helpers[name];
   assert(`[BUG] Missing helper: ${name}`, helper);
   let fn = helper.method;
   let meta = helper.meta;

packages/ember-testing/lib/public-api.ts

@@ -5,5 +5,4 @@ export { default as QUnitAdapter } from './adapters/qunit';
 
 import './ext/application';
 import './ext/rsvp'; // setup RSVP + run loop integration
-import './helpers'; // adds helpers to helpers object in Test
 import './initializers'; // to setup initializer

packages/ember-testing/lib/test.ts

@@ -1,7 +1,6 @@
 /**
   @module ember
 */
-import { helpers, registerHelper, registerAsyncHelper, unregisterHelper } from './test/helpers';
 import { onInjectHelpers } from './test/on_inject_helpers';
 import TestPromise, { promise, resolve } from './test/promise';
 import { checkWaiters, registerWaiter, unregisterWaiter } from './test/waiters';
@@ -21,18 +20,6 @@ import { getAdapter, setAdapter } from './test/adapter';
   @public
 */
 const Test = {
-  /**
-    Hash containing all known test helpers.
-
-    @property _helpers
-    @private
-    @since 1.7.0
-  */
-  _helpers: helpers,
-
-  registerHelper,
-  registerAsyncHelper,
-  unregisterHelper,
   onInjectHelpers,
   Promise: TestPromise,
   promise,