Add test case

WordPress · Nov 16, 2018 · e6a4094 · e6a4094
1 parent 3c8dbd5
commit e6a4094
Show file tree

Hide file tree

Showing 2 changed files with 120 additions and 22 deletions.
diff --git a/test/e2e/specs/__snapshots__/writing-flow.test.js.snap b/test/e2e/specs/__snapshots__/writing-flow.test.js.snap
@@ -32,6 +32,52 @@ exports[`adding blocks should clean TinyMCE content 2`] = `
 <!-- /wp:paragraph -->"
 `;
 
+exports[`adding blocks should create valid paragraph blocks when rapidly pressing Enter 1`] = `
+"<!-- wp:paragraph -->
+<p></p>
+<!-- /wp:paragraph -->
+
+<!-- wp:paragraph -->
+<p></p>
+<!-- /wp:paragraph -->
+
+<!-- wp:paragraph -->
+<p></p>
+<!-- /wp:paragraph -->
+
+<!-- wp:paragraph -->
+<p></p>
+<!-- /wp:paragraph -->
+
+<!-- wp:paragraph -->
+<p></p>
+<!-- /wp:paragraph -->
+
+<!-- wp:paragraph -->
+<p></p>
+<!-- /wp:paragraph -->
+
+<!-- wp:paragraph -->
+<p></p>
+<!-- /wp:paragraph -->
+
+<!-- wp:paragraph -->
+<p></p>
+<!-- /wp:paragraph -->
+
+<!-- wp:paragraph -->
+<p></p>
+<!-- /wp:paragraph -->
+
+<!-- wp:paragraph -->
+<p></p>
+<!-- /wp:paragraph -->
+
+<!-- wp:paragraph -->
+<p></p>
+<!-- /wp:paragraph -->"
+`;
+
 exports[`adding blocks should insert line break at end 1`] = `
 "<!-- wp:paragraph -->
 <p>a<br><br></p>
@@ -75,3 +121,39 @@ exports[`adding blocks should navigate around inline boundaries 1`] = `
 <p>BeforeThird</p>
 <!-- /wp:paragraph -->"
 `;
+
+exports[`adding blocks should not delete surrounding space when deleting a selected word 1`] = `
+"<!-- wp:paragraph -->
+<p>alpha  gamma</p>
+<!-- /wp:paragraph -->"
+`;
+
+exports[`adding blocks should not delete surrounding space when deleting a selected word 2`] = `
+"<!-- wp:paragraph -->
+<p>alpha betagamma</p>
+<!-- /wp:paragraph -->"
+`;
+
+exports[`adding blocks should not delete surrounding space when deleting a word with Alt+Backspace 1`] = `
+"<!-- wp:paragraph -->
+<p>alpha  gamma</p>
+<!-- /wp:paragraph -->"
+`;
+
+exports[`adding blocks should not delete surrounding space when deleting a word with Alt+Backspace 2`] = `
+"<!-- wp:paragraph -->
+<p>alpha beta gamma</p>
+<!-- /wp:paragraph -->"
+`;
+
+exports[`adding blocks should not delete surrounding space when deleting a word with Backspace 1`] = `
+"<!-- wp:paragraph -->
+<p>1  3</p>
+<!-- /wp:paragraph -->"
+`;
+
+exports[`adding blocks should not delete surrounding space when deleting a word with Backspace 2`] = `
+"<!-- wp:paragraph -->
+<p>1 2 3</p>
+<!-- /wp:paragraph -->"
+`;
diff --git a/test/e2e/specs/writing-flow.test.js b/test/e2e/specs/writing-flow.test.js
@@ -224,42 +224,58 @@ describe( 'adding blocks', () => {
 		expect( isInBlock ).toBe( true );
 	} );
 
-	it( 'should not delete trailing spaces when deleting a word with backspace', async () => {
+	it( 'should not delete surrounding space when deleting a word with Backspace', async () => {
 		await clickBlockAppender();
-		await page.keyboard.type( '1 2 3 4' );
+		await page.keyboard.type( '1 2 3' );
+		await pressTimes( 'ArrowLeft', ' 3'.length );
 		await page.keyboard.press( 'Backspace' );
-		await page.keyboard.type( '4' );
-		const blockText = await page.evaluate( () => document.activeElement.textContent );
-		expect( blockText ).toBe( '1 2 3 4' );
+
+		expect( await getEditedPostContent() ).toMatchSnapshot();
+
+		await page.keyboard.type( '2' );
+
+		expect( await getEditedPostContent() ).toMatchSnapshot();
 	} );
 
-	it( 'should not delete trailing spaces when deleting a word with alt + backspace', async () => {
+	it( 'should not delete surrounding space when deleting a word with Alt+Backspace', async () => {
 		await clickBlockAppender();
-		await page.keyboard.type( 'alpha beta gamma delta' );
+		await page.keyboard.type( 'alpha beta gamma' );
+		await pressTimes( 'ArrowLeft', ' gamma'.length );
+
 		if ( process.platform === 'darwin' ) {
 			await pressWithModifier( 'Alt', 'Backspace' );
 		} else {
 			await pressWithModifier( META_KEY, 'Backspace' );
 		}
-		await page.keyboard.type( 'delta' );
-		const blockText = await page.evaluate( () => document.activeElement.textContent );
-		expect( blockText ).toBe( 'alpha beta gamma delta' );
+
+		expect( await getEditedPostContent() ).toMatchSnapshot();
+
+		await page.keyboard.type( 'beta' );
+
+		expect( await getEditedPostContent() ).toMatchSnapshot();
+	} );
+
+	it( 'should not delete surrounding space when deleting a selected word', async () => {
+		await clickBlockAppender();
+		await page.keyboard.type( 'alpha beta gamma' );
+		await pressTimes( 'ArrowLeft', ' gamma'.length );
+		await page.keyboard.down( 'Shift' );
+		await pressTimes( 'ArrowLeft', 'beta'.length );
+		await page.keyboard.up( 'Shift' );
+		await page.keyboard.press( 'Backspace' );
+
+		expect( await getEditedPostContent() ).toMatchSnapshot();
+
+		await page.keyboard.type( 'beta' );
+
+		expect( await getEditedPostContent() ).toMatchSnapshot();
 	} );
 
 	it( 'should create valid paragraph blocks when rapidly pressing Enter', async () => {
 		await clickBlockAppender();
-		await page.keyboard.press( 'Enter' );
-		await page.keyboard.press( 'Enter' );
-		await page.keyboard.press( 'Enter' );
-		await page.keyboard.press( 'Enter' );
-		await page.keyboard.press( 'Enter' );
-		await page.keyboard.press( 'Enter' );
-		await page.keyboard.press( 'Enter' );
-		await page.keyboard.press( 'Enter' );
-		await page.keyboard.press( 'Enter' );
-		await page.keyboard.press( 'Enter' );
+		await pressTimes( 'Enter', 10 );
+
 		// Check that none of the paragraph blocks have <br> in them.
-		const postContent = await getEditedPostContent();
-		expect( postContent.indexOf( 'br' ) ).toBe( -1 );
+		expect( await getEditedPostContent() ).toMatchSnapshot();
 	} );
 } );