Fix parsing of table.el-style dividers

See amake/orgro#177

Author: amake

lib/src/org/grammar.dart

@@ -591,9 +591,13 @@ class OrgContentGrammarDefinition extends GrammarDefinition {
           .flatten(message: 'Trailing line content expected') &
       ref0(lineTrailing).flatten(message: 'Trailing line content expected');
 
+  // This grammar can actually be customized in table.el; see
+  // `table-cell-*-char(s)`. See `table-recognize` for where they are combined
+  // for parsing purposes.
   Parser tableDotElDivider() =>
       ref0(indent).flatten(message: 'Table.el divider indent expected') &
-      (string('+-') & anyOf('+-').starString())
+      // At least three characters required
+      (string('+-') & anyOf('+-').plusString())
           .flatten(message: 'Table divider expected') &
       ref0(lineTrailing).flatten(message: 'Trailing line content expected');
 

test/org/grammar/content_test.dart

@@ -39,6 +39,38 @@ bazinga''');
         ]
       ]);
     });
+    group('markup', () {
+      test('markup edge case', () {
+        // https://github.com/amake/orgro/issues/177
+        final result = parser.parse('''+- Some text here+
+- Some text here''');
+        expect(result.value, [
+          [
+            '',
+            [
+              ['+', '- Some text here', '+'],
+              '\n'
+            ],
+            ''
+          ],
+          [
+            [
+              [
+                '',
+                '-',
+                [
+                  ' ',
+                  null,
+                  null,
+                  ['Some text here']
+                ]
+              ]
+            ],
+            ''
+          ]
+        ]);
+      });
+    });
     group('link', () {
       test('bare HTTP URL', () {
         final result = parser.parse('a http://example.com b');

test/org/grammar/table_test.dart

@@ -3,9 +3,9 @@ import 'package:petitparser/petitparser.dart';
 import 'package:test/test.dart';
 
 void main() {
+  final grammar = OrgContentGrammarDefinition();
+  final parser = grammar.buildFrom(grammar.table()).end();
   test('table', () {
-    final grammar = OrgContentGrammarDefinition();
-    final parser = grammar.buildFrom(grammar.table()).end();
     final result = parser.parse('''  | foo | bar | baz |
   |-----+-----+-----|
   |   1 |   2 |   3 |
@@ -39,8 +39,6 @@ void main() {
   });
   test('table containing plus', () {
     // https://github.com/amake/orgro/issues/175
-    final grammar = OrgContentGrammarDefinition();
-    final parser = grammar.buildFrom(grammar.table()).end();
     final result = parser.parse('''  |-------+------|
 | +Text | Text |
 |-------+------|
@@ -74,4 +72,28 @@ void main() {
       ''
     ]);
   });
+  test('minimal table', () {
+    final result = parser.parse('+-+');
+    expect(result.value, [
+      [
+        ['', '+-+', '']
+      ],
+      ''
+    ]);
+  });
+  test('minimal table 2', () {
+    // This doesn't look valid but `table-recognize` accepts it.
+    final result = parser.parse('+--');
+    expect(result.value, [
+      [
+        ['', '+--', '']
+      ],
+      ''
+    ]);
+  });
+  test('invalid', () {
+    // https://github.com/amake/orgro/issues/177
+    final result = parser.parse('''+- Some text here+''');
+    expect(result, isA<Failure>());
+  });
 }