Unify test infrastructure on precompileTemplate, remove addTemplate/addComponent/registerComponent

Remove duplicated test helper methods (addTemplate, addComponent, registerComponent,
defComponent, defineComponent) from the test infrastructure and migrate all ~90 test
files to use precompileTemplate from @ember/template-compilation directly.

Test files now use:
- precompileTemplate(template, { strictMode: false }) for loose-mode templates
- precompileTemplate(template, { strictMode: true, scope: () => ({...}) }) for strict-mode
- setComponentTemplate() + this.owner.register() / this.add() for component registration
- Runtime compile from internal-test-helpers/lib/compile only for edge cases
  (keyword shadowing, dynamic template strings, intentional compile errors)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: NullVoxPopuli

packages/@ember/-internals/glimmer/tests/integration/application/debug-render-tree-test.ts

@@ -2,7 +2,6 @@ import {
   moduleFor,
   ApplicationTestCase,
   ModuleBasedTestResolver,
-  strip,
   runTaskNext,
 } from 'internal-test-helpers';
 
@@ -13,7 +12,7 @@ import Controller from '@ember/controller';
 import Engine from '@ember/engine';
 import { next } from '@ember/runloop';
 
-import { compile } from '../../utils/helpers';
+import { precompileTemplate } from '@ember/template-compilation';
 import { setComponentTemplate } from '@glimmer/manager';
 import { templateOnlyComponent } from '@glimmer/runtime';
 
