From 8cea0cf94effed6c4d77e2e4ccba14656a9c8166 Mon Sep 17 00:00:00 2001
From: Philipp Muens <raute1337@gmx.de>
Date: Tue, 11 Apr 2017 13:31:39 +0200
Subject: [PATCH] Update AwsDeploy tests to reflect calling order

---
 lib/plugins/aws/deploy/index.test.js | 22 ++++++++++++++--------
 1 file changed, 14 insertions(+), 8 deletions(-)

diff --git a/lib/plugins/aws/deploy/index.test.js b/lib/plugins/aws/deploy/index.test.js
index eb4f3463d..598f9d21a 100644
--- a/lib/plugins/aws/deploy/index.test.js
+++ b/lib/plugins/aws/deploy/index.test.js
@@ -126,9 +126,9 @@ describe('AwsDeploy', () => {
 
         return awsDeploy.hooks['before:deploy:deploy']().then(() => {
           expect(spawnAwsCommonValidateStub.calledOnce).to.equal(true);
-          expect(spawnPackageStub.calledOnce).to.equal(true);
+          expect(spawnPackageStub.calledAfter(spawnAwsCommonValidateStub)).to.equal(true);
+          expect(extendedValidateStub.calledAfter(spawnPackageStub)).to.equal(true);
           expect(spawnAwsCommonMoveArtifactsToTemp.calledOnce).to.equal(false);
-          expect(extendedValidateStub.calledOnce).to.equal(true);
         });
       });
 
@@ -138,9 +138,11 @@ describe('AwsDeploy', () => {
 
         return awsDeploy.hooks['before:deploy:deploy']().then(() => {
           expect(spawnAwsCommonValidateStub.calledOnce).to.equal(true);
+          expect(spawnAwsCommonMoveArtifactsToTemp.calledAfter(spawnAwsCommonValidateStub))
+            .to.equal(true);
+          expect(extendedValidateStub.calledAfter(spawnAwsCommonMoveArtifactsToTemp))
+            .to.equal(true);
           expect(spawnPackageStub.calledOnce).to.equal(false);
-          expect(spawnAwsCommonMoveArtifactsToTemp.calledOnce).to.equal(true);
-          expect(extendedValidateStub.calledOnce).to.equal(true);
         });
       });
 
@@ -150,9 +152,11 @@ describe('AwsDeploy', () => {
 
         return awsDeploy.hooks['before:deploy:deploy']().then(() => {
           expect(spawnAwsCommonValidateStub.calledOnce).to.equal(true);
+          expect(spawnAwsCommonMoveArtifactsToTemp.calledAfter(spawnAwsCommonValidateStub))
+            .to.equal(true);
+          expect(extendedValidateStub.calledAfter(spawnAwsCommonMoveArtifactsToTemp))
+            .to.equal(true);
           expect(spawnPackageStub.calledOnce).to.equal(false);
-          expect(spawnAwsCommonMoveArtifactsToTemp.calledOnce).to.equal(true);
-          expect(extendedValidateStub.calledOnce).to.equal(true);
         });
       });
     });
@@ -223,7 +227,8 @@ describe('AwsDeploy', () => {
 
         return awsDeploy.hooks['aws:deploy:finalize:cleanup']().then(() => {
           expect(cleanupS3BucketStub.calledOnce).to.equal(true);
-          expect(spawnAwsCommonCleanupTempDirStub.calledOnce).to.equal(true);
+          expect(spawnAwsCommonCleanupTempDirStub.calledAfter(cleanupS3BucketStub))
+            .to.equal(true);
         });
       });
 
@@ -233,7 +238,8 @@ describe('AwsDeploy', () => {
 
         return awsDeploy.hooks['aws:deploy:finalize:cleanup']().then(() => {
           expect(cleanupS3BucketStub.calledOnce).to.equal(true);
-          expect(spawnAwsCommonCleanupTempDirStub.calledOnce).to.equal(true);
+          expect(spawnAwsCommonCleanupTempDirStub.calledAfter(cleanupS3BucketStub))
+            .to.equal(true);
         });
       });
     });