fix: correctly handle a variable passed to the t macro (#2048)

packages/babel-plugin-lingui-macro/src/macroJs.ts

@@ -165,12 +165,13 @@ export class MacroJs {
     ctx: MacroJsContext,
     linguiInstance?: babelTypes.Expression
   ): babelTypes.CallExpression => {
-    const descriptor = processDescriptor(
-      node.arguments[0] as ObjectExpression,
-      ctx
-    )
+    let arg: Expression = node.arguments[0] as Expression
+
+    if (t.isObjectExpression(arg)) {
+      arg = processDescriptor(arg, ctx)
+    }
 
-    return this.createI18nCall(descriptor, linguiInstance)
+    return this.createI18nCall(arg, linguiInstance)
   }
 
   /**
@@ -274,7 +275,7 @@ export class MacroJs {
         // t(messageDescriptor)
         if (
           currentPath.isCallExpression() &&
-          currentPath.get("arguments")[0].isObjectExpression()
+          currentPath.get("arguments")[0]?.isObjectExpression()
         ) {
           let descriptor = processDescriptor(
             (currentPath.get("arguments")[0] as NodePath<ObjectExpression>)
@@ -302,15 +303,15 @@ export class MacroJs {
   }
 
   private createI18nCall(
-    messageDescriptor: ObjectExpression,
+    messageDescriptor: Expression | undefined,
     linguiInstance?: Expression
   ) {
     return t.callExpression(
       t.memberExpression(
         linguiInstance ?? t.identifier(this.i18nImportName),
         t.identifier("_")
       ),
-      [messageDescriptor]
+      messageDescriptor ? [messageDescriptor] : []
     )
   }
 }

packages/babel-plugin-lingui-macro/test/__snapshots__/js-t.test.ts.snap

@@ -271,6 +271,17 @@ _i18n._(
 
 `;
 
+exports[`Should not crash when a variable passed 1`] = `
+import { t } from "@lingui/core/macro";
+const msg = t(msg);
+
+↓ ↓ ↓ ↓ ↓ ↓
+
+import { i18n as _i18n } from "@lingui/core";
+const msg = _i18n._(msg);
+
+`;
+
 exports[`Support id and comment in t macro as callExpression 1`] = `
 import { t, plural } from "@lingui/core/macro";
 const msg = t({
@@ -548,6 +559,17 @@ _i18n._(
 
 `;
 
+exports[`should not crash when no params passed 1`] = `
+import { t } from "@lingui/core/macro";
+const msg = t();
+
+↓ ↓ ↓ ↓ ↓ ↓
+
+import { i18n as _i18n } from "@lingui/core";
+const msg = _i18n._();
+
+`;
+
 exports[`t\`\` macro could be renamed 1`] = `
 import { t as t2 } from "@lingui/core/macro";
 const a = t2\`Expression assignment\`;

packages/babel-plugin-lingui-macro/test/__snapshots__/js-useLingui.test.ts.snap

@@ -1,5 +1,22 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[`does not crash when no params 1`] = `
+import { useLingui } from "@lingui/react/macro";
+function MyComponent() {
+  const { t } = useLingui();
+  const a = t();
+}
+
+↓ ↓ ↓ ↓ ↓ ↓
+
+import { useLingui as _useLingui } from "@lingui/react";
+function MyComponent() {
+  const { _: _t } = _useLingui();
+  const a = _t();
+}
+
+`;
+
 exports[`inserted statement should not clash with existing variables 1`] = `
 import { useLingui } from "@lingui/react/macro";
 function MyComponent() {
@@ -85,6 +102,23 @@ function MyComponent() {
 
 `;
 
+exports[`support a variable 1`] = `
+import { useLingui } from "@lingui/react/macro";
+function MyComponent() {
+  const { t } = useLingui();
+  const a = t(msg);
+}
+
+↓ ↓ ↓ ↓ ↓ ↓
+
+import { useLingui as _useLingui } from "@lingui/react";
+function MyComponent() {
+  const { _: _t } = _useLingui();
+  const a = _t(msg);
+}
+
+`;
+
 exports[`support configuring runtime module import using LinguiConfig.runtimeConfigModule 1`] = `
 import { useLingui } from "@lingui/react/macro";
 function MyComponent() {

packages/babel-plugin-lingui-macro/test/js-t.test.ts

@@ -142,6 +142,20 @@ macroTester({
         const msg = t({ id: \`msgId\` })
       `,
     },
+    {
+      name: "Should not crash when a variable passed",
+      code: `
+        import { t } from '@lingui/core/macro'
+        const msg = t(msg)
+      `,
+    },
+    {
+      name: "should not crash when no params passed",
+      code: `
+        import { t } from '@lingui/core/macro'
+        const msg = t()
+      `,
+    },
     {
       name: "Production - only essential props are kept",
       production: true,

packages/babel-plugin-lingui-macro/test/js-useLingui.test.ts

@@ -86,6 +86,28 @@ import { useLingui } from '@lingui/react/macro';
 function MyComponent() {
   const { t } = useLingui();
   const a = t({ message: "Hello", context: "my custom" });
+}
+    `,
+    },
+    {
+      name: "support a variable",
+      code: `
+import { useLingui } from '@lingui/react/macro';
+
+function MyComponent() {
+  const { t } = useLingui();
+  const a = t(msg);
+}
+    `,
+    },
+    {
+      name: "does not crash when no params",
+      code: `
+import { useLingui } from '@lingui/react/macro';
+
+function MyComponent() {
+  const { t } = useLingui();
+  const a = t();
 }
     `,
     },

packages/babel-plugin-lingui-macro/test/jsx-plural.test.ts

@@ -1,5 +1,7 @@
 import { macroTester } from "./macroTester"
 
+describe.skip("", () => {})
+
 macroTester({
   cases: [
     {