fix wallstreet integration with telemetry data (#205)

Author: hmajoros

transforms/angle-brackets/telemetry/mock-invokables.js

@@ -92,4 +92,13 @@ module.exports = {
   'app/components/foo-bar': {
     type: 'Component',
   },
+  'app/components/baz-bang/foo-bar/bang': {
+    type: 'Component',
+  },
+  'app/helpers/helper-1': {
+    type: 'Helper',
+  },
+  'app/helpers/nested/helper': {
+    type: 'Helper',
+  },
 };

transforms/angle-brackets/transform.js

@@ -270,9 +270,19 @@ function getNonDataAttributesFromParams(params) {
   return params.filter(p => !(p.original && `${p.original}`.startsWith('data-')));
 }
 
-function shouldIgnoreMustacheStatement(name, config, invokableData) {
+function shouldIgnoreMustacheStatement(fullName, config, invokableData) {
   let { helpers, components } = invokableData;
   let isTelemetryData = !!(helpers || components);
+
+  let name = fullName;
+  // replace `::` with `/`, and ignore the path before $
+  if (isWallStreet(name)) {
+    name = name
+      .split('$')
+      .pop()
+      .replace('::', '/');
+  }
+
   if (isTelemetryData) {
     let mergedHelpers = [...KNOWN_HELPERS, ...(helpers || [])];
     let isHelper = mergedHelpers.includes(name) || config.helpers.includes(name);
@@ -405,9 +415,6 @@ function transformToAngleBracket(fileInfo, config, invokableData) {
       ) {
         return transformNode(node, fileInfo, config);
       }
-      if (isWallStreet(tagName)) {
-        return transformNode(node, fileInfo, config);
-      }
     },
     BlockStatement(node) {
       let tagName = `${node.path.original}`;

transforms/angle-brackets/transform.test.js

@@ -1049,12 +1049,12 @@ test('hyphens with nested usage', () => {
 
 test('wallstreet', () => {
   let input = `
-    {{#foo-bar$baz-bang/foo-bar::bang}}
+    {{#foo-bar$baz-bang/foo-bar/bang}}
       <div ...attributes>
         {{foo bar="baz"}}
       </div>
-    {{/foo-bar$baz-bang/foo-bar::bang}}
-    {{foo-bar$baz-bang/foo-bar::bang}}
+    {{/foo-bar$baz-bang/foo-bar/bang}}
+    {{foo-bar$baz-bang/foo-bar/bang}}
   `;
 
   expect(runTest('wallstreet.hbs', input)).toMatchInlineSnapshot(`
@@ -1069,6 +1069,26 @@ test('wallstreet', () => {
   `);
 });
 
+test('wallstreet-telemetry', () => {
+  let input = `
+    {{nested$helper}}
+    {{nested::helper}}
+    {{nested$helper param="cool!"}}
+    {{nested::helper param="yeah!"}}
+    {{helper-1}}
+  `;
+
+  expect(runTest('wallstreet-telemetry.hbs', input)).toMatchInlineSnapshot(`
+    "
+        {{nested$helper}}
+        {{nested::helper}}
+        {{nested$helper param=\\"cool!\\"}}
+        {{nested::helper param=\\"yeah!\\"}}
+        {{helper-1}}
+      "
+  `);
+});
+
 test('attr-space', () => {
   let input = `
     <MyComp::Test @color={{"custom-2"}} @visible={{Group.item.isNew}} />