Fix memory leak from snapshot printing (#5279)

* Fix memory leak from obsolete snapshot printing * Update issue number * Rm "obsolete" to remove confusion * Update index.js
jestjs · Jan 11, 2018 · 4bcfe19 · 4bcfe19
1 parent 239f1ec
commit 4bcfe19
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,4 +1,9 @@
 ## master
+## jest 22.0.6
+
+### Fixes
+
+* `[jest-jasmine2]` Fix memory leak in snapshot reporting ([#5279](https://github.com/facebook/jest/pull/5279))
 
 ## jest 22.0.5
 

diff --git a/packages/jest-jasmine2/src/index.js b/packages/jest-jasmine2/src/index.js
@@ -174,7 +174,8 @@ const addSnapshotData = (results, snapshotState) => {
   results.snapshot.unmatched = snapshotState.unmatched;
   results.snapshot.updated = snapshotState.updated;
   results.snapshot.unchecked = !status.deleted ? uncheckedCount : 0;
-  results.snapshot.uncheckedKeys = uncheckedKeys;
+  // Copy the array to prevent memory leaks
+  results.snapshot.uncheckedKeys = Array.from(uncheckedKeys);
 
   return results;
 };