insert-test add case

mtipnis · Apr 26, 2012 · d5c4493 · d5c4493
1 parent 91d39e3
commit d5c4493
Show file tree

Hide file tree

Showing 3 changed files with 34 additions and 4 deletions.
diff --git a/modules/compiler/lib/peg/ql.js b/modules/compiler/lib/peg/ql.js
diff --git a/modules/compiler/pegs/ql.peg b/modules/compiler/pegs/ql.peg
@@ -699,13 +699,19 @@ InsertStatement = 'insert' insig 'into' insig s:Source insig c:ColumnsParen? ins
     if (!c && v.value.length > 1){
         throw new this.SyntaxError("Line " + line + ": Values do not have paired columns.");
     }
-    return {
+    ret = {
         type: 'insert',
         source: s,
-        columns: c,
         values: v.value,
         line: line
     }
+    if (c){
+        ret.columns = c,
+        ret.values = v.value;
+    }else{
+        ret.values = v.value[0];
+    }
+    return ret
 }
 
 // Delete statement

diff --git a/modules/compiler/test/insert-test.js b/modules/compiler/test/insert-test.js
@@ -109,3 +109,21 @@ exports['insert-no-table'] = function(test) {
         test.done();
     }
 };
+
+exports['insert-opaque'] = function(test) {
+    var q = "insert into suppliers values ('24553')"
+    var statement = compiler.compile(q);
+    var e = [
+        {
+            "type": "insert",
+            "source": {
+                "name": "suppliers"
+            },
+            "values": "24553",
+            "line": 1,
+            "id": 0
+        }
+    ];
+    test.deepEqual(statement, e);
+    test.done();
+};