Skip to content

Commit d6926f7

Browse files
committed
Fix ide not exists bug
1 parent 53a66d2 commit d6926f7

File tree

2 files changed

+15
-1
lines changed

2 files changed

+15
-1
lines changed

components/server/src/workspace/workspace-starter.spec.ts

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -249,5 +249,19 @@ describe("workspace-starter", function () {
249249
const result = chooseIDE("unknown-custom", customOptions, useLatest, hasPerm);
250250
expect(result.ideImage).to.equal(ideOptions.options["code"].latestImage);
251251
});
252+
253+
it("not exists ide with custom permission", function () {
254+
const useLatest = true;
255+
const hasPerm = true;
256+
const result = chooseIDE("not-exists", ideOptions, useLatest, hasPerm);
257+
expect(result.ideImage).to.equal(ideOptions.options["code"].latestImage);
258+
});
259+
260+
it("not exists ide with custom permission", function () {
261+
const useLatest = true;
262+
const hasPerm = false;
263+
const result = chooseIDE("not-exists", ideOptions, useLatest, hasPerm);
264+
expect(result.ideImage).to.equal(ideOptions.options["code"].latestImage);
265+
});
252266
});
253267
});

components/server/src/workspace/workspace-starter.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,7 @@ export const chooseIDE = (
156156
const data: { desktopIdeImage?: string; ideImage: string } = {
157157
ideImage: defaultIdeImage,
158158
};
159-
const chooseOption = ideOptions.options[ideChoice];
159+
const chooseOption = ideOptions.options[ideChoice] ?? defaultIDEOption;
160160
const isDesktopIde = chooseOption.type === "desktop";
161161
if (isDesktopIde) {
162162
data.desktopIdeImage = useLatest ? chooseOption?.latestImage ?? chooseOption?.image : chooseOption?.image;

0 commit comments

Comments
 (0)