Merge pull request #108 from conveyal/dev

v3.2.1
conveyal · Dec 15, 2016 · 184e667 · 184e667
2 parents c1139e9 + aa651e0
commit 184e667
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/lib/__tests__/babel.js b/lib/__tests__/babel.js
@@ -0,0 +1,11 @@
+/* globals describe, expect, it */
+
+describe('babel', () => {
+  /** As of 2016-12-15 this test fails in babel loose mode. https://github.com/babel/babel/issues/4916 */
+  it('should iterate correctly over a set after using the spread operator', () => {
+    const originalArr = [1, 1, 2]
+    const set = new Set(originalArr)
+    const arr = [...set]
+    expect(arr.length).toBe(2)
+  })
+})
diff --git a/lib/babel-config.js b/lib/babel-config.js
@@ -13,7 +13,7 @@ module.exports = function (env) {
     presets: [
       [babelEnv, {
         targets: {browsers},
-        loose: true
+        loose: false // Loose mode breaks spread operator on `Set`s
       }],
       react,
       stage2