Merge pull request #2958 from murgatroid99/node_deadline_internal

Made deadline tests accept INTERNAL status
pull/2986/head
Tim Emiola 9 years ago
commit 0137ae3561
  1. 4
      src/node/interop/interop_client.js
  2. 6
      src/node/test/surface_test.js

@ -264,7 +264,9 @@ function timeoutOnSleepingServer(client, done) {
payload: {body: zeroBuffer(27182)}
});
call.on('error', function(error) {
assert.strictEqual(error.code, grpc.status.DEADLINE_EXCEEDED);
assert(error.code === grpc.status.DEADLINE_EXCEEDED ||
error.code === grpc.status.INTERNAL);
done();
});
}

@ -784,7 +784,8 @@ describe('Other conditions', function() {
client.clientStream(function(err, value) {
try {
assert(err);
assert.strictEqual(err.code, grpc.status.DEADLINE_EXCEEDED);
assert(err.code === grpc.status.DEADLINE_EXCEEDED ||
err.code === grpc.status.INTERNAL);
} finally {
callback(err, value);
done();
@ -809,7 +810,8 @@ describe('Other conditions', function() {
null, {parent: parent, propagate_flags: deadline_flags});
child.on('error', function(err) {
assert(err);
assert.strictEqual(err.code, grpc.status.DEADLINE_EXCEEDED);
assert(err.code === grpc.status.DEADLINE_EXCEEDED ||
err.code === grpc.status.INTERNAL);
done();
});
};

Loading…
Cancel
Save