Skip to content

Commit 2a14127

Browse files
committed
fix: fixing test cases
1 parent f92fa04 commit 2a14127

File tree

1 file changed

+33
-6
lines changed

1 file changed

+33
-6
lines changed

packages/core/src/test/shared/vscode/env.test.ts

Lines changed: 33 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -139,18 +139,45 @@ describe('env', function () {
139139
})
140140

141141
it('returns false in SageMaker environment with SAGEMAKER_APP_TYPE', function () {
142-
sandbox.stub(process.env, 'SAGEMAKER_APP_TYPE').value('JupyterLab')
143-
assert.strictEqual(isAmazonLinux2(), false)
142+
const originalValue = process.env.SAGEMAKER_APP_TYPE
143+
process.env.SAGEMAKER_APP_TYPE = 'JupyterLab'
144+
try {
145+
assert.strictEqual(isAmazonLinux2(), false)
146+
} finally {
147+
if (originalValue === undefined) {
148+
delete process.env.SAGEMAKER_APP_TYPE
149+
} else {
150+
process.env.SAGEMAKER_APP_TYPE = originalValue
151+
}
152+
}
144153
})
145154

146155
it('returns false in SageMaker environment with SM_APP_TYPE', function () {
147-
sandbox.stub(process.env, 'SM_APP_TYPE').value('JupyterLab')
148-
assert.strictEqual(isAmazonLinux2(), false)
156+
const originalValue = process.env.SM_APP_TYPE
157+
process.env.SM_APP_TYPE = 'JupyterLab'
158+
try {
159+
assert.strictEqual(isAmazonLinux2(), false)
160+
} finally {
161+
if (originalValue === undefined) {
162+
delete process.env.SM_APP_TYPE
163+
} else {
164+
process.env.SM_APP_TYPE = originalValue
165+
}
166+
}
149167
})
150168

151169
it('returns false in SageMaker environment with SERVICE_NAME', function () {
152-
sandbox.stub(process.env, 'SERVICE_NAME').value('SageMakerUnifiedStudio')
153-
assert.strictEqual(isAmazonLinux2(), false)
170+
const originalValue = process.env.SERVICE_NAME
171+
process.env.SERVICE_NAME = 'SageMakerUnifiedStudio'
172+
try {
173+
assert.strictEqual(isAmazonLinux2(), false)
174+
} finally {
175+
if (originalValue === undefined) {
176+
delete process.env.SERVICE_NAME
177+
} else {
178+
process.env.SERVICE_NAME = originalValue
179+
}
180+
}
154181
})
155182

156183
it('returns false when /etc/os-release indicates Ubuntu in container', function () {

0 commit comments

Comments
 (0)