Add asserts checking that async ops return null (#2463)

1 files changed, 4 insertions, 3 deletions

diff --git a/js/dispatch.ts b/js/dispatch.ts
index 64bdfda17..0c6e70709 100644
--- a/js/dispatch.ts
+++ b/js/dispatch.ts
@@ -91,7 +91,7 @@ export function sendAsync(
     data,
     false
   );
-  util.assert(response == null);
+  util.assert(response == null); // null indicates async.
   const promise = util.createResolvable<msg.Base>();
   promiseTable.set(cmdId, promise);
   return promise;
@@ -106,10 +106,11 @@ export function sendSync(
 ): null | msg.Base {
   const [cmdId, response] = sendInternal(builder, innerType, inner, data, true);
   util.assert(cmdId >= 0);
-  if (response == null || response.length === 0) {
+  util.assert(response != null); // null indicates async.
+  if (response!.length === 0) {
     return null;
   } else {
-    const bb = new flatbuffers.ByteBuffer(response);
+    const bb = new flatbuffers.ByteBuffer(response!);
     const baseRes = msg.Base.getRootAsBase(bb);
     errors.maybeThrowError(baseRes);
     return baseRes;