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

test/checks-test.js: Avoid function definitions in loops

Browse source 
Fixes the following JSHint errors:

  ./test/checks-test.js: line 26, col 8, Don't make functions within a loop.
  ./test/checks-test.js: line 31, col 5, Don't make functions within a loop.
  ./test/checks-test.js: line 59, col 8, Don't make functions within a loop.
  ./test/checks-test.js: line 64, col 5, Don't make functions within a loop.
This commit is contained in:
David Majda 2011-09-14 12:45:41 +02:00
parent 96192417d8
commit 0fe5769024
1 changed files with 28 additions and 24 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

52
test/checks-test.js
View file

@ -3,6 +3,19 @@
module("PEG.compiler.checks"); module("PEG.compiler.checks");
test("reports missing referenced rules", function() { test("reports missing referenced rules", function() {
function testGrammar(grammar) {
raises(
function() {
var ast = PEG.parser.parse(grammar);
PEG.compiler.checks.missingReferencedRules(ast);
},
function(e) {
return e instanceof PEG.GrammarError
&& e.message === "Referenced rule \"missing\" does not exist.";
}
);
}
var grammars = [ var grammars = [
'start = missing', 'start = missing',
'start = missing / "a" / "b"', 'start = missing / "a" / "b"',
@ -18,21 +31,23 @@ test("reports missing referenced rules", function() {
'start = missing { }' 'start = missing { }'
]; ];
for (var i = 0; i < grammars.length; i++) { for (var i = 0; i < grammars.length; i++) { testGrammar(grammars[i]); }
raises(
function() {
var ast = PEG.parser.parse(grammars[i]);
PEG.compiler.checks.missingReferencedRules(ast);
},
function(e) {
return e instanceof PEG.GrammarError
&& e.message === "Referenced rule \"missing\" does not exist.";
}
);
}
}); });
test("reports left recursion", function() { test("reports left recursion", function() {
function testGrammar(grammar) {
raises(
function() {
var ast = PEG.parser.parse(grammar);
PEG.compiler.checks.leftRecursion(ast);
},
function(e) {
return e instanceof PEG.GrammarError
&& e.message === "Left recursion detected for rule \"start\".";
}
);
}
var grammars = [ var grammars = [
/* Direct */ /* Direct */
'start = start', 'start = start',
@ -51,18 +66,7 @@ test("reports left recursion", function() {
'start = stop; stop = start' 'start = stop; stop = start'
]; ];
for (var i = 0; i < grammars.length; i++) { for (var i = 0; i < grammars.length; i++) { testGrammar(grammars[i]); }
raises(
function() {
var ast = PEG.parser.parse(grammars[i]);
PEG.compiler.checks.leftRecursion(ast);
},
function(e) {
return e instanceof PEG.GrammarError
&& e.message === "Left recursion detected for rule \"start\".";
}
);
}
}); });
})(); })();