change test case

syzh · syzh · commit 35bc4576eb57 · 2019-11-08T18:49:55.000+08:00
diff --git a/deploy/tencentDeploy.test.js b/deploy/tencentDeploy.test.js
@@ -1,5 +1,7 @@
 'use strict';
 
+const fs              = require('fs');
+const os              = require('os');
 const sinon           = require('sinon');
 const TencentProvider = require('../provider/tencentProvider');
 const Serverless      = require('../test/serverless');
@@ -9,6 +11,8 @@ describe('TencentDeploy', () => {
     let serverless;
     let options;
     let tencentDeploy;
+    let readFileSyncStub;
+    let homedirStub;
 
     beforeEach(() => {
         serverless = new Serverless();
@@ -17,10 +21,24 @@ describe('TencentDeploy', () => {
             region: 'ap-guangzhou',
             function: 'test'
         };
+
+        readFileSyncStub = sinon.stub(fs, 'readFileSync')
+            .returns(`[default]
+tencent_secret_key = PYR4a0HSZ******eVvHRe
+tencent_secret_id = AKIDoM*****mxsfOirI
+tencent_appid = 12561*****`);
+        homedirStub = sinon.stub(os, 'homedir')
+            .returns('/root');
+
         serverless.setProvider('tencent', new TencentProvider(serverless, options));
         tencentDeploy = new TencentDeploy(serverless, options);
     });
 
+    afterEach(() => {
+        fs.readFileSync.restore();
+        os.homedir.restore();
+    });
+
     describe('#constructor()', () => {
         it('should set the serverless instance', () => {
             tencentDeploy.serverless.should.equal(serverless);
diff --git a/deploy/tencentDeployFunction.test.js b/deploy/tencentDeployFunction.test.js
@@ -1,5 +1,7 @@
 'use strict';
 
+const fs                    = require('fs');
+const os                    = require('os');
 const sinon                 = require('sinon');
 const TencentProvider       = require('../provider/tencentProvider');
 const Serverless            = require('../test/serverless');
@@ -9,6 +11,8 @@ describe('TencentDeployFunction', () => {
     let serverless;
     let options;
     let tencentDeployFunction;
+    let readFileSyncStub;
+    let homedirStub;
 
     beforeEach(() => {
         serverless = new Serverless();
@@ -17,10 +21,24 @@ describe('TencentDeployFunction', () => {
             region: 'ap-guangzhou',
             function: 'test'
         };
+
+        readFileSyncStub = sinon.stub(fs, 'readFileSync')
+            .returns(`[default]
+tencent_secret_key = PYR4a0HSZ******eVvHRe
+tencent_secret_id = AKIDoM*****mxsfOirI
+tencent_appid = 12561*****`);
+        homedirStub = sinon.stub(os, 'homedir')
+            .returns('/root');
+
         serverless.setProvider('tencent', new TencentProvider(serverless, options));
         tencentDeployFunction = new TencentDeployFunction(serverless, options);
     });
 
+    afterEach(() => {
+        fs.readFileSync.restore();
+        os.homedir.restore();
+    });
+
     describe('#constructor()', () => {
         it('should set the serverless instance', () => {
             tencentDeployFunction.serverless.should.equal(serverless);
diff --git a/deploy/tencentDeployList.test.js b/deploy/tencentDeployList.test.js
@@ -1,5 +1,7 @@
 'use strict';
 
+const fs                = require('fs');
+const os                = require('os');
 const sinon             = require('sinon');
 const TencentProvider   = require('../provider/tencentProvider');
 const Serverless        = require('../test/serverless');
@@ -9,6 +11,8 @@ describe('TencentDeployList', () => {
     let serverless;
     let options;
     let tencentDeployList;
+    let readFileSyncStub;
+    let homedirStub;
 
     beforeEach(() => {
         serverless = new Serverless();
@@ -17,10 +21,24 @@ describe('TencentDeployList', () => {
             region: 'ap-guangzhou',
             function: 'test'
         };
+
+        readFileSyncStub = sinon.stub(fs, 'readFileSync')
+            .returns(`[default]
+tencent_secret_key = PYR4a0HSZ******eVvHRe
+tencent_secret_id = AKIDoM*****mxsfOirI
+tencent_appid = 12561*****`);
+        homedirStub = sinon.stub(os, 'homedir')
+            .returns('/root');
+
         serverless.setProvider('tencent', new TencentProvider(serverless, options));
         tencentDeployList = new TencentDeployList(serverless, options);
     });
 
+    afterEach(() => {
+        fs.readFileSync.restore();
+        os.homedir.restore();
+    });
+
     describe('#constructor()', () => {
         it('should set the serverless instance', () => {
             tencentDeployList.serverless.should.equal(serverless);
diff --git a/deploy/tencentDeployListFunctions.test.js b/deploy/tencentDeployListFunctions.test.js
@@ -1,5 +1,7 @@
 'use strict';
 
+const fs                         = require('fs');
+const os                         = require('os');
 const sinon                      = require('sinon');
 const TencentProvider            = require('../provider/tencentProvider');
 const Serverless                 = require('../test/serverless');
@@ -9,6 +11,8 @@ describe('TencentDeployListFunctions', () => {
     let serverless;
     let options;
     let tencentDeployListFunctions;
+    let readFileSyncStub;
+    let homedirStub;
 
     beforeEach(() => {
         serverless = new Serverless();
@@ -17,10 +21,24 @@ describe('TencentDeployListFunctions', () => {
             region: 'ap-guangzhou',
             function: 'test'
         };
+
+        readFileSyncStub = sinon.stub(fs, 'readFileSync')
+            .returns(`[default]
+tencent_secret_key = PYR4a0HSZ******eVvHRe
+tencent_secret_id = AKIDoM*****mxsfOirI
+tencent_appid = 12561*****`);
+        homedirStub = sinon.stub(os, 'homedir')
+            .returns('/root');
+
         serverless.setProvider('tencent', new TencentProvider(serverless, options));
         tencentDeployListFunctions = new TencentDeployListFunctions(serverless, options);
     });
 
+    afterEach(() => {
+        fs.readFileSync.restore();
+        os.homedir.restore();
+    });
+
     describe('#constructor()', () => {
         it('should set the serverless instance', () => {
             tencentDeployListFunctions.serverless.should.equal(serverless);
diff --git a/provider/tencentProvider.js b/provider/tencentProvider.js
@@ -21,7 +21,7 @@ class TencentProvider {
 		this.getCredentials(this.serverless, this.options);
 		this.serverless.setProvider(constants.providerName, this);
 		this.provider = this;
-		this.getServiceResource();
+		// this.getServiceResource();
 	}
 
 	jsonObjectIsEmpty(jsonObject) {
diff --git a/rollback/tencentRollback.test.js b/rollback/tencentRollback.test.js
@@ -1,5 +1,7 @@
 'use strict';
 
+const fs              = require('fs');
+const os              = require('os');
 const sinon           = require('sinon');
 const TencentProvider = require('../provider/tencentProvider');
 const Serverless      = require('../test/serverless');
@@ -9,6 +11,8 @@ describe('TencentRollback', () => {
     let serverless;
     let options;
     let tencentRollback;
+    let readFileSyncStub;
+    let homedirStub;
 
     beforeEach(() => {
         serverless = new Serverless();
@@ -17,10 +21,24 @@ describe('TencentRollback', () => {
             region: 'ap-guangzhou',
             function: 'test'
         };
+
+        readFileSyncStub = sinon.stub(fs, 'readFileSync')
+            .returns(`[default]
+tencent_secret_key = PYR4a0HSZ******eVvHRe
+tencent_secret_id = AKIDoM*****mxsfOirI
+tencent_appid = 12561*****`);
+        homedirStub = sinon.stub(os, 'homedir')
+            .returns('/root');
+
         serverless.setProvider('tencent', new TencentProvider(serverless, options));
         tencentRollback = new TencentRollback(serverless, options);
     });
 
+    afterEach(() => {
+        fs.readFileSync.restore();
+        os.homedir.restore();
+    });
+
     describe('#constructor()', () => {
         it('should set the serverless instance', () => {
             tencentRollback.serverless.should.equal(serverless);

Original file line number	Diff line number	Diff line change
`@@ -21,7 +21,7 @@ class TencentProvider {`
`21`	`21`	`this.getCredentials(this.serverless, this.options);`
`22`	`22`	`this.serverless.setProvider(constants.providerName, this);`
`23`	`23`	`this.provider = this;`
`24`		`- this.getServiceResource();`
	`24`	`+ // this.getServiceResource();`
`25`	`25`	`}`
`26`	`26`
`27`	`27`	`jsonObjectIsEmpty(jsonObject) {`