Skip to content
Closed
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
test: add coverage for client._addHandle()
`Client.prototype._addHandle()` in the `_debugger` module has conditions
around invalid properties that are not currently tested. This change
adds some minimal unit tests.
  • Loading branch information
Trott committed Sep 13, 2016
commit 29470f8fffa9744eb2cd5960460c70ca5407eed5
44 changes: 44 additions & 0 deletions test/parallel/test-debugger-client-addhandle.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,44 @@
'use strict';

require('../common');
const assert = require('assert');
const Client = require('_debugger').Client;

{
const client = new Client();
assert.deepStrictEqual(client.handles, {});
}

{
const client = new Client();
client._addHandle(null);
assert.deepStrictEqual(client.handles, {});
}

{
const client = new Client();
client._addHandle('not an object');
assert.deepStrictEqual(client.handles, {});
}

{
const client = new Client();
client._addHandle({handle: 'not a number'});
assert.deepStrictEqual(client.handles, {});
}

{
const client = new Client();
const validNoScript = { handle: 6, id: 'foo', type: 'not a script' };
client._addHandle(validNoScript);
assert.deepStrictEqual(client.handles, { 6: validNoScript });
assert.deepStrictEqual(client.scripts, {});
}

{
const client = new Client();
const validWithScript = { handle: 5, id: 'bar', type: 'script' };
client._addHandle(validWithScript);
assert.deepStrictEqual(client.handles, { 5: validWithScript });
assert.deepStrictEqual(client.scripts, { bar: validWithScript });
}