Ensure round-trip tests don't mutate output before checking against it => reveals 4 tests are broken (#4188)

* Ensure round-trip tests don't mutate output before checking against it.  Need deep copy instead of shallow.

* Skip the 4 tests that are broken - they should perhaps be fixed in a separate PR

packages/slate-history/test/undo/delete_backward/block-text.tsx

@@ -1,6 +1,7 @@
 /** @jsx jsx */
 import { Transforms } from 'slate'
 import { jsx } from '../..'
+import { cloneDeep } from 'lodash'
 
 export const run = editor => {
   Transforms.delete(editor)
@@ -14,4 +15,5 @@ export const input = (
     </block>
   </editor>
 )
-export const output = input
+export const skip = true // TODO: see https://github.com/ianstormtaylor/slate/pull/4188
+export const output = cloneDeep(input)

packages/slate-history/test/undo/delete_backward/custom-prop.tsx

@@ -1,6 +1,7 @@
 /** @jsx jsx */
 import { Transforms } from 'slate'
 import { jsx } from '../..'
+import { cloneDeep } from 'lodash'
 
 export const run = editor => {
   Transforms.delete(editor)
@@ -17,4 +18,5 @@ export const input = (
     </block>
   </editor>
 )
-export const output = input
+export const skip = true // TODO: see https://github.com/ianstormtaylor/slate/pull/4188
+export const output = cloneDeep(input)

packages/slate-history/test/undo/delete_backward/inline-across.tsx

@@ -1,6 +1,7 @@
 /** @jsx jsx */
 import { Transforms } from 'slate'
 import { jsx } from '../..'
+import { cloneDeep } from 'lodash'
 
 export const run = editor => {
   Transforms.delete(editor)
@@ -25,4 +26,5 @@ export const input = (
     </block>
   </editor>
 )
-export const output = input
+export const skip = true // TODO: see https://github.com/ianstormtaylor/slate/pull/4188
+export const output = cloneDeep(input)

packages/slate-history/test/undo/insert_break/basic.tsx

@@ -1,6 +1,7 @@
 /** @jsx jsx */
 import { Editor } from 'slate'
 import { jsx } from '../..'
+import { cloneDeep } from 'lodash'
 
 export const run = editor => {
   editor.insertBreak()
@@ -16,4 +17,4 @@ export const input = (
     </block>
   </editor>
 )
-export const output = input
+export const output = cloneDeep(input)

packages/slate-history/test/undo/insert_fragment/basic.tsx

@@ -1,5 +1,6 @@
 /** @jsx jsx */
 import { jsx } from '../..'
+import { cloneDeep } from 'lodash'
 
 const fragment = (
   <block type="d">
@@ -33,5 +34,5 @@ export const input = (
     </block>
   </editor>
 )
-export const output = input
+export const output = cloneDeep(input)
 export const skip = true

packages/slate-history/test/undo/insert_text/basic.tsx

@@ -1,5 +1,6 @@
 /** @jsx jsx */
 import { jsx } from '../..'
+import { cloneDeep } from 'lodash'
 
 export const run = editor => {
   editor.insertText('text')
@@ -12,4 +13,4 @@ export const input = (
     </block>
   </editor>
 )
-export const output = input
+export const output = cloneDeep(input)

packages/slate-history/test/undo/insert_text/contiguous.tsx

@@ -1,5 +1,6 @@
 /** @jsx jsx */
 import { jsx } from '../..'
+import { cloneDeep } from 'lodash'
 
 export const run = editor => {
   editor.insertText('t')
@@ -14,4 +15,4 @@ export const input = (
     </block>
   </editor>
 )
-export const output = input
+export const output = cloneDeep(input)

packages/slate/test/interfaces/Node/ancestor/success.tsx

@@ -1,6 +1,7 @@
 /** @jsx jsx  */
 import { Node } from 'slate'
 import { jsx } from 'slate-hyperscript'
+import { cloneDeep } from 'lodash'
 
 export const input = (
   <editor>
@@ -12,4 +13,4 @@ export const input = (
 export const test = value => {
   return Node.ancestor(value, [0])
 }
-export const output = input.children[0]
+export const output = cloneDeep(input.children[0])

packages/slate/test/interfaces/Node/child/success.tsx

@@ -1,6 +1,7 @@
 /** @jsx jsx  */
 import { Node } from 'slate'
 import { jsx } from 'slate-hyperscript'
+import { cloneDeep } from 'lodash'
 
 export const input = (
   <editor>
@@ -12,4 +13,4 @@ export const input = (
 export const test = value => {
   return Node.child(value, 0)
 }
-export const output = input.children[0]
+export const output = cloneDeep(input.children[0])

packages/slate/test/interfaces/Node/descendant/success.tsx

@@ -1,6 +1,7 @@
 /** @jsx jsx  */
 import { Node } from 'slate'
 import { jsx } from 'slate-hyperscript'
+import { cloneDeep } from 'lodash'
 
 export const input = (
   <editor>
@@ -12,4 +13,4 @@ export const input = (
 export const test = value => {
   return Node.descendant(value, [0])
 }
-export const output = input.children[0]
+export const output = cloneDeep(input.children[0])

packages/slate/test/interfaces/Node/get/root.tsx

@@ -1,6 +1,7 @@
 /** @jsx jsx  */
 import { Node } from 'slate'
 import { jsx } from 'slate-hyperscript'
+import { cloneDeep } from 'lodash'
 
 export const input = (
   <editor>
@@ -12,4 +13,5 @@ export const input = (
 export const test = value => {
   return Node.get(value, [])
 }
-export const output = input
+export const skip = true // TODO: see https://github.com/ianstormtaylor/slate/pull/4188
+export const output = cloneDeep(input)

packages/slate/test/transforms/delete/unit-line/text-end.tsx

@@ -1,6 +1,7 @@
 /** @jsx jsx */
 import { Transforms } from 'slate'
 import { jsx } from '../../..'
+import { cloneDeep } from 'lodash'
 
 export const run = editor => {
   Transforms.delete(editor, { unit: 'line' })
@@ -13,4 +14,4 @@ export const input = (
     </block>
   </editor>
 )
-export const output = input
+export const output = cloneDeep(input)

packages/slate/test/transforms/delete/unit-line/text-start-reverse.tsx

@@ -1,6 +1,7 @@
 /** @jsx jsx */
 import { Transforms } from 'slate'
 import { jsx } from '../../..'
+import { cloneDeep } from 'lodash'
 
 export const run = editor => {
   Transforms.delete(editor, { unit: 'line', reverse: true })
@@ -13,4 +14,4 @@ export const input = (
     </block>
   </editor>
 )
-export const output = input
+export const output = cloneDeep(input)