Skip to content

Commit 5bde084

Browse files
committed
Wrap test cases in parentheses
This lets people use test cases like: 10 + something(20) // => 30
1 parent 469f13a commit 5bde084

File tree

4 files changed

+4
-3
lines changed

4 files changed

+4
-3
lines changed

examples/add.js

+1
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
* add(10, 20)
1010
* // => 30
1111
* add(10, 50) // => 60
12+
* 10 + add(10, 50) // => 70
1213
*/
1314

1415
function add(x, y) {

lib/index.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ exports.toJsdocRegister = function jsdoctest$toJsdocRegister(comment) {
9393
var compiled = _.map(comment.examples, function(example) {
9494
var id = baseId + example.testCase + ' => ' + example.expectedResult;
9595
var fn = 'function() {' +
96-
example.testCase + '.should.equal(' + example.expectedResult + ');' +
96+
'(' + example.testCase + ').should.equal(' + example.expectedResult + ');' +
9797
'}';
9898
return '__registerTest(\'' + id + '\', ' + fn + ');';
9999
}).join('');

lib/mocha.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ exports.commentToMochaSpec = function commentToMochaSpec(comment) {
4848
return '\ndescribe(\'' + ctx.string + '\', function() {' +
4949
_.map(comment.examples, function(example) {
5050
return 'it(\'' + example.testCase + '\', function() {' +
51-
example.testCase +
51+
'(' + example.testCase + ')' +
5252
'.should.eql(' + example.expectedResult + ');' +
5353
'});';
5454
}).join('\n') +

test/mocha.test.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ describe('jsdoctest/mocha', function() {
1717
content.should.containEql(
1818
'\ndescribe(\'add()\', function() {' +
1919
'it(\'add(1, 2)\', function() {' +
20-
'add(1, 2).should.eql(3);' +
20+
'(add(1, 2)).should.eql(3);' +
2121
'});' +
2222
'});'
2323
);

0 commit comments

Comments
 (0)