Skip to content

Commit cf71a55

Browse files
authored
Added vscode-insiders path installed via snap (#137117)
Closes #137116 <b>Before</b> VS Code Insiders installed via Snap isn't detected. <b>After</b> ```sh [�] VS Code (version 1.84.0-insider) � VS Code at /snap/code-insiders/current/usr/share/code-insiders � Flutter extension version 3.75.20231002 ```
1 parent 7d9010c commit cf71a55

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

packages/flutter_tools/lib/src/vscode/vscode.dart

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -240,6 +240,7 @@ class VsCode {
240240
// /usr/share/code/bin/code
241241
// /snap/code/current/usr/share/code
242242
// /usr/share/code-insiders/bin/code-insiders
243+
// /snap/code-insiders/current/usr/share/code-insiders
243244
// Linux Extensions:
244245
// $HOME/.vscode/extensions
245246
// $HOME/.vscode-insiders/extensions
@@ -251,6 +252,10 @@ class VsCode {
251252
'/usr/share/code-insiders',
252253
'.vscode-insiders',
253254
),
255+
const VsCodeInstallLocation(
256+
'/snap/code-insiders/current/usr/share/code-insiders',
257+
'.vscode-insiders',
258+
),
254259
], fileSystem, platform);
255260
}
256261

packages/flutter_tools/test/general.shard/vscode/vscode_test.dart

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,11 +46,12 @@ void main() {
4646
final Platform platform = FakePlatform(environment: <String, String>{'HOME': home});
4747

4848
fileSystem.directory(fileSystem.path.join('/snap/code/current/usr/share/code', '.vscode')).createSync(recursive: true);
49+
fileSystem.directory(fileSystem.path.join('/snap/code-insiders/current/usr/share/code-insiders', '.vscode-insiders')).createSync(recursive: true);
4950

5051
final FakeProcessManager processManager = FakeProcessManager.list(<FakeCommand>[]);
5152

5253
final List<VsCode> installed = VsCode.allInstalled(fileSystem, platform, processManager);
53-
expect(installed.length, 1);
54+
expect(installed.length, 2);
5455
});
5556

5657
testWithoutContext('can locate installations on macOS', () {

0 commit comments

Comments
 (0)