Fix failing e2e test

WordPress · Oct 24, 2018 · 381d0c7 · 381d0c7
1 parent 3c37ef7
commit 381d0c7
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/packages/rich-text/src/test/to-dom.js b/packages/rich-text/src/test/to-dom.js
@@ -29,6 +29,7 @@ describe( 'recordToDom', () => {
 				value: record,
 				multilineTag,
 				multilineWrapperTags,
+				createLinePadding: ( doc ) => doc.createElement( 'br' ),
 			} );
 			expect( body ).toMatchSnapshot();
 			expect( selection ).toEqual( { startPath, endPath } );

diff --git a/packages/rich-text/src/to-dom.js b/packages/rich-text/src/to-dom.js
@@ -105,10 +105,7 @@ function remove( node ) {
 	return node.parentNode.removeChild( node );
 }
 
-function padEmptyLines( {
-	element,
-	createLinePadding = ( doc ) => doc.createElement( 'br' ),
-} ) {
+function padEmptyLines( { element, createLinePadding } ) {
 	const length = element.childNodes.length;
 	const doc = element.ownerDocument;
 
@@ -158,7 +155,9 @@ export function toDom( {
 		},
 	} );
 
-	padEmptyLines( { element: tree, createLinePadding } );
+	if ( createLinePadding ) {
+		padEmptyLines( { element: tree, createLinePadding } );
+	}
 
 	return {
 		body: tree,

diff --git a/test/e2e/specs/__snapshots__/deprecated-node-matcher.test.js.snap b/test/e2e/specs/__snapshots__/deprecated-node-matcher.test.js.snap
@@ -8,6 +8,6 @@ exports[`Deprecated Node Matcher should insert block with children source 1`] =
 
 exports[`Deprecated Node Matcher should insert block with node source 1`] = `
 "<!-- wp:deprecated-node-matcher -->
-<blockquote class=\\"wp-block-deprecated-node-matcher\\"><p>test</p><p><br data-mce-bogus=\\"1\\"/></p></blockquote>
+<blockquote class=\\"wp-block-deprecated-node-matcher\\"><p>test</p><p></p></blockquote>
 <!-- /wp:deprecated-node-matcher -->"
 `;