Merge pull request #1723 from murgatroid99/node_interop_flake

Fixed ordering assumptions in server_streaming interop test
pull/1730/merge
Craig Tiller 10 years ago
commit a152b1cf39
  1. 6
      src/node/interop/interop_client.js
  2. 4
      src/node/src/client.js

@ -154,13 +154,15 @@ function serverStreaming(client, done) {
arg.response_parameters[resp_index].size); arg.response_parameters[resp_index].size);
resp_index += 1; resp_index += 1;
}); });
call.on('status', function(status) { call.on('end', function() {
assert.strictEqual(status.code, grpc.status.OK);
assert.strictEqual(resp_index, 4); assert.strictEqual(resp_index, 4);
if (done) { if (done) {
done(); done();
} }
}); });
call.on('status', function(status) {
assert.strictEqual(status.code, grpc.status.OK);
});
} }
/** /**

@ -125,10 +125,6 @@ function _read(size) {
self.finished = true; self.finished = true;
return; return;
} }
if (self.finished) {
self.push(null);
return;
}
var data = event.read; var data = event.read;
if (self.push(self.deserialize(data)) && data !== null) { if (self.push(self.deserialize(data)) && data !== null) {
var read_batch = {}; var read_batch = {};

Loading…
Cancel
Save