From 334d02d26eead0f26b338871936d195df9e5827c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Raphael=20von=20der=20Gr=C3=BCn?= Date: Tue, 6 Jul 2021 13:01:37 +0200 Subject: [PATCH] tests(prepare): factor out common vars (#1259) --- spec/unit/prepare.spec.js | 98 +++++++++------------------------------ 1 file changed, 21 insertions(+), 77 deletions(-) diff --git a/spec/unit/prepare.spec.js b/spec/unit/prepare.spec.js index 3b197089..2b6e4af8 100644 --- a/spec/unit/prepare.spec.js +++ b/spec/unit/prepare.spec.js @@ -94,23 +94,37 @@ function mockGetSplashScreenItem (data) { } describe('prepare', () => { - describe('updateIcons method', function () { // Rewire - let prepare; + let prepare; + // Spies + let emitSpy; + let updatePathsSpy; + + beforeEach(() => { + prepare = rewire('../../bin/templates/cordova/lib/prepare'); + + emitSpy = jasmine.createSpy('emit'); + prepare.__set__('events', { + emit: emitSpy + }); + + updatePathsSpy = jasmine.createSpy('updatePaths'); + prepare.__set__('FileUpdater', { + updatePaths: updatePathsSpy + }); + }); + + describe('updateIcons method', function () { // Spies let updateIconResourceForAdaptiveSpy; let updateIconResourceForLegacySpy; - let emitSpy; - let updatePathsSpy; // Mock Data let cordovaProject; let platformResourcesDir; beforeEach(function () { - prepare = rewire('../../bin/templates/cordova/lib/prepare'); - cordovaProject = { root: '/mock', projectConfig: { @@ -124,16 +138,6 @@ describe('prepare', () => { }; platformResourcesDir = PATH_RESOURCE; - emitSpy = jasmine.createSpy('emit'); - prepare.__set__('events', { - emit: emitSpy - }); - - updatePathsSpy = jasmine.createSpy('updatePaths'); - prepare.__set__('FileUpdater', { - updatePaths: updatePathsSpy - }); - // mocking initial responses for mapImageResources prepare.__set__('mapImageResources', function (rootDir, subDir, type, resourceName) { if (resourceName.includes('ic_launcher.png')) { @@ -506,20 +510,10 @@ describe('prepare', () => { }); describe('prepareIcons method', function () { - let prepare; - let emitSpy; let prepareIcons; beforeEach(function () { - prepare = rewire('../../bin/templates/cordova/lib/prepare'); - prepareIcons = prepare.__get__('prepareIcons'); - - // Creating Spies - emitSpy = jasmine.createSpy('emit'); - prepare.__set__('events', { - emit: emitSpy - }); }); it('Test#001 : should emit extra default icon found for adaptive use case.', function () { @@ -570,8 +564,6 @@ describe('prepare', () => { }); describe('updateIconResourceForLegacy method', function () { - let prepare; - // Spies let fsWriteFileSyncSpy; @@ -581,8 +573,6 @@ describe('prepare', () => { let resourceMap; beforeEach(function () { - prepare = rewire('../../bin/templates/cordova/lib/prepare'); - // Mocked Data platformResourcesDir = PATH_RESOURCE; preparedIcons = { @@ -618,8 +608,6 @@ describe('prepare', () => { }); describe('updateIconResourceForAdaptive method', function () { - let prepare; - // Spies let fsWriteFileSyncSpy; @@ -629,8 +617,6 @@ describe('prepare', () => { let resourceMap; beforeEach(function () { - prepare = rewire('../../bin/templates/cordova/lib/prepare'); - // Mocked Data platformResourcesDir = PATH_RESOURCE; preparedIcons = { @@ -669,24 +655,6 @@ describe('prepare', () => { }); describe('cleanIcons method', function () { - let prepare; - let emitSpy; - let updatePathsSpy; - - beforeEach(function () { - prepare = rewire('../../bin/templates/cordova/lib/prepare'); - - emitSpy = jasmine.createSpy('emit'); - prepare.__set__('events', { - emit: emitSpy - }); - - updatePathsSpy = jasmine.createSpy('updatePaths'); - prepare.__set__('FileUpdater', { - updatePaths: updatePathsSpy - }); - }); - it('Test#001 : should detect that the app does not have defined icons.', function () { // Mock const icons = []; @@ -767,10 +735,9 @@ describe('prepare', () => { }); describe('prepare arguments', () => { - // Rewire + // Rewire let Api; let api; - let prepare; // Spies let gradlePropertiesParserSpy; @@ -781,7 +748,6 @@ describe('prepare', () => { beforeEach(function () { Api = rewire('../../bin/templates/cordova/Api'); - prepare = rewire('../../bin/templates/cordova/lib/prepare'); cordovaProject = { root: '/mock', @@ -806,9 +772,6 @@ describe('prepare', () => { Api.__set__('prepare', prepare.prepare); - prepare.__set__('events', { - emit: jasmine.createSpy('emit') - }); prepare.__set__('updateUserProjectGradleConfig', jasmine.createSpy()); prepare.__set__('updateWww', jasmine.createSpy()); prepare.__set__('updateProjectAccordingTo', jasmine.createSpy('updateProjectAccordingTo') @@ -848,20 +811,11 @@ describe('prepare', () => { }); describe('updateSplashes method', function () { - // Rewire - let prepare; - - // Spies - let emitSpy; - let updatePathsSpy; - // Mock Data let cordovaProject; let platformResourcesDir; beforeEach(function () { - prepare = rewire('../../bin/templates/cordova/lib/prepare'); - cordovaProject = { root: '/mock', projectConfig: { @@ -875,16 +829,6 @@ describe('prepare', () => { }; platformResourcesDir = PATH_RESOURCE; - emitSpy = jasmine.createSpy('emit'); - prepare.__set__('events', { - emit: emitSpy - }); - - updatePathsSpy = jasmine.createSpy('updatePaths'); - prepare.__set__('FileUpdater', { - updatePaths: updatePathsSpy - }); - // mocking initial responses for mapImageResources prepare.__set__('makeSplashCleanupMap', (rootDir, resourcesDir) => ({ [path.join(resourcesDir, 'drawable-mdpi/screen.png')]: null,