Merge pull request #8 from Mouradif/master

Added support for empty arrays in blocks annotations
glayzzle · Feb 16, 2019 · dacf670 · dacf670
2 parents 6c97351 + 1b2be8e
commit dacf670
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/src/parser.js b/src/parser.js
@@ -644,6 +644,10 @@ Parser.prototype.parseStatement = function () {
 Parser.prototype.readArray = function (endChar) {
   var result = [];
   this.token = this.lexer.lex(); // consume start char
+  if (this.token === endChar) {
+    this.token = this.lexer.lex();
+    return result;
+  }
   do {
     var item = this.parseTopStatement();
     if (item !== null) { // ignore

diff --git a/test/parser.js b/test/parser.js
@@ -332,4 +332,18 @@ describe('Test parser', function () {
     ast.body[0].kind.should.be.exactly('annotation');
     ast.body[0].arguments.length.should.be.exactly(0);
   });
+
+  it('test empty array annotation', function () {
+    var ast = doc.parse([
+      '/**',
+      ' * @SomeArray[]',
+      ' * @SomeOtherAnnotation',
+      ' */'
+    ].join('\n'));
+    ast.body.length.should.be.exactly(2);
+    ast.body[0].kind.should.be.exactly('block');
+    ast.body[0].options.length.should.be.exactly(1);
+    ast.body[0].options[0].kind.should.be.exactly('array');
+    ast.body[0].options[0].value.length.should.be.exactly(0);
+  });
 });