joepie91/pegjs
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Merge pull request #541 from Mingun/correct-options-in-tests

Browse source 
Auto-add `allowedStartRules` to test options for `changeAST` method if it not defined
This commit is contained in:
Futago-za Ryuu 2017-12-18 21:10:32 +00:00 committed by GitHub
parent 75d655409c 3e89534bb7
commit a02f8091e3
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 21 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
lib/compiler/passes/report-undefined-rules.js
View file

@ -24,19 +24,15 @@ function reportUndefinedRules( ast, options ) {
check( ast );
if ( options.allowedStartRules ) {
options.allowedStartRules.forEach( rule => {
options.allowedStartRules.forEach( rule => {
if ( ! asts.findRule( ast, rule ) ) {
if ( ! asts.findRule( ast, rule ) ) {
throw new GrammarError( `Start rule "${ rule }" is not defined.` );
throw new GrammarError( `Start rule "${ rule }" is not defined.` );
}
}
} );
}
} );
}

16
test/spec/unit/compiler/passes/helpers.js
View file

@ -55,6 +55,14 @@ module.exports = function ( chai, utils ) {
const ast = parser.parse( grammar );
if ( ! options.allowedStartRules ) {
options.allowedStartRules = ast.rules.length > 0
? [ ast.rules[ 0 ].name ]
: [];
}
utils.flag( this, "object" )( ast, options );
this.assert(
@ -73,6 +81,14 @@ module.exports = function ( chai, utils ) {
const ast = parser.parse( grammar );
if ( ! options.allowedStartRules ) {
options.allowedStartRules = ast.rules.length > 0
? [ ast.rules[ 0 ].name ]
: [];
}
let passed, result;
try {