Skip to content

test: enable addons test to pass with debug build #8836

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
danbev wants to merge 4 commits into from
Closed

test: enable addons test to pass with debug build #8836

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
846da75
test: enable addons test to pass with debug build
danbev Sep 28, 2016
0c3ddf8
update addons.md to consider the build type
danbev Oct 1, 2016
79c407c
Revert "update addons.md to consider the build type"
danbev Oct 4, 2016
c991295
update addon-verify.js to consider build type
danbev Oct 4, 2016
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
Prev Previous commit
Next Next commit
Revert "update addons.md to consider the build type" 
This reverts commit 0c3ddf8.
  • Loading branch information
@danbev
danbev committed Oct 4, 2016
commit 79c407c1e956fccee3f4ca870a2b573d184d5d3b
27 changes: 9 additions & 18 deletions doc/api/addons.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -138,8 +138,7 @@ Once built, the binary Addon can be used from within Node.js by pointing

```js
// hello.js
const buildType = process.config.target_defaults.default_configuration;
const addon = require(`./build/${buildType}/addon`);
const addon = require('./build/Release/addon');

console.log(addon.hello()); // 'world'
```
Expand Down Expand Up @@ -317,8 +316,7 @@ Once compiled, the example Addon can be required and used from within Node.js:

```js
// test.js
const buildType = process.config.target_defaults.default_configuration;
const addon = require(`./build/${buildType}/addon`);
const addon = require('./build/Release/addon');

console.log('This should be eight:', addon.add(3, 5));
```
Expand Down Expand Up @@ -371,8 +369,7 @@ To test it, run the following JavaScript:

```js
// test.js
const buildType = process.config.target_defaults.default_configuration;
const addon = require(`./build/${buildType}/addon`);
const addon = require('./build/Release/addon');

addon((msg) => {
console.log(msg); // 'hello world'
Expand Down Expand Up @@ -422,8 +419,7 @@ To test it in JavaScript:

```js
// test.js
const buildType = process.config.target_defaults.default_configuration;
const addon = require(`./build/${buildType}/addon`);
const addon = require('./build/Release/addon');

var obj1 = addon('hello');
var obj2 = addon('world');
Expand Down Expand Up @@ -481,8 +477,7 @@ To test:

```js
// test.js
const buildType = process.config.target_defaults.default_configuration;
const addon = require(`./build/${buildType}/addon`);
const addon = require('./build/Release/addon');

var fn = addon();
console.log(fn()); // 'hello world'
Expand Down Expand Up @@ -644,8 +639,7 @@ Test it with:

```js
// test.js
const buildType = process.config.target_defaults.default_configuration;
const addon = require(`./build/${buildType}/addon`);
const addon = require('./build/Release/addon');

var obj = new addon.MyObject(10);
console.log(obj.plusOne()); // 11
Expand Down Expand Up @@ -837,8 +831,7 @@ Test it with:

```js
// test.js
const buildType = process.config.target_defaults.default_configuration;
const createObject = require(`./build/${buildType}/addon`);
const createObject = require('./build/Release/addon');

var obj = createObject(10);
console.log(obj.plusOne()); // 11
Expand Down Expand Up @@ -1014,8 +1007,7 @@ Test it with:

```js
// test.js
const buildType = process.config.target_defaults.default_configuration;
const addon = require(`./build/${buildType}/addon`);
const addon = require('./build/Release/addon');

var obj1 = addon.createObject(10);
var obj2 = addon.createObject(20);
Expand Down Expand Up @@ -1099,8 +1091,7 @@ Test in JavaScript by running:

```js
// test.js
const buildType = process.config.target_defaults.default_configuration;
const addon = require(`./build/${buildType}/addon`);
const addon = require('./build/Release/addon');
```

[bindings]: https://github.com/TooTallNate/node-bindings
Expand Down