feat: add support for object-spread syntax (#82)

istanbuljs · Sep 4, 2017 · 28d5566 · 28d5566
1 parent 6695de8
commit 28d5566
Show file tree

Hide file tree

Showing 7 changed files with 342 additions and 181 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -5,4 +5,5 @@ node_js:
   - '6'
   - 'stable'
 
+before_install: npm i yarn -g
 after_script: nyc report --reporter=text-lcov | coveralls
diff --git a/package.json b/package.json
@@ -26,7 +26,7 @@
   "devDependencies": {
     "coveralls": "^2.13.0",
     "cross-env": "^5.0.0",
-    "lerna": "^2.0.0-rc.4",
+    "lerna": "^2.1.2",
     "nyc": "^11.0.1"
   },
   "nyc": {

diff --git a/packages/istanbul-lib-instrument/package.json b/packages/istanbul-lib-instrument/package.json
@@ -18,7 +18,7 @@
     "babel-template": "^6.16.0",
     "babel-traverse": "^6.18.0",
     "babel-types": "^6.18.0",
-    "babylon": "^6.17.4",
+    "babylon": "^6.18.0",
     "istanbul-lib-coverage": "^1.1.1",
     "semver": "^5.3.0"
   },

diff --git a/packages/istanbul-lib-instrument/src/instrumenter.js b/packages/istanbul-lib-instrument/src/instrumenter.js
@@ -84,6 +84,7 @@ class Instrumenter {
             plugins: [
               'asyncGenerators',
               'dynamicImport',
+              'objectRestSpread',
               'flow',
               'jsx'
             ]

diff --git a/packages/istanbul-lib-instrument/test/specs/object-spread.yaml b/packages/istanbul-lib-instrument/test/specs/object-spread.yaml
@@ -0,0 +1,15 @@
+---
+name: ES6 Object Spread
+guard: isObjectSpreadAvailable
+code: |
+  const a = {foo: args[0]}
+  const b = {...a}
+  output = b.foo
+tests:
+  - name: is instrumented
+    args: [10]
+    out: 10
+    lines: { '1': 1, '2': 1, '3': 1 }
+    statements: {'0': 1, '1': 1, '2': 1 }
+    functions: {}
+    branches: {}
diff --git a/packages/istanbul-lib-instrument/test/util/guards.js b/packages/istanbul-lib-instrument/test/util/guards.js
@@ -24,6 +24,10 @@ export function isArrowFnAvailable() {
     return tryThis('[1 ,2, 3].map(x => x * x)', 'arrow function');
 }
 
+export function isObjectSpreadAvailable() {
+    return tryThis('const a = {...{b: 33}}', 'object-spread');
+}
+
 export function isObjectFreezeAvailable() {
     "use strict";
     if (!Object.freeze) {