Merge pull request #111 from ember-cli/fix-type-casting

Properly transpile `(emberImportedThing as any)()`

Author: Robert Jackson

__tests__/index-test.js

@@ -308,4 +308,46 @@ describe('when used with typescript', () => {
 
     expect(actual).toEqual(``);
   });
+
+  it('works when type casting', () => {
+    let source = `
+      import { addObserver } from '@ember/object/observers';
+      (addObserver as any)();
+   `;
+
+    let actual = transform7(source, [
+      '@babel/plugin-transform-typescript',
+      Plugin,
+    ]);
+
+    expect(actual).toEqual(`Ember.addObserver();`);
+  });
+
+  it('works for type assertions', () => {
+    let source = `
+      import { addObserver } from '@ember/object/observers';
+      <foo>addObserver();
+   `;
+
+    let actual = transform7(source, [
+      '@babel/plugin-transform-typescript',
+      Plugin,
+    ]);
+
+    expect(actual).toEqual(`Ember.addObserver();`);
+  });
+
+  it('works for non-null expression', () => {
+    let source = `
+      import { addObserver } from '@ember/object/observers';
+      addObserver!();
+   `;
+
+    let actual = transform7(source, [
+      '@babel/plugin-transform-typescript',
+      Plugin,
+    ]);
+
+    expect(actual).toEqual(`Ember.addObserver();`);
+  });
 });

src/index.js

@@ -16,7 +16,14 @@ function isBlacklisted(blacklist, importPath, exportName) {
 module.exports = function (babel) {
   const t = babel.types;
 
-  const isTypescriptNode = (node) => node.type.startsWith('TS');
+  const TSTypesRequiringModification = [
+    'TSAsExpression',
+    'TSTypeAssertion',
+    'TSNonNullExpression',
+  ];
+  const isTypescriptNode = (node) =>
+    node.type.startsWith('TS') &&
+    !TSTypesRequiringModification.includes(node.type);
 
   const GLOBALS_MAP = new Map();