Remove '@ember/application's lazy load / onLoad stuff

Cleanup

Cleanup

Cleanup

Cleanup

Cleanup

Cleanup

manual advanceReadiness can't be used in Test.registerHelper -- do we even support Test.registerHelper anymore? advanceReadiness is public, so that's annoying

Author: NullVoxPopuli

lib/index.js

@@ -54,7 +54,6 @@ const shim = addonV1Shim(path.join(__dirname, '..'), {
           './dist/dev/packages/@ember/-internals/views/lib/views/states.js',
           './dist/dev/packages/@ember/application/index.js',
           './dist/dev/packages/@ember/application/instance.js',
-          './dist/dev/packages/@ember/application/lib/lazy_load.js',
           './dist/dev/packages/@ember/application/namespace.js',
           './dist/dev/packages/@ember/array/-internals.js',
           './dist/dev/packages/@ember/array/index.js',

packages/@ember/application/index.ts

@@ -10,7 +10,6 @@ import { assert } from '@ember/debug';
 import { DEBUG } from '@glimmer/env';
 import { join, once, run, schedule } from '@ember/runloop';
 import { libraries } from '@ember/-internals/metal';
-import { _loaded, onLoad, runLoadHooks } from './lib/lazy_load';
 import { RSVP } from '@ember/-internals/runtime';
 import { EventDispatcher } from '@ember/-internals/views';
 import Route from '@ember/routing/route';
@@ -711,7 +710,6 @@ class Application extends Engine {
 
     try {
       this.runInitializers();
-      runLoadHooks('application', this);
       this.advanceReadiness();
       // Continues to `didBecomeReady`
     } catch (error) {
@@ -883,10 +881,6 @@ class Application extends Engine {
   willDestroy() {
     super.willDestroy();
 
-    if (_loaded['application'] === this) {
-      _loaded['application'] = undefined;
-    }
-
     if (this._applicationInstances.size) {
       this._applicationInstances.forEach((i) => i.destroy());
       this._applicationInstances.clear();
@@ -1137,4 +1131,4 @@ function commonSetupRegistry(registry: Registry) {
   registry.register('service:router', RouterService);
 }
 
-export { Application as default, _loaded, onLoad, runLoadHooks };
+export { Application as default };

packages/@ember/application/lib/lazy_load.ts

@@ -6,7 +6,6 @@ import { getDebugFunction, setDebugFunction } from '@ember/debug';
 import EmberRoute from '@ember/routing/route';
 import Router from '@ember/routing/router';
 import NoneLocation from '@ember/routing/none-location';
-import { _loaded } from '@ember/application';
 import Controller from '@ember/controller';
 import EmberObject from '@ember/object';
 import {
@@ -298,14 +297,6 @@ moduleFor(
       );
     }
 
-    [`@test does not leak itself in onLoad._loaded`](assert) {
-      assert.equal(_loaded.application, undefined);
-      runTask(() => this.createApplication());
-      assert.equal(_loaded.application, this.application);
-      runTask(() => this.application.destroy());
-      assert.equal(_loaded.application, undefined);
-    }
-
     [`@test can build a registry via Application.buildRegistry() --- simulates ember-test-helpers`](
       assert
     ) {

packages/@ember/application/tests/application_test.js

@@ -79,8 +79,6 @@ import EmberArrayProxy from '@ember/array/proxy';
 import EmberApplication, {
   getOwner as applicationGetOwner,
   setOwner as applicationSetOwner,
-  onLoad as applicationOnLoad,
-  runLoadHooks as applicationRunLoadHooks,
 } from '@ember/application';
 import EmberApplicationInstance from '@ember/application/instance';
 import EmberNamespace from '@ember/application/namespace';
@@ -207,8 +205,6 @@ namespace Ember {
 
   // ****@ember/application****
   export const getOwner = applicationGetOwner;
-  export const onLoad = applicationOnLoad;
-  export const runLoadHooks = applicationRunLoadHooks;
   export const setOwner = applicationSetOwner;
   export const Application = EmberApplication;
   export type Application = EmberApplication;
@@ -657,8 +653,6 @@ Object.defineProperty(Ember, 'testing', {
   enumerable: false,
 });
 
-applicationRunLoadHooks('Ember.Application', EmberApplication);
-
 let EmberHandlebars: EmberHandlebars = {
   template,
   Utils: {},
@@ -727,7 +721,4 @@ function defineEmberTestingLazyLoad(key: 'Test') {
 
 defineEmberTestingLazyLoad('Test');
 
-// @ts-expect-error Per types, runLoadHooks requires a second parameter. Should we loosen types?
-applicationRunLoadHooks('Ember');
-
 export default Ember;

packages/@ember/application/tests/lazy_load_test.js

@@ -120,8 +120,6 @@ import { DEPRECATIONS } from '@ember/-internals/deprecations';
 let allExports = [
   ['Application', '@ember/application', 'default', test0],
   ['getOwner', '@ember/application', 'getOwner', test0],
-  ['onLoad', '@ember/application', 'onLoad', test0],
-  ['runLoadHooks', '@ember/application', 'runLoadHooks', test0],
   ['setOwner', '@ember/application', 'setOwner', test0],
   ['ApplicationInstance', '@ember/application/instance', 'default', test1],
   ['Namespace', '@ember/application/namespace', 'default', test2],

packages/ember/barrel.ts

@@ -352,7 +352,6 @@ module.exports = {
     'once',
     'one',
     'oneWay',
-    'onLoad',
     'onUpdateURL',
     'options',
     'optionsForType',
@@ -438,7 +437,6 @@ module.exports = {
     'runInDebug',
     'runInitializers',
     'runInstanceInitializers',
-    'runLoadHooks',
     'schedule',
     'scheduleOnce',
     'send',