full coverage for optimizer

Author: reedM2

src/optimizer.js

@@ -44,7 +44,6 @@ const optimizers = {
         case '<=': result = left <= right; break;
         case '>': result = left > right; break;
         case '>=': result = left >= right; break;
-        default: return e;
       }
       return result ? e.thenBranch : e.elseBranch;
     }
@@ -126,12 +125,6 @@ const optimizers = {
     return g;
   },
 
-  numrange(nr) {
-    nr.start = optimize(nr.start);
-    nr.end = optimize(nr.end);
-    return nr;
-  },
-
   num(n) {
     return n;
   },

test/optimizer.test.js

@@ -220,17 +220,19 @@ const tests = [
     core.printStmt(id("x"))
   ],
   [
-    "optimizes nested expressions in numrange start/end",
-    core.numRange(
-      add(num(2), "+", num(3)),
-      mul(num(4), "*", num(2))
-    ),
-    core.numRange(num(5), num(8)),
+    "returns unoptimized conditional with non-numeric operands",
+    condExpr(id("a"), "==", id("b"), num(10), num(20)),
+    condExpr(id("a"), "==", id("b"), num(10), num(20)),
+  ],
+  [
+    "returns unoptimized addition with non-numeric operands",
+    add(id("x"), "+", id("y")),
+    add(id("x"), "+", id("y")),
   ],
   [
-    "handles numrange with only start value",
-    core.numRange(num(5)),
-    core.numRange(num(5)), 
+    "returns unoptimized multiplication with non-numeric operands",
+    mul(id("x"), "*", id("y")),
+    mul(id("x"), "*", id("y")),
   ],
 ];