Skip to content

Commit 30f7d62

Browse files
committed
revert --no-sandbox
1 parent a0af8d4 commit 30f7d62

File tree

2 files changed

+3
-11
lines changed

2 files changed

+3
-11
lines changed

scripts/test/launchTestUtilities.ts

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -49,10 +49,7 @@ export async function setupVSCodeTestInstance(): Promise<string> {
4949

5050
export async function invokeVSCodeCli(vsCodeExecutablePath: string, args: string[]): Promise<string> {
5151
const [cli, ...cliArgs] = resolveCliArgsFromVSCodeExecutablePath(vsCodeExecutablePath)
52-
// Disable sandbox to avoid: "Running as root without --no-sandbox is not supported".
53-
// https://github.com/aws/aws-toolkit-vscode/issues/3550
54-
// https://crbug.com/638180
55-
const cmdArgs = ['--no-sandbox', ...cliArgs, ...args]
52+
const cmdArgs = [...cliArgs, ...args]
5653

5754
// Workaround: set --user-data-dir to avoid this error in CI:
5855
// "You are trying to start Visual Studio Code as a super user …"
@@ -133,11 +130,6 @@ async function setupVSCode(): Promise<string> {
133130
}
134131

135132
export async function runToolkitTests(suiteName: string, relativeEntryPoint: string) {
136-
// Disable sandbox to avoid: "Running as root without --no-sandbox is not supported".
137-
// https://github.com/aws/aws-toolkit-vscode/issues/3550
138-
// https://crbug.com/638180
139-
const noSandboxArgs = ['--no-sandbox', '--disable-gpu-sandbox']
140-
141133
try {
142134
console.log(`Running ${suiteName} test suite...`)
143135
const vsCodeExecutablePath = await setupVSCode()
@@ -162,7 +154,7 @@ export async function runToolkitTests(suiteName: string, relativeEntryPoint: str
162154
vscodeExecutablePath: vsCodeExecutablePath,
163155
extensionDevelopmentPath: cwd,
164156
extensionTestsPath: testEntrypoint,
165-
launchArgs: [...noSandboxArgs, ...disableExtensions, workspacePath, disableWorkspaceTrust],
157+
launchArgs: [...disableExtensions, workspacePath, disableWorkspaceTrust],
166158
extensionTestsEnv: {
167159
['DEVELOPMENT_PATH']: cwd,
168160
['AWS_TOOLKIT_AUTOMATION']: suiteName,

scripts/test/test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ import { setupVSCodeTestInstance } from './launchTestUtilities'
2121
extensionDevelopmentPath: rootDir,
2222
extensionTestsPath: testEntrypoint,
2323
// For verbose VSCode logs, add "--verbose --log debug". c2165cf48e62c
24-
launchArgs: ['--no-sandbox', '--disable-gpu-sandbox', testWorkspace],
24+
launchArgs: [testWorkspace],
2525
extensionTestsEnv: {
2626
['DEVELOPMENT_PATH']: rootDir,
2727
['AWS_TOOLKIT_AUTOMATION']: 'unit',

0 commit comments

Comments
 (0)