- Incorporates #69

Author: Dig-Doug

src/change_import_style.ts

@@ -19,8 +19,8 @@ function main() {
   const umdContents = convertToUmd(args, initialContents);
   fs.writeFileSync(args.output_umd_path, umdContents, 'utf8');
 
-  const commonJsContents = convertToESM(args, initialContents);
-  fs.writeFileSync(args.output_es6_path, commonJsContents, 'utf8');
+  const esmContents = convertToESM(args, initialContents);
+  fs.writeFileSync(args.output_es6_path, esmContents, 'utf8');
 }
 
 function replaceRecursiveFilePaths(args: any) {
@@ -88,12 +88,18 @@ function convertToESM(args: any, initialContents: string): string {
   };
 
   const replaceRequiresWithSubpackageImports = (contents: string) => {
-    return contents.replace(/var ([\w\d_]+) = require\((['"][\w\d@/_-]+['"])\)\.([\w\d_]+);/g, 'import * as $1 from $2;')
-  }
+    // Example:
+    //   Changes: var foo = require("@improbable-eng/grpc-web").bar;
+    //   To:      import {bar as foo} from "@improbable-eng/grpc-web";
+    return contents.replace(
+        /var ([\w\d_]+) = require\((['"][\w\d@/_-]+['"])\)\.([\w\d_]+);/g,
+        (_, varName, moduleIdentifier, propertyName) =>
+            `import { ${propertyName} as ${varName} } from ${moduleIdentifier};`);
+  };
 
   const replaceCJSExportsWithECMAExports = (contents: string) => {
     return contents.replace(/exports\.([\w\d_]+) = .*;/g, 'export { $1 };')
-  }
+  };
 
   const transformations: ((c: string) => string)[] = [
     replaceRecursiveFilePaths(args),

test/BUILD.bazel

@@ -106,13 +106,13 @@ rollup_bundle(
     entry_points = {
         ":test_bundling.ts": "index",
     },
+    format = "cjs",
     output_dir = True,
     deps = [
         ":test_bundling_lib",
         "@npm//@rollup/plugin-commonjs",
         "@npm//@rollup/plugin-node-resolve",
     ],
-    format = "cjs",
 )
 
 ts_library(
@@ -121,20 +121,49 @@ ts_library(
     tsconfig = ":test_bundling_tsconfig.json",
     deps = [
         "//test/proto:naming_styles_ts_proto",
-        "//test/proto/common:pizza_ts_proto",
         "//test/proto:pizza_service_ts_proto",
         "//test/proto/common:delivery_person_ts_proto",
-        "@npm//google-protobuf",
+        "//test/proto/common:pizza_ts_proto",
         "@npm//@improbable-eng/grpc-web",
+        "@npm//google-protobuf",
     ],
 )
 
 jasmine_node_test(
     name = "rollup_test",
     srcs = [
-        ":rollup_test.spec.js"
+        ":rollup_test.spec.js",
     ],
     data = [
-        ":test_es6_bundling"
-    ]
+        ":test_es6_bundling",
+    ],
+)
+
+ts_library(
+    name = "rollup_test_spec",
+    testonly = True,
+    srcs = ["rollup_test.spec.ts"],
+    tsconfig = ":tsconfig.json",
+    deps = ["@npm//@types/jasmine"],
+)
+
+karma_web_test_suite(
+    name = "rollup_browser_test_suite",
+    srcs = [
+        "require.config.js",
+    ],
+    browsers = [
+        "@io_bazel_rules_webtesting//browsers:chromium-local",
+    ],
+    # Files are served under /base/<WORKSPACE_NAME>/<path>
+    static_files = [
+        "@npm//:node_modules/@improbable-eng/grpc-web/dist/grpc-web-client.umd.js",
+        "@npm//:node_modules/browser-headers/dist/browser-headers.umd.js",
+        ":test_es6_bundling",
+    ],
+    tags = ["native"],
+    deps = [
+        ":rollup_test_spec",
+        ":test_es6_bundling",
+    ],
 )

test/integration/client/rollup.config.js

@@ -7,6 +7,11 @@ module.exports = {
       mainFields: ['browser', 'es2015', 'module', 'jsnext:main', 'main'],
     }),
     commonjs({
+      namedExports: {
+        'node_modules/@improbable-eng/grpc-web/dist/grpc-web-client.umd.js': [
+          'grpc',
+        ],
+      },
       extensions: ['.js', '.mjs'],
     }),
   ],

test/pizza_service_proto_test.spec.ts

@@ -1,6 +1,6 @@
 import {Pizza, PizzaSize} from 'rules_typescript_proto/test/proto/common/pizza_pb';
 import {OrderPizzaRequest, OrderPizzaResponse} from 'rules_typescript_proto/test/proto/pizza_service_pb';
-import {PizzaService} from 'rules_typescript_proto/test/proto/pizza_service_pb_service';
+import {PizzaService, PizzaServiceClient} from 'rules_typescript_proto/test/proto/pizza_service_pb_service';
 
 declare function require(module: string): any;
 
@@ -14,6 +14,12 @@ describe('DeliveryPerson', () => {
     expect(PizzaService).toBeDefined();
   });
 
+  it('PizzaServiceClient.orderPizza should return a UnaryResponse', () => {
+    const client = new PizzaServiceClient('http://localhost', {});
+    const response = client.orderPizza(new OrderPizzaRequest(), (_) => {});
+    expect(typeof response.cancel).toBe('function');
+  });
+
   it('Generated code seems to work', () => {
     const request = new OrderPizzaRequest();
 

test/require.config.js

@@ -2,6 +2,7 @@ require.config({
   paths: {
     '@improbable-eng/grpc-web':
         'base/npm/node_modules/@improbable-eng/grpc-web/dist/grpc-web-client.umd',
-    'browser-headers': 'base/npm/node_modules/browser-headers/dist/browser-headers.umd'
+    'browser-headers': 'base/npm/node_modules/browser-headers/dist/browser-headers.umd',
+    'test_es6_bundling': 'base/rules_typescript_proto/test/test_es6_bundling/index',
   }
 });

test/rollup.config.js

@@ -7,10 +7,10 @@ module.exports = {
     commonjs({
       // Temporary fix until https://github.com/improbable-eng/grpc-web/issues/369 is resolved.
       namedExports: {
-        './node_modules/@improbable-eng/grpc-web/dist/grpc-web-client.js': [
+        'node_modules/@improbable-eng/grpc-web/dist/grpc-web-client.umd.js': [
           'grpc',
         ],
-      }
+      },
     }),
   ],
 };