@wordpress/block-editor 12.2.0 → 12.3.0

package/src/store/test/private-selectors.js

@@ -1,3 +1,8 @@
+/**
+ * WordPress dependencies
+ */
+import { select } from '@wordpress/data';
+
 /**
  * Internal dependencies
  */
@@ -5,8 +10,13 @@ import {
 	isBlockInterfaceHidden,
 	getLastInsertedBlocksClientIds,
 	getBlockEditingMode,
+	isBlockSubtreeDisabled,
 } from '../private-selectors';
 
+jest.mock( '@wordpress/data/src/select', () => ( {
+	select: jest.fn(),
+} ) );
+
 describe( 'private selectors', () => {
 	describe( 'isBlockInterfaceHidden', () => {
 		it( 'should return the true if toggled true in state', () => {
@@ -51,7 +61,7 @@ describe( 'private selectors', () => {
 		} );
 	} );
 
-	describe( 'getBlockEditingMode', () => {
+	describe( 'block editing mode selectors', () => {
 		const baseState = {
 			settings: {},
 			blocks: {
@@ -63,6 +73,27 @@ describe( 'private selectors', () => {
 					[ 'b3247f75-fd94-4fef-97f9-5bfd162cc416', {} ], // | |  Paragraph
 					[ 'e178812d-ce5e-48c7-a945-8ae4ffcbbb7c', {} ], // | |  Paragraph
 				] ),
+				order: new Map( [
+					[ '', [ '6cf70164-9097-4460-bcbf-200560546988' ] ],
+					[ '6cf70164-9097-4460-bcbf-200560546988', [] ],
+					[
+						'ef45d5fd-5234-4fd5-ac4f-c3736c7f9337',
+						[
+							'b26fc763-417d-4f01-b81c-2ec61e14a972',
+							'9b9c5c3f-2e46-4f02-9e14-9fe9515b958f',
+						],
+					],
+					[ 'b26fc763-417d-4f01-b81c-2ec61e14a972', [] ],
+					[
+						'9b9c5c3f-2e46-4f02-9e14-9fe9515b958f',
+						[
+							'b3247f75-fd94-4fef-97f9-5bfd162cc416',
+							'e178812d-ce5e-48c7-a945-8ae4ffcbbb7c',
+						],
+					],
+					[ 'b3247f75-fd94-4fef-97f9-5bfd162cc416', [] ],
+					[ 'e178812d-ce5e-48c7-a945-8ae4ffcbbb7c', [] ],
+				] ),
 				parents: new Map( [
 					[ '6cf70164-9097-4460-bcbf-200560546988', '' ],
 					[ 'ef45d5fd-5234-4fd5-ac4f-c3736c7f9337', '' ],
@@ -91,28 +122,28 @@ describe( 'private selectors', () => {
 			blockEditingModes: new Map( [] ),
 		};
 
-		const __experimentalHasContentRoleAttribute = jest.fn( () => false );
-		getBlockEditingMode.registry = {
-			select: jest.fn( () => ( {
+		describe( 'getBlockEditingMode', () => {
+			const __experimentalHasContentRoleAttribute = jest.fn(
+				() => false
+			);
+			select.mockReturnValue( {
 				__experimentalHasContentRoleAttribute,
-			} ) ),
-		};
+			} );
 
-		it( 'should return default by default', () => {
-			expect(
-				getBlockEditingMode(
-					baseState,
-					'b3247f75-fd94-4fef-97f9-5bfd162cc416'
-				)
-			).toBe( 'default' );
-		} );
+			it( 'should return default by default', () => {
+				expect(
+					getBlockEditingMode(
+						baseState,
+						'b3247f75-fd94-4fef-97f9-5bfd162cc416'
+					)
+				).toBe( 'default' );
+			} );
 
-		[ 'disabled', 'contentOnly' ].forEach( ( mode ) => {
-			it( `should return ${ mode } if explicitly set`, () => {
+			it( 'should return disabled if explicitly set', () => {
 				const state = {
 					...baseState,
 					blockEditingModes: new Map( [
-						[ 'b3247f75-fd94-4fef-97f9-5bfd162cc416', mode ],
+						[ 'b3247f75-fd94-4fef-97f9-5bfd162cc416', 'disabled' ],
 					] ),
 				};
 				expect(
@@ -120,14 +151,17 @@ describe( 'private selectors', () => {
 						state,
 						'b3247f75-fd94-4fef-97f9-5bfd162cc416'
 					)
-				).toBe( mode );
+				).toBe( 'disabled' );
 			} );
 
-			it( `should return ${ mode } if explicitly set on a parent`, () => {
+			it( 'should return contentOnly if explicitly set', () => {
 				const state = {
 					...baseState,
 					blockEditingModes: new Map( [
-						[ 'ef45d5fd-5234-4fd5-ac4f-c3736c7f9337', mode ],
+						[
+							'b3247f75-fd94-4fef-97f9-5bfd162cc416',
+							'contentOnly',
+						],
 					] ),
 				};
 				expect(
@@ -135,16 +169,49 @@ describe( 'private selectors', () => {
 						state,
 						'b3247f75-fd94-4fef-97f9-5bfd162cc416'
 					)
-				).toBe( mode );
+				).toBe( 'contentOnly' );
 			} );
 
-			it( `should return ${ mode } if overridden by a parent`, () => {
+			it( 'should return disabled if explicitly set on a parent', () => {
 				const state = {
 					...baseState,
 					blockEditingModes: new Map( [
-						[ '', mode ],
+						[ 'ef45d5fd-5234-4fd5-ac4f-c3736c7f9337', 'disabled' ],
+					] ),
+				};
+				expect(
+					getBlockEditingMode(
+						state,
+						'b3247f75-fd94-4fef-97f9-5bfd162cc416'
+					)
+				).toBe( 'disabled' );
+			} );
+
+			it( 'should return default if parent is set to contentOnly', () => {
+				const state = {
+					...baseState,
+					blockEditingModes: new Map( [
+						[
+							'ef45d5fd-5234-4fd5-ac4f-c3736c7f9337',
+							'contentOnly',
+						],
+					] ),
+				};
+				expect(
+					getBlockEditingMode(
+						state,
+						'b3247f75-fd94-4fef-97f9-5bfd162cc416'
+					)
+				).toBe( 'default' );
+			} );
+
+			it( 'should return disabled if overridden by a parent', () => {
+				const state = {
+					...baseState,
+					blockEditingModes: new Map( [
+						[ '', 'disabled' ],
 						[ 'ef45d5fd-5234-4fd5-ac4f-c3736c7f9337', 'default' ],
-						[ '9b9c5c3f-2e46-4f02-9e14-9fe9515b958f', mode ],
+						[ '9b9c5c3f-2e46-4f02-9e14-9fe9515b958f', 'disabled' ],
 					] ),
 				};
 				expect(
@@ -152,59 +219,167 @@ describe( 'private selectors', () => {
 						state,
 						'b3247f75-fd94-4fef-97f9-5bfd162cc416'
 					)
-				).toBe( mode );
+				).toBe( 'disabled' );
 			} );
 
-			it( `should return ${ mode } if explicitly set on root`, () => {
+			it( 'should return disabled if explicitly set on root', () => {
 				const state = {
 					...baseState,
-					blockEditingModes: new Map( [ [ '', mode ] ] ),
+					blockEditingModes: new Map( [ [ '', 'disabled' ] ] ),
 				};
 				expect(
 					getBlockEditingMode(
 						state,
 						'b3247f75-fd94-4fef-97f9-5bfd162cc416'
 					)
-				).toBe( mode );
+				).toBe( 'disabled' );
 			} );
-		} );
 
-		it( 'should return disabled if parent is locked and the block has no content role', () => {
-			const state = {
-				...baseState,
-				blockListSettings: {
-					...baseState.blockListSettings,
-					'9b9c5c3f-2e46-4f02-9e14-9fe9515b958f': {
-						templateLock: 'contentOnly',
+			it( 'should return default if root is contentOnly', () => {
+				const state = {
+					...baseState,
+					blockEditingModes: new Map( [ [ '', 'contentOnly' ] ] ),
+				};
+				expect(
+					getBlockEditingMode(
+						state,
+						'b3247f75-fd94-4fef-97f9-5bfd162cc416'
+					)
+				).toBe( 'default' );
+			} );
+
+			it( 'should return disabled if parent is locked and the block has no content role', () => {
+				const state = {
+					...baseState,
+					blockListSettings: {
+						...baseState.blockListSettings,
+						'9b9c5c3f-2e46-4f02-9e14-9fe9515b958f': {
+							templateLock: 'contentOnly',
+						},
 					},
-				},
-			};
-			__experimentalHasContentRoleAttribute.mockReturnValueOnce( false );
-			expect(
-				getBlockEditingMode(
-					state,
-					'b3247f75-fd94-4fef-97f9-5bfd162cc416'
-				)
-			).toBe( 'disabled' );
-		} );
+				};
+				__experimentalHasContentRoleAttribute.mockReturnValueOnce(
+					false
+				);
+				expect(
+					getBlockEditingMode(
+						state,
+						'b3247f75-fd94-4fef-97f9-5bfd162cc416'
+					)
+				).toBe( 'disabled' );
+			} );
 
-		it( 'should return contentOnly if parent is locked and the block has a content role', () => {
-			const state = {
-				...baseState,
-				blockListSettings: {
-					...baseState.blockListSettings,
-					'9b9c5c3f-2e46-4f02-9e14-9fe9515b958f': {
-						templateLock: 'contentOnly',
+			it( 'should return contentOnly if parent is locked and the block has a content role', () => {
+				const state = {
+					...baseState,
+					blockListSettings: {
+						...baseState.blockListSettings,
+						'9b9c5c3f-2e46-4f02-9e14-9fe9515b958f': {
+							templateLock: 'contentOnly',
+						},
 					},
-				},
-			};
-			__experimentalHasContentRoleAttribute.mockReturnValueOnce( true );
-			expect(
-				getBlockEditingMode(
-					state,
-					'b3247f75-fd94-4fef-97f9-5bfd162cc416'
-				)
-			).toBe( 'contentOnly' );
+				};
+				__experimentalHasContentRoleAttribute.mockReturnValueOnce(
+					true
+				);
+				expect(
+					getBlockEditingMode(
+						state,
+						'b3247f75-fd94-4fef-97f9-5bfd162cc416'
+					)
+				).toBe( 'contentOnly' );
+			} );
+		} );
+
+		describe( 'isBlockSubtreeDisabled', () => {
+			it( 'should return false when top level block is not disabled', () => {
+				const state = {
+					...baseState,
+					blockEditingModes: new Map( [] ),
+				};
+				expect(
+					isBlockSubtreeDisabled(
+						state,
+						'ef45d5fd-5234-4fd5-ac4f-c3736c7f9337'
+					)
+				).toBe( false );
+			} );
+
+			it( 'should return true when top level block is disabled and there are no editing modes within it', () => {
+				const state = {
+					...baseState,
+					blockEditingModes: new Map( [
+						[ 'ef45d5fd-5234-4fd5-ac4f-c3736c7f9337', 'disabled' ],
+					] ),
+				};
+				expect(
+					isBlockSubtreeDisabled(
+						state,
+						'ef45d5fd-5234-4fd5-ac4f-c3736c7f9337'
+					)
+				).toBe( true );
+			} );
+
+			it( 'should return true when top level block is disabled via inheritence and there are no editing modes within it', () => {
+				const state = {
+					...baseState,
+					blockEditingModes: new Map( [ [ '', 'disabled' ] ] ),
+				};
+				expect(
+					isBlockSubtreeDisabled(
+						state,
+						'ef45d5fd-5234-4fd5-ac4f-c3736c7f9337'
+					)
+				).toBe( true );
+			} );
+
+			it( 'should return true when top level block is disabled and there are disabled editing modes within it', () => {
+				const state = {
+					...baseState,
+					blockEditingModes: new Map( [
+						[ 'ef45d5fd-5234-4fd5-ac4f-c3736c7f9337', 'disabled' ],
+						[ 'b3247f75-fd94-4fef-97f9-5bfd162cc416', 'disabled' ],
+					] ),
+				};
+				expect(
+					isBlockSubtreeDisabled(
+						state,
+						'ef45d5fd-5234-4fd5-ac4f-c3736c7f9337'
+					)
+				).toBe( true );
+			} );
+
+			it( 'should return false when top level block is disabled and there are non-disabled editing modes within it', () => {
+				const state = {
+					...baseState,
+					blockEditingModes: new Map( [
+						[ 'ef45d5fd-5234-4fd5-ac4f-c3736c7f9337', 'disabled' ],
+						[ 'b3247f75-fd94-4fef-97f9-5bfd162cc416', 'default' ],
+					] ),
+				};
+				expect(
+					isBlockSubtreeDisabled(
+						state,
+						'ef45d5fd-5234-4fd5-ac4f-c3736c7f9337'
+					)
+				).toBe( false );
+			} );
+
+			it( 'should return false when top level block is disabled via inheritence and there are non-disabled editing modes within it', () => {
+				const state = {
+					...baseState,
+					blockEditingModes: new Map( [
+						[ '', 'disabled' ],
+						[ 'b3247f75-fd94-4fef-97f9-5bfd162cc416', 'default' ],
+					] ),
+				};
+				expect(
+					isBlockSubtreeDisabled(
+						state,
+						'ef45d5fd-5234-4fd5-ac4f-c3736c7f9337'
+					)
+				).toBe( false );
+			} );
 		} );
 	} );
 } );

package/src/store/test/reducer.js

@@ -2889,7 +2889,6 @@ describe( 'state', () => {
 					'core/embed': {
 						time: 123456,
 						count: 1,
-						insert: { name: 'core/embed' },
 					},
 				},
 			} );
@@ -2900,7 +2899,6 @@ describe( 'state', () => {
 						'core/embed': {
 							time: 123456,
 							count: 1,
-							insert: { name: 'core/embed' },
 						},
 					},
 				} ),
@@ -2926,12 +2924,10 @@ describe( 'state', () => {
 					'core/embed': {
 						time: 123457,
 						count: 2,
-						insert: { name: 'core/embed' },
 					},
 					'core/block/123': {
 						time: 123457,
 						count: 1,
-						insert: { name: 'core/block', ref: 123 },
 					},
 				},
 			} );
@@ -2994,17 +2990,14 @@ describe( 'state', () => {
 						[ orangeVariationName ]: {
 							time: 123456,
 							count: 1,
-							insert: { name: orangeVariationName },
 						},
 						[ appleVariationName ]: {
 							time: 123456,
 							count: 1,
-							insert: { name: appleVariationName },
 						},
 						[ blockWithVariations ]: {
 							time: 123456,
 							count: 2,
-							insert: { name: blockWithVariations },
 						},
 					} ),
 				} );