fix tests after adding sandbox as required param

Author: hugomontero

src/cmd/secrets.test.js

@@ -101,7 +101,7 @@ describe('SecretsCommand', () => {
 
 			await secretsCommand.list({ org: 'my-org' });
 
-			expect(secretsStub.list).to.have.been.calledWith({ org: 'my-org', api: secretsCommand.api });
+			expect(secretsStub.list).to.have.been.calledWith({ org: 'my-org', api: secretsCommand.api, sandbox: undefined });
 			expect(secretsCommand.ui.showBusySpinnerUntilResolved).to.have.been.calledWith('Retrieving secrets');
 		});
 
@@ -149,7 +149,8 @@ describe('SecretsCommand', () => {
 			expect(secretsStub.get).to.have.been.calledWith({
 				api: secretsCommand.api,
 				name: 'MY_SECRET',
-				org: undefined
+				org: undefined,
+				sandbox: true
 			});
 			expect(secretsCommand.ui.showBusySpinnerUntilResolved).to.have.been.calledWith('Retrieving secret');
 		});
@@ -170,7 +171,8 @@ describe('SecretsCommand', () => {
 			expect(secretsStub.get).to.have.been.calledWith({
 				api: secretsCommand.api,
 				name: 'ORG_SECRET',
-				org: 'my-org'
+				org: 'my-org',
+				sandbox: undefined
 			});
 		});
 	});
@@ -184,6 +186,7 @@ describe('SecretsCommand', () => {
 			expect(secretsStub.remove).to.have.been.calledWith({
 				api: secretsCommand.api,
 				org: undefined,
+				sandbox: true,
 				name: 'MY_SECRET'
 			});
 			expect(secretsCommand.ui.write).to.have.been.calledWith('Secret MY_SECRET deleted successfully.');
@@ -197,6 +200,7 @@ describe('SecretsCommand', () => {
 			expect(secretsStub.remove).to.have.been.calledWith({
 				api: secretsCommand.api,
 				org: 'my-org',
+				sandbox: undefined,
 				name: 'ORG_SECRET'
 			});
 			expect(secretsCommand.ui.write).to.have.been.calledWith('Secret ORG_SECRET deleted successfully.');
@@ -221,6 +225,7 @@ describe('SecretsCommand', () => {
 				api: secretsCommand.api,
 				name: 'MY_SECRET',
 				org: undefined,
+				sandbox: true,
 				value: 'secret-value'
 			});
 			expect(secretsCommand.ui.write).to.have.been.calledWith('Secret MY_SECRET set successfully.');
@@ -243,6 +248,7 @@ describe('SecretsCommand', () => {
 				api: secretsCommand.api,
 				name: 'MY_SECRET',
 				org: undefined,
+				sandbox: true,
 				value: 'secret-value'
 			});
 		});
@@ -264,6 +270,7 @@ describe('SecretsCommand', () => {
 				api: secretsCommand.api,
 				name: 'ORG_SECRET',
 				org: 'my-org',
+				sandbox: undefined,
 				value: 'org-value'
 			});
 		});

src/lib/secrets.test.js

@@ -20,15 +20,15 @@ describe('secrets', () => {
 				.intercept('/secrets', 'GET')
 				.reply(200, emptySecretsList);
 
-			const secretsResponse = await secrets.list({ api });
+			const secretsResponse = await secrets.list({ api, sandbox: true });
 			expect(secretsResponse).to.deep.equal([]);
 		});
 
 		it('returns a list of secrets', async () => {
 			nock(baseUrl)
 				.intercept('/secrets', 'GET')
 				.reply(200, secretsList);
-			const secretsResponse = await secrets.list({ api });
+			const secretsResponse = await secrets.list({ api, sandbox: true });
 			expect(secretsResponse).to.deep.equal(formattedSecretList);
 		});
 
@@ -51,7 +51,7 @@ describe('secrets', () => {
 			nock(baseUrl)
 				.intercept(`/secrets/${secret.name}`, 'PUT')
 				.reply(200, secretGenericResponse);
-			const secretResponse = await secrets.update({ api, ...secret });
+			const secretResponse = await secrets.update({ api, ...secret, sandbox: true });
 			expect(secretResponse).to.deep.equal(formattedGenericSecretGet);
 		});
 
@@ -82,7 +82,7 @@ describe('secrets', () => {
 				name: 'secret_name',
 				value: 'value'
 			};
-			const secretResponse = secrets.update({ api, ...secret });
+			const secretResponse = secrets.update({ api, ...secret, sandbox: true });
 			await expect(secretResponse).to.be.rejectedWith('Keys may include only uppercase letters, digits, and underscores, and must not begin with a digit.');
 		});
 
@@ -94,7 +94,7 @@ describe('secrets', () => {
 			nock(baseUrl)
 				.intercept(`/secrets/${secret.name}`, 'PUT')
 				.reply(400, { ok: false, error: 'Failed to create secret: Name already in use' });
-			const secretResponse = secrets.update({ api, ...secret });
+			const secretResponse = secrets.update({ api, ...secret, sandbox: true });
 			await expect(secretResponse).to.be.rejectedWith('Failed to create secret: Name already in use');
 		});
 	});
@@ -109,7 +109,7 @@ describe('secrets', () => {
 				.intercept(`/secrets/${secret.name}`, 'PUT')
 				.reply(200, secretGenericResponse);
 
-			const secretResponse = await secrets.update({ api, ...secret });
+			const secretResponse = await secrets.update({ api, ...secret, sandbox: true });
 			expect(secretResponse).to.deep.equal(formattedGenericSecretGet);
 		});
 		it('updates an org secret', async () => {
@@ -129,7 +129,7 @@ describe('secrets', () => {
 			nock(baseUrl)
 				.intercept(`/secrets/${name}`, 'GET')
 				.reply(200, secretGenericResponse);
-			const secretResponse = await secrets.get({ api, name });
+			const secretResponse = await secrets.get({ api, name, sandbox: true });
 			expect(secretResponse).to.deep.equal(formattedGenericSecretGet);
 		});
 
@@ -149,7 +149,7 @@ describe('secrets', () => {
 			nock(baseUrl)
 				.intercept(`/secrets/${name}`, 'GET')
 				.reply(404, { ok: false, error: 'not found' });
-			const secretResponse = secrets.get({ api, name });
+			const secretResponse = secrets.get({ api, name, sandbox: true });
 			await expect(secretResponse).to.be.rejectedWith(errorMessage);
 		});
 
@@ -161,7 +161,7 @@ describe('secrets', () => {
 			nock(baseUrl)
 				.intercept(`/secrets/${name}`, 'DELETE')
 				.reply(204);
-			const secretResponse = await secrets.remove({ api, name });
+			const secretResponse = await secrets.remove({ api, name, sandbox: true });
 			expect(secretResponse).to.equal(true);
 		});
 
@@ -185,7 +185,7 @@ describe('secrets', () => {
 				.intercept(`/secrets/${name}`, 'DELETE')
 				.reply(400, errorMessage);
 
-			const secretResponse = secrets.remove({ api, name });
+			const secretResponse = secrets.remove({ api, name, sandbox: true });
 			await expect(secretResponse).to.be.rejectedWith(errorMessage.error);
 			scope.done();
 		});