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

Label scope specs: Add negative specs (subexpressions)

Browse Source 
Semantic predicate and action specs which verified label scope didn't
exercise labels in subexpressions. This commit adds cases exercising
them, including few commented-out cases which reveal #396 (these will be
uncommented when the bug gets fixed).

Note that added specs exercise all relevant expression types. This is
needed because code that makes subexpression labels invisible to the
outside is separate for each expression type so one generic test
wouldn't generate enough coverage.

Part of a fix of #396.
redux
David Majda 8 years ago
parent ffd90a8c9e
commit 58806a3d77
1 changed files with 177 additions and 0 deletions
Show Stats Download Patch File Download Diff File

177
spec/behavior/generated-parser-behavior.spec.js
Unescape Escape View File

@ -463,6 +463,65 @@ describe("generated parser behavior", function() {
expect(parser).toFailToParse("a");
});
it("cannot access variables defined by subexpressions", function() {
var testcases = [
// TODO: Fix #396.
//
// {
// grammar: 'start = (a:"a") &{ return a === "a"; }',
// input: "a"
// },
{
grammar: 'start = (a:"a")? &{ return a === "a"; }',
input: "a"
},
{
grammar: 'start = (a:"a")* &{ return a === "a"; }',
input: "a"
},
{
grammar: 'start = (a:"a")+ &{ return a === "a"; }',
input: "a"
},
{
grammar: 'start = $(a:"a") &{ return a === "a"; }',
input: "a"
},
{
grammar: 'start = &(a:"a") "a" &{ return a === "a"; }',
input: "a"
},
{
grammar: 'start = !(a:"a") "b" &{ return a === "a"; }',
input: "b"
},
{
grammar: 'start = b:(a:"a") &{ return a === "a"; }',
input: "a"
},
// TODO: Fix #396.
//
// {
// grammar: 'start = ("a" b:"b" "c") &{ return b === "b"; }',
// input: "abc"
// },
{
grammar: 'start = (a:"a" { return a; }) &{ return a === "a"; }',
input: "a"
},
{
grammar: 'start = ("a" / b:"b" / "c") &{ return b === "b"; }',
input: "b"
}
],
parser, i;
for (i = 0; i < testcases.length; i++) {
parser = PEG.buildParser(testcases[i].grammar, options);
expect(parser).toFailToParse(testcases[i].input);
}
});
});
describe("in outer sequence", function() {
@ -608,6 +667,65 @@ describe("generated parser behavior", function() {
expect(parser).toFailToParse("a");
});
it("cannot access variables defined by subexpressions", function() {
var testcases = [
// TODO: Fix #396.
//
// {
// grammar: 'start = (a:"a") !{ return a !== "a"; }',
// input: "a"
// },
{
grammar: 'start = (a:"a")? !{ return a !== "a"; }',
input: "a"
},
{
grammar: 'start = (a:"a")* !{ return a !== "a"; }',
input: "a"
},
{
grammar: 'start = (a:"a")+ !{ return a !== "a"; }',
input: "a"
},
{
grammar: 'start = $(a:"a") !{ return a !== "a"; }',
input: "a"
},
{
grammar: 'start = &(a:"a") "a" !{ return a !== "a"; }',
input: "a"
},
{
grammar: 'start = !(a:"a") "b" !{ return a !== "a"; }',
input: "b"
},
{
grammar: 'start = b:(a:"a") !{ return a !== "a"; }',
input: "a"
},
// TODO: Fix #396.
//
// {
// grammar: 'start = ("a" b:"b" "c") !{ return b !== "b"; }',
// input: "abc"
// },
{
grammar: 'start = (a:"a" { return a; }) !{ return a !== "a"; }',
input: "a"
},
{
grammar: 'start = ("a" / b:"b" / "c") !{ return b !== "b"; }',
input: "b"
}
],
parser, i;
for (i = 0; i < testcases.length; i++) {
parser = PEG.buildParser(testcases[i].grammar, options);
expect(parser).toFailToParse(testcases[i].input);
}
});
});
describe("in outer sequence", function() {
@ -931,6 +1049,65 @@ describe("generated parser behavior", function() {
expect(parser).toParse("abc", ["a", "b", "c"]);
});
it("cannot access variables defined by subexpressions", function() {
var testcases = [
// TODO: Fix #396.
//
// {
// grammar: 'start = (a:"a") { return a; }',
// input: "a"
// },
{
grammar: 'start = (a:"a")? { return a; }',
input: "a"
},
{
grammar: 'start = (a:"a")* { return a; }',
input: "a"
},
{
grammar: 'start = (a:"a")+ { return a; }',
input: "a"
},
{
grammar: 'start = $(a:"a") { return a; }',
input: "a"
},
{
grammar: 'start = &(a:"a") "a" { return a; }',
input: "a"
},
{
grammar: 'start = !(a:"a") "b" { return a; }',
input: "b"
},
{
grammar: 'start = b:(a:"a") { return a; }',
input: "a"
},
// TODO: Fix #396.
//
// {
// grammar: 'start = ("a" b:"b" "c") { return b; }',
// input: "abc"
// },
{
grammar: 'start = (a:"a" { return a; }) { return a; }',
input: "a"
},
{
grammar: 'start = ("a" / b:"b" / "c") { return b; }',
input: "b"
}
],
parser, i;
for (i = 0; i < testcases.length; i++) {
parser = PEG.buildParser(testcases[i].grammar, options);
expect(parser).toFailToParse(testcases[i].input);
}
});
});
describe("in outer sequence", function() {

Write Preview
Loading…
Cancel
Save