Fix existing JS unit tests

WordPress · Jul 14, 2020 · fa51d1a · fa51d1a
1 parent 6953ad4
commit fa51d1a
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 22 deletions.
diff --git a/packages/edit-site/src/store/reducer.js b/packages/edit-site/src/store/reducer.js
@@ -165,7 +165,7 @@ export function page( state, action ) {
  *
  * @return {Object} Updated state.
  */
-export function homeTemplateId( state = {}, action ) {
+export function homeTemplateId( state, action ) {
 	switch ( action.type ) {
 		case 'SET_HOME_TEMPLATE':
 			return action.homeTemplateId;

diff --git a/packages/edit-site/src/store/test/reducer.js b/packages/edit-site/src/store/test/reducer.js
@@ -14,7 +14,6 @@ import {
 	templatePartId,
 	templateType,
 	page,
-	showOnFront,
 } from '../reducer';
 import { PREFERENCES_DEFAULTS } from '../defaults';
 
@@ -165,7 +164,7 @@ describe( 'state', () => {
 
 	describe( 'page()', () => {
 		it( 'should apply default state', () => {
-			expect( page( undefined, {} ) ).toEqual( {} );
+			expect( page( undefined, {} ) ).toEqual( undefined );
 		} );
 
 		it( 'should default to returning the same state', () => {
@@ -183,15 +182,4 @@ describe( 'state', () => {
 			).toBe( newPage );
 		} );
 	} );
-
-	describe( 'showOnFront()', () => {
-		it( 'should apply default state', () => {
-			expect( showOnFront( undefined, {} ) ).toEqual( undefined );
-		} );
-
-		it( 'should default to returning the same state', () => {
-			const state = {};
-			expect( showOnFront( state, {} ) ).toBe( state );
-		} );
-	} );
 } );
diff --git a/packages/edit-site/src/store/test/selectors.js b/packages/edit-site/src/store/test/selectors.js
@@ -10,7 +10,6 @@ import {
 	getTemplatePartId,
 	getTemplateType,
 	getPage,
-	getShowOnFront,
 } from '../selectors';
 
 describe( 'selectors', () => {
@@ -142,11 +141,4 @@ describe( 'selectors', () => {
 			expect( getPage( state ) ).toBe( state.page );
 		} );
 	} );
-
-	describe( 'getShowOnFront', () => {
-		it( 'returns the `show_on_front` setting', () => {
-			const state = { showOnFront: {} };
-			expect( getShowOnFront( state ) ).toBe( state.showOnFront );
-		} );
-	} );
 } );