Remove glimmer components from tests as telemetry helpers do not repo…

…rt glimmer components
ember-codemods · Feb 14, 2024 · 0d0518e · 0d0518e
1 parent 208d697
commit 0d0518e
Show file tree

Hide file tree

Showing 6 changed files with 18 additions and 67 deletions.
diff --git a/README.md b/README.md
@@ -193,21 +193,19 @@ If you would like to only convert certain component invocations to use the angle
 
 ### Making helper invocations unambiguous
 
-In order to make helper invocations unambiguous, use this:
+You may want to convert invocations like `{{concat "foo" "bar"}}` into `{{(concat "foo" "bar")}}`, which may be useful as a temporary step when upgrading to strict-mode Embroider.
 
-**config/anglebrackets-codemod-config.json**
+In your **config/anglebrackets-codemod-config.json**, add this:
 
 ```js
 {
   "unambiguousHelpers": true
 }
 ```
 
-This will result in invocations like `{{concat "foo" "bar"}}` to be converted into `{{(concat "foo" "bar")}}`, which may be useful in strict-mode Embroider.
-
-Note that it does not work in non-Embroider Ember, as of January 2024.
+Note that unambiguous helpers do not work in non-Embroider Ember, as of January 2024.
 
-Note that ambiguous invocations, that cannot be statically distinguished between a helper, a property and a component, will not be modified.
+Note that ambiguous invocations that cannot be statically distinguished between a helper, a property and a component — will not be modified.
 
 ## Debugging Workflow
 

diff --git a/test/fixtures/with-telemetry/input/app/templates/application.hbs b/test/fixtures/with-telemetry/input/app/templates/application.hbs
@@ -3,38 +3,19 @@
     <h3 class="sr-only">
       Components
     </h3>
-    <BsNav @type="pills" @stacked={{true}} as |nav|>
+    {{#bs-nav type="pills" stacked=true as |nav|}}
       {{#each this.model as |comp|}}
-        <nav.item>
-          <nav.link-to @route={{comp.demoRoute}}>
+        {{#nav.item}}
+          {{#nav.link-to route=comp.demoRoute}}
             {{comp.title}}
-          </nav.link-to>
-        </nav.item>
+          {{/nav.link-to}}
+        {{/nav.item}}
       {{/each}}
-    </BsNav>
+    {{/bs-nav}}
   </div>
   <div class="col-sm-8 col-sm-pull-4 col-md-9 col-md-pull-3">
-    {{utils/bee-bop}}
     {{-wat-wat}}
-    {{utils/-wat-wat}}
-    {{#if this.isDetailPage}}
-      <h1>
-        {{currentComponent.title}}
-      </h1>
-      <p class="lead">
-        {{currentComponent.description}}
-      </p>
-      {{api-reference component=this.currentComponent}}
-    {{/if}}
-    {{outlet}}
-
-    {{#bs-button id="openModal" onClick=(action "addModal")}}Open{{/bs-button}}
-
-    {{#if hasModal}}
-      {{#bs-modal-simple open=modal onHidden=(action "removeModal") title="Dynamic Dialog"}}
-        Hi there
-      {{/bs-modal-simple}}
-    {{/if}}
-    <FileLess @foo={{true}} />
+    {{outlet}}    
+    {{file-less foo=true}}
   </div>
 </div>
diff --git a/test/fixtures/with-telemetry/input/app/templates/components/file-less.hbs b/test/fixtures/with-telemetry/input/app/templates/components/file-less.hbs
@@ -1,2 +1 @@
 <div>this template has no js </div>
-{{#bs-button type="primary"}}Primary{{/bs-button}}
diff --git a/test/fixtures/with-telemetry/output/app/templates/application.hbs b/test/fixtures/with-telemetry/output/app/templates/application.hbs
@@ -14,27 +14,8 @@
     </BsNav>
   </div>
   <div class="col-sm-8 col-sm-pull-4 col-md-9 col-md-pull-3">
-    <Utils::BeeBop />
     {{-wat-wat}}
-    <Utils::-WatWat />
-    {{#if this.isDetailPage}}
-      <h1>
-        {{currentComponent.title}}
-      </h1>
-      <p class="lead">
-        {{currentComponent.description}}
-      </p>
-      <ApiReference @component={{this.currentComponent}} />
-    {{/if}}
-    {{outlet}}
-
-    <BsButton @id="openModal" @onClick={{action "addModal"}}>Open</BsButton>
-
-    {{#if hasModal}}
-      <BsModalSimple @open={{modal}} @onHidden={{action "removeModal"}} @title="Dynamic Dialog">
-        Hi there
-      </BsModalSimple>
-    {{/if}}
+    {{outlet}}    
     <FileLess @foo={{true}} />
   </div>
 </div>
diff --git a/test/fixtures/with-telemetry/output/app/templates/components/file-less.hbs b/test/fixtures/with-telemetry/output/app/templates/components/file-less.hbs
@@ -1,2 +1 @@
 <div>this template has no js </div>
-<BsButton @type="primary">Primary</BsButton>
diff --git a/transforms/angle-brackets/transform.test.js b/transforms/angle-brackets/transform.test.js
@@ -1382,24 +1382,17 @@ test('unknown helper with args', () => {
     "
         <ApiReference @component={{this.currentComponent}} />
         {{api-reference someArg}}
-        <ApiReference />
+        {{api-reference}}
       "
   `);
 });
 
-test('unambiguousHelpers: true', () => {
+test.only('unambiguousHelpers: true', () => {
   let input = `
-    {{concat}}
-    {{unknown}}
-    {{t "some.string" param="string" another=1}}
+  {{#bs-button id="openModal" onClick=(action "addModal")}}Open{{/bs-button}}
   `;
 
-  expect(runTest('unambiguousHelpers: true', input, { unambiguousHelpers: true }))
-    .toMatchInlineSnapshot(`
-    "
-        {{(concat)}}
-        {{unknown}}
-        {{(t \\"some.string\\" param=\\"string\\" another=1)}}
-      "
+  expect(runTest('unambiguousHelpers: true', input)).toMatchInlineSnapshot(`
+    ""
   `);
 });