Merge pull request #2732 from Automattic/update/refactor-domains-purc…

…hases-tests

Domains: Purchases: Refactor domains and purchases tests

client/lib/domains/dns/test/reducer-test.js

@@ -1,6 +1,7 @@
 /**
  * External dependencies
  */
+import deepFreeze from 'deep-freeze';
 import { expect } from 'chai';
 import pick from 'lodash/object/pick';
 
@@ -13,11 +14,11 @@ import { reducer } from '../reducer';
 
 describe( 'Domains: DNS reducer', () => {
 	it( 'should return the same state when no matching record passed in the delete action', () => {
-		const state = {
+		const state = deepFreeze( {
 				[ DOMAIN_NAME ]: {
 					records: [ RECORD_A ]
 				}
-			},
+			} ),
 			payload = {
 				action: {
 					type: ActionTypes.DNS_DELETE_COMPLETED,
@@ -32,11 +33,11 @@ describe( 'Domains: DNS reducer', () => {
 	} );
 
 	it( 'should return state without record passed in the delete action', () => {
-		const state = {
+		const state = deepFreeze( {
 				[ DOMAIN_NAME ]: {
 					records: [ RECORD_TXT ]
 				}
-			},
+			} ),
 			payload = {
 				action: {
 					type: ActionTypes.DNS_DELETE_COMPLETED,
@@ -52,11 +53,11 @@ describe( 'Domains: DNS reducer', () => {
 
 	it( 'should return state without record (having no id) passed in the delete action', () => {
 		const RECORD_TXT_WITHOUT_ID = pick( RECORD_TXT, [ 'data', 'name', 'type' ] ),
-			state = {
+			state = deepFreeze( {
 				[ DOMAIN_NAME ]: {
 					records: [ RECORD_TXT_WITHOUT_ID ]
 				}
-			},
+			} ),
 			payload = {
 				action: {
 					type: ActionTypes.DNS_DELETE_COMPLETED,

client/lib/domains/email-forwarding/test/reducer-test.js

@@ -1,6 +1,7 @@
 /**
  * External dependencies
  */
+import deepFreeze from 'deep-freeze';
 import { expect } from 'chai';
 
 /**
@@ -12,11 +13,11 @@ import { reducer } from './../reducer';
 
 describe( 'Domains: Email Forwarding Reducer', () => {
 	it( 'should return the same state when no matching record passed in the delete complete action', () => {
-		const state = {
+		const state = deepFreeze( {
 				[ DOMAIN_NAME ]: {
 					list: EMAIL_FORWARDS
 				}
-			},
+			} ),
 			payload = {
 				action: {
 					type: ActionTypes.EMAIL_FORWARDING_DELETE_COMPLETED,
@@ -35,11 +36,11 @@ describe( 'Domains: Email Forwarding Reducer', () => {
 	} );
 
 	it( 'should return state without record passed in the delete completed action', () => {
-		const state = {
+		const state = deepFreeze( {
 				[ DOMAIN_NAME ]: {
 					list: EMAIL_FORWARDS
 				}
-			},
+			} ),
 			payload = {
 				action: {
 					type: ActionTypes.EMAIL_FORWARDING_DELETE_COMPLETED,

client/lib/purchases/stored-cards/test/assembler-test.js

@@ -7,7 +7,7 @@ import { expect } from 'chai';
  * Internal dependencies
  */
 import { createStoredCardsArray } from '../assembler.js';
-import { STORED_CARDS_FROM_API, STORED_CARDS } from './constants';
+import { STORED_CARDS_FROM_API, STORED_CARDS } from './data';
 
 describe( 'Stored Cards assembler', () => {
 	it( 'should be a function', () => {

client/lib/purchases/stored-cards/test/store-test.js

@@ -11,7 +11,7 @@ import rest from 'lodash/array/rest';
  */
 import { action as ActionTypes } from 'lib/upgrades/constants';
 import Dispatcher from 'dispatcher';
-import { STORED_CARDS_FROM_API, STORED_CARDS } from './constants';
+import { STORED_CARDS_FROM_API, STORED_CARDS } from './data';
 import StoredCardsStore from './../store';
 
 describe( 'Stored Cards Store', () => {

package.json

@@ -115,6 +115,7 @@
     "blanket": "1.1.6",
     "chai": "2.0.0",
     "chai-immutable": "^1.4.0",
+    "deep-freeze": "0.0.1",
     "enzyme": "1.1.0",
     "esformatter": "0.7.3",
     "esformatter-braces": "1.2.1",