fix: use ':' instead of '=' for esbuild 'define' argument (#2469)

Author: mattsoulanille

packages/esbuild/esbuild.bzl

@@ -49,7 +49,7 @@ def _esbuild_impl(ctx):
     args.add_joined(["--target", ctx.attr.target], join_with = "=")
     args.add_joined(["--log-level", "info"], join_with = "=")
     args.add_joined(["--metafile", metafile.path], join_with = "=")
-    args.add_all(ctx.attr.define, format_each = "--define=%s")
+    args.add_all(ctx.attr.define, format_each = "--define:%s")
     args.add_all(ctx.attr.external, format_each = "--external=%s")
 
     # disable the error limit and show all errors

packages/esbuild/test/define/BUILD.bazel

@@ -0,0 +1,28 @@
+load("//packages/esbuild/test:tests.bzl", "esbuild")
+load("//packages/jasmine:index.bzl", "jasmine_node_test")
+load("//packages/typescript:index.bzl", "ts_library")
+
+ts_library(
+    name = "main",
+    srcs = [
+        "main.ts",
+    ],
+    deps = [
+        "@npm//@types/node",
+    ],
+)
+
+esbuild(
+    name = "bundle",
+    define = [
+        "process.env.NODE_ENV=\"defined_in_bundle\"",
+    ],
+    entry_point = "main.ts",
+    deps = [":main"],
+)
+
+jasmine_node_test(
+    name = "bundle_test",
+    srcs = ["bundle_test.js"],
+    data = [":bundle"],
+)

packages/esbuild/test/define/bundle_test.js

@@ -0,0 +1,11 @@
+const {readFileSync} = require('fs');
+
+const helper = require(process.env.BAZEL_NODE_RUNFILES_HELPER);
+const location = helper.resolve('build_bazel_rules_nodejs/packages/esbuild/test/define/bundle.js');
+
+describe('esbuild define', () => {
+  it('defines variables', () => {
+    const bundle = readFileSync(location, {encoding: 'utf8'});
+    expect(bundle).toContain(`nodeEnv = "defined_in_bundle"`);
+  });
+})

packages/esbuild/test/define/main.ts

@@ -0,0 +1 @@
+export const nodeEnv = process.env.NODE_ENV;