@@ -62,7 +61,7 @@ moduleFor(
     setupAppAndRoutableEngine(hooks = []) {
       let self = this;
 
-      this.addTemplate('application', 'Application{{outlet}}');
+      this.add('template:application', precompileTemplate('Application{{outlet}}'));
 
       this.router.map(function () {
         this.mount('blog');
@@ -110,7 +109,10 @@ moduleFor(
                 queryParams = ['official'];
               }
             );
-            this.register('template:application', compile('Engine{{this.lang}}{{outlet}}'));
+            this.register(
+              'template:application',
+              precompileTemplate('Engine{{this.lang}}{{outlet}}')
+            );
             this.register(
               'route:application',
               class extends Route {
@@ -144,7 +146,7 @@ moduleFor(
 
           init() {
             super.init(...arguments);
-            this.register('template:application', compile('Engine'));
+            this.register('template:application', precompileTemplate('Engine'));
             this.register(
               'controller:application',
               class extends Controller {
@@ -160,7 +162,7 @@ moduleFor(
     }
 
     setupRoutelessEngine(hooks) {
-      this.addTemplate('application', 'Application{{mount "chat-engine"}}');
+      this.add('template:application', precompileTemplate('Application{{mount "chat-engine"}}'));
       this.add(
         'route:application',
         class extends Route {
@@ -182,12 +184,9 @@ moduleFor(
     ['@test sharing a template between engine and application has separate refinements']() {
       this.assert.expect(1);
 
-      let sharedTemplate = compile(strip`
-      <h1>{{this.contextType}}</h1>
-      {{ambiguous-curlies}}
-
-      {{outlet}}
-    `);
+      let sharedTemplate = precompileTemplate(
+        '<h1>{{this.contextType}}</h1>{{ambiguous-curlies}}{{outlet}}'
+      );
 
       this.add('template:application', sharedTemplate);
       this.add(
@@ -220,7 +219,7 @@ moduleFor(
             this.register('template:application', sharedTemplate);
             this.register(
               'component:ambiguous-curlies',
-              setComponentTemplate(compile(`<p>Component!</p>`), templateOnlyComponent())
+              setComponentTemplate(precompileTemplate(`<p>Component!</p>`), templateOnlyComponent())
             );
           }
         }
@@ -234,21 +233,17 @@ moduleFor(
     ['@test sharing a layout between engine and application has separate refinements']() {
       this.assert.expect(1);
 
-      let sharedLayout = compile(strip`
-        {{ambiguous-curlies}}
-      `);
+      let sharedLayout = precompileTemplate('{{ambiguous-curlies}}');
 
       let sharedComponent = class extends Component {
         layout = sharedLayout;
       };
 
-      this.addTemplate(
-        'application',
-        strip`
-          <h1>Application</h1>
-          {{my-component ambiguous-curlies="Local Data!"}}
-          {{outlet}}
-        `
+      this.add(
+        'template:application',
+        precompileTemplate(
+          '<h1>Application</h1>{{my-component ambiguous-curlies="Local Data!"}}{{outlet}}'
+        )
       );
 
       this.add('component:my-component', sharedComponent);
@@ -267,16 +262,12 @@ moduleFor(
             super.init(...arguments);
             this.register(
               'template:application',
-              compile(strip`
-                <h1>Engine</h1>
-                {{my-component}}
-                {{outlet}}
-              `)
+              precompileTemplate('<h1>Engine</h1>{{my-component}}{{outlet}}')
             );
             this.register('component:my-component', sharedComponent);
             this.register(
               'component:ambiguous-curlies',
-              setComponentTemplate(compile(`<p>Component!</p>`), templateOnlyComponent())
+              setComponentTemplate(precompileTemplate(`<p>Component!</p>`), templateOnlyComponent())
             );
           }
         }
@@ -363,7 +354,7 @@ moduleFor(
 
           init() {
             super.init(...arguments);
-            this.register('template:application', compile('Engine{{outlet}}'));
+            this.register('template:application', precompileTemplate('Engine{{outlet}}'));
             this.register(
               'route:application',
               class extends Route {
@@ -405,7 +396,10 @@ moduleFor(
             }
           }
         );
-        this.register('template:application_error', compile('Error! {{@model.message}}'));
+        this.register(
+          'template:application_error',
+          precompileTemplate('Error! {{@model.message}}')
+        );
         this.register(
           'route:post',
           class extends Route {
@@ -444,7 +438,7 @@ moduleFor(
             }
           }
         );
-        this.register('template:error', compile('Error! {{@model.message}}'));
+        this.register('template:error', precompileTemplate('Error! {{@model.message}}'));
         this.register(
           'route:post',
           class extends Route {
@@ -483,7 +477,7 @@ moduleFor(
             }
           }
         );
-        this.register('template:post_error', compile('Error! {{@model.message}}'));
+        this.register('template:post_error', precompileTemplate('Error! {{@model.message}}'));
         this.register(
           'route:post',
           class extends Route {
@@ -522,7 +516,7 @@ moduleFor(
             }
           }
         );
-        this.register('template:post.error', compile('Error! {{@model.message}}'));
+        this.register('template:post.error', precompileTemplate('Error! {{@model.message}}'));
         this.register(
           'route:post.comments',
           class extends Route {
@@ -563,8 +557,8 @@ moduleFor(
             }
           }
         );
-        this.register('template:application_loading', compile('Loading'));
-        this.register('template:post', compile('Post'));
+        this.register('template:application_loading', precompileTemplate('Loading'));
+        this.register('template:post', precompileTemplate('Post'));
         this.register(
           'route:post',
           class extends Route {
@@ -610,8 +604,8 @@ moduleFor(
             }
           }
         );
-        this.register('template:loading', compile('Loading'));
-        this.register('template:post', compile('Post'));
+        this.register('template:loading', precompileTemplate('Loading'));
+        this.register('template:post', precompileTemplate('Post'));
         this.register(
           'route:post',
           class extends Route {
@@ -647,10 +641,10 @@ moduleFor(
 
       this.setupAppAndRoutableEngine();
       this.additionalEngineRegistrations(function () {
-        this.register('template:post', compile('{{outlet}}'));
-        this.register('template:post.comments', compile('Comments'));
-        this.register('template:post.likes_loading', compile('Loading'));
-        this.register('template:post.likes', compile('Likes'));
+        this.register('template:post', precompileTemplate('{{outlet}}'));
+        this.register('template:post.comments', precompileTemplate('Comments'));
+        this.register('template:post.likes_loading', precompileTemplate('Loading'));
+        this.register('template:post.likes', precompileTemplate('Likes'));
         this.register(
           'route:post.likes',
           class extends Route {
@@ -687,8 +681,8 @@ moduleFor(
 
       this.setupAppAndRoutableEngine();
       this.additionalEngineRegistrations(function () {
-        this.register('template:post', compile('{{outlet}}'));
-        this.register('template:post.comments', compile('Comments'));
+        this.register('template:post', precompileTemplate('{{outlet}}'));
+        this.register('template:post.comments', precompileTemplate('Comments'));
         this.register(
           'route:post.loading',
           class extends Route {
@@ -697,8 +691,8 @@ moduleFor(
             }
           }
         );
-        this.register('template:post.loading', compile('Loading'));
-        this.register('template:post.likes', compile('Likes'));
+        this.register('template:post.loading', precompileTemplate('Loading'));
+        this.register('template:post.likes', precompileTemplate('Likes'));
         this.register(
           'route:post.likes',
           class extends Route {
@@ -730,11 +724,12 @@ moduleFor(
     ["@test query params don't have stickiness by default between model"](assert) {
       assert.expect(1);
 
-      let tmpl = '<LinkTo @route="category" @model={{1337}}>Category 1337</LinkTo>';
-
       this.setupAppAndRoutableEngine();
       this.additionalEngineRegistrations(function () {
-        this.register('template:category', compile(tmpl));
+        this.register(
+          'template:category',
+          precompileTemplate('<LinkTo @route="category" @model={{1337}}>Category 1337</LinkTo>')
+        );
       });
 
       return this.visit('/blog/category/1?type=news').then(() => {
@@ -749,11 +744,12 @@ moduleFor(
     '@test query params only transitions work properly'(assert) {
       assert.expect(1);
 
-      let tmpl = '<LinkTo @query={{hash type="news"}}>News</LinkTo>';
-
       this.setupAppAndRoutableEngine();
       this.additionalEngineRegistrations(function () {
-        this.register('template:category', compile(tmpl));
+        this.register(
+          'template:category',
+          precompileTemplate('<LinkTo @query={{hash type="news"}}>News</LinkTo>')
+        );
       });
 
       return this.visit('/blog/category/1').then(() => {
@@ -769,11 +765,14 @@ moduleFor(
       assert
     ) {
       assert.expect(2);
-      let tmpl =
-        '<LinkTo @route="author" @model={{1337}} class="author-1337">Author 1337</LinkTo><LinkTo @route="author" @model=1 class="author-1">Author 1</LinkTo>';
       this.setupAppAndRoutableEngine();
       this.additionalEngineRegistrations(function () {
-        this.register('template:author', compile(tmpl));
+        this.register(
+          'template:author',
+          precompileTemplate(
+            '<LinkTo @route="author" @model={{1337}} class="author-1337">Author 1337</LinkTo><LinkTo @route="author" @model=1 class="author-1">Author 1</LinkTo>'
+          )
+        );
       });
 
       await this.visit('/blog/author/1?official=true');
@@ -840,7 +839,9 @@ moduleFor(
 
             this.register(
               'template:application',
-              compile('Engine<div class="lazy-query-param">{{this.lazyQueryParam}}</div>{{outlet}}')
+              precompileTemplate(
+                'Engine<div class="lazy-query-param">{{this.lazyQueryParam}}</div>{{outlet}}'
+              )
             );
 
             this.register(

packages/@ember/-internals/glimmer/tests/integration/application/engine-test.js

@@ -2,12 +2,16 @@ import { moduleFor, ApplicationTestCase } from 'internal-test-helpers';
 import Controller from '@ember/controller';
 import Service, { service } from '@ember/service';
 import { Helper, helper } from '@ember/-internals/glimmer';
+import { precompileTemplate } from '@ember/template-compilation';
 
 moduleFor(
   'Application Lifecycle - Helper Registration',
   class extends ApplicationTestCase {
     ['@test Unbound dashed helpers registered on the container can be late-invoked'](assert) {
-      this.addTemplate('application', `<div id='wrapper'>{{x-borf}} {{x-borf 'YES'}}</div>`);
+      this.add(
+        'template:application',
+        precompileTemplate(`<div id='wrapper'>{{x-borf}} {{x-borf 'YES'}}</div>`)
+      );
 
       let myHelper = helper((params) => params[0] || 'BORF');
       this.application.register('helper:x-borf', myHelper);
@@ -22,9 +26,9 @@ moduleFor(
     }
 
     ['@test Bound helpers registered on the container can be late-invoked'](assert) {
-      this.addTemplate(
-        'application',
-        `<div id='wrapper'>{{x-reverse}} {{x-reverse this.foo}}</div>`
+      this.add(
+        'template:application',
+        precompileTemplate(`<div id='wrapper'>{{x-reverse}} {{x-reverse this.foo}}</div>`)
       );
 
       this.add(
@@ -51,9 +55,9 @@ moduleFor(
     }
 
     ['@test Undashed helpers registered on the container can be invoked'](assert) {
-      this.addTemplate(
-        'application',
-        `<div id='wrapper'>{{omg}}|{{yorp 'boo'}}|{{yorp 'ya'}}</div>`
+      this.add(
+        'template:application',
+        precompileTemplate(`<div id='wrapper'>{{omg}}|{{yorp 'boo'}}|{{yorp 'ya'}}</div>`)
       );
 
       this.application.register(
@@ -76,7 +80,7 @@ moduleFor(
     }
 
     ['@test Helpers can receive injections'](assert) {
-      this.addTemplate('application', `<div id='wrapper'>{{full-name}}</div>`);
+      this.add('template:application', precompileTemplate(`<div id='wrapper'>{{full-name}}</div>`));
 
       let serviceCalled = false;