Add tests for utility functions

Author: Mario Aguiar

plugin/assets/src/settings/utils/index.js

@@ -0,0 +1,18 @@
+/**
+ * Copyright 2021 Google LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+export * from './update';
+export * from './promises';

plugin/assets/src/settings/utils.js …in/assets/src/settings/utils/promises.jsplugin/assets/src/settings/utils.js renamed to plugin/assets/src/settings/utils/promises.js plugin/assets/src/settings/utils.js renamed to plugin/assets/src/settings/utils/promises.js

@@ -22,34 +22,13 @@ import apiFetch from '@wordpress/api-fetch';
 /**
  * Internal dependencies
  */
-import { UPDATERS } from './constants';
-import getConfig from '../admin/get-config';
-
-/**
- * Handles each kind of update
- *
- * @param {*} type Item to update
- */
-export const update = type => {
-	if ( type === UPDATERS.FONTS.type ) {
-		return updateFonts();
-	}
-
-	if ( type === UPDATERS.ICONS.type ) {
-		return updateIcons();
-	}
-
-	if ( isCoreUpdate( type ) ) {
-		return new Promise( () => {
-			window.location.href = getConfig( 'coreUpdateUrl' );
-		} );
-	}
-};
+import { UPDATERS } from '../constants';
+import getConfig from '../../admin/get-config';
 
 /**
  * Update fonts.
  */
-const updateFonts = () => {
+export const updateFonts = () => {
 	return new Promise( ( resolve, reject ) => {
 		apiFetch( {
 			path: getConfig( 'assetsRestPath' ) + 'retrieve-fonts/force',
@@ -66,7 +45,7 @@ const updateFonts = () => {
 /**
  * Update icons.
  */
-const updateIcons = () => {
+export const updateIcons = () => {
 	return new Promise( ( resolve, reject ) => {
 		apiFetch( {
 			path: getConfig( 'assetsRestPath' ) + 'retrieve-icons/force',
@@ -80,7 +59,7 @@ const updateIcons = () => {
 	} );
 };
 
-const isCoreUpdate = type =>
+export const isCoreUpdate = type =>
 	[ UPDATERS.PLUGIN.type, UPDATERS.THEME.type ].includes( type );
 
 /**

plugin/assets/src/settings/utils/update.js

@@ -0,0 +1,27 @@
+/**
+ * Internal dependencies
+ */
+import { UPDATERS } from '../constants';
+import getConfig from '../../admin/get-config';
+import { updateFonts, updateIcons, isCoreUpdate } from './promises';
+
+/**
+ * Handles each kind of update
+ *
+ * @param {*} type Item to update
+ */
+export const update = type => {
+	if ( type === UPDATERS.FONTS.type ) {
+		return updateFonts();
+	}
+
+	if ( type === UPDATERS.ICONS.type ) {
+		return updateIcons();
+	}
+
+	if ( isCoreUpdate( type ) ) {
+		return new Promise( () => {
+			window.location.href = getConfig( 'coreUpdateUrl' );
+		} );
+	}
+};

plugin/tests/js/settings/reducer.test.js

@@ -37,7 +37,7 @@ const initialState = {
 	errors: {},
 };
 
-describe( 'Reducer', () => {
+describe( 'Settings: Reducer', () => {
 	it( 'should return default state', () => {
 		const action = {
 			type: null,

plugin/tests/js/settings/utils/update.test.js

@@ -0,0 +1,45 @@
+/**
+ * Copyright 2020 Google LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+import '@testing-library/jest-dom/extend-expect';
+
+jest.mock( '../../../../assets/src/settings/utils/promises', () => ( {
+	updateFonts: jest.fn(),
+	updateIcons: jest.fn(),
+} ) );
+
+/**
+ * Internal dependencies
+ */
+import { update } from '../../../../assets/src/settings/utils/update';
+import {
+	updateFonts,
+	updateIcons,
+} from '../../../../assets/src/settings/utils/promises';
+
+describe( 'Settings: Update', () => {
+	it( 'should call fonts', () => {
+		update( 'FONTS' );
+
+		expect( updateFonts ).toHaveBeenCalledWith();
+	} );
+
+	it( 'should call icons', () => {
+		update( 'ICONS' );
+
+		expect( updateIcons ).toHaveBeenCalledWith();
+	} );
+} );