From 99e943dd1392f5ac4213f83346b9e3f259039e03 Mon Sep 17 00:00:00 2001
From: Ben Alpert <spicyjalapeno@gmail.com>
Date: Sat, 29 Mar 2014 17:01:23 -1000
Subject: [PATCH] Clone objects in immutable tests

Fixes #1314, fixes #1328.

Test Plan:
`grunt test`, `grunt test --debug` in Chrome and Firefox.
---
 .../__tests__/ImmutableObject-test.js         | 20 +++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

diff --git a/src/vendor/immutable/__tests__/ImmutableObject-test.js b/src/vendor/immutable/__tests__/ImmutableObject-test.js
index f126cd70dd..c39953003e 100644
--- a/src/vendor/immutable/__tests__/ImmutableObject-test.js
+++ b/src/vendor/immutable/__tests__/ImmutableObject-test.js
@@ -56,6 +56,14 @@ describe('ImmutableObject', function() {
     testProd(message + ':PROD', testFunc);
   };
 
+  /**
+   * Deep clone a JSON-ifiable object. Jasmine doesn't support comparing frozen
+   * objects, so we clone before asserting equality.
+   */
+  var deepClone = function(obj) {
+    return JSON.parse(JSON.stringify(obj));
+  };
+
   testDev('should be running in DEV', function() {
     expect(window.__DEV__).toBe(true);
   });
@@ -165,7 +173,7 @@ describe('ImmutableObject', function() {
 
       var beforeIO = new ImmutableObject(beforeStructure);
       var afterIO = ImmutableObject.set(beforeIO, delta);
-      expect(afterIO).toEqual(expectedAfterStructure);
+      expect(deepClone(afterIO)).toEqual(expectedAfterStructure);
       expect(afterIO).not.toBe(beforeIO);
     }
   );
@@ -192,7 +200,7 @@ describe('ImmutableObject', function() {
 
       var beforeIO = new ImmutableObject(beforeStructure);
       var afterIO = ImmutableObject.set(beforeIO, delta);
-      expect(afterIO).toEqual(expectedAfterStructure);
+      expect(deepClone(afterIO)).toEqual(expectedAfterStructure);
       expect(afterIO).not.toBe(beforeIO);
     }
   );
@@ -220,7 +228,7 @@ describe('ImmutableObject', function() {
 
     var beforeIO = new ImmutableObject(beforeStructure);
     var afterIO = ImmutableObject.set(beforeIO, delta);
-    expect(afterIO).toEqual(expectedAfterStructure);
+    expect(deepClone(afterIO)).toEqual(expectedAfterStructure);
     expect(afterIO).not.toBe(beforeIO);
   });
 
@@ -251,7 +259,7 @@ describe('ImmutableObject', function() {
     var beforeIO = new ImmutableObject({initialField: null});
     var afterIO =
       ImmutableObject.setProperty(beforeIO, 'anotherField', 'anotherValue');
-    expect(afterIO).toEqual({
+    expect(deepClone(afterIO)).toEqual({
       initialField: null,
       anotherField: 'anotherValue'
     });
@@ -267,7 +275,7 @@ describe('ImmutableObject', function() {
     var afterIO = ImmutableObject.setDeep(beforeIO, {
       a: {b: {}, c: 'C', e: {f: 'F', g: 'G'}, h: 'H'}
     });
-    expect(afterIO).toEqual({
+    expect(deepClone(afterIO)).toEqual({
       a: {b: {}, c: 'C', d: 'd', e: {f: 'F', g: 'G'}, h: 'H'}
     });
     expect(afterIO).not.toBe(beforeIO);
@@ -283,7 +291,7 @@ describe('ImmutableObject', function() {
     var afterIO = ImmutableObject.setDeep(beforeIO, {
       a: {b: {d: 'D'}, e: new ImmutableObject({g: 'G'})}
     });
-    expect(afterIO).toEqual({
+    expect(deepClone(afterIO)).toEqual({
       a: {b: {c: 'c', d: 'D'}, e: {f: 'f', g: 'G'}}
     });
     expect(afterIO instanceof Immutable).toBe(true);