Remember the choice when asking project selection (#3428)

- Remember the choice when asking project selection
- Update @vscode/test-electron to fix macos ci

Signed-off-by: Sheng Chen <sheche@microsoft.com>

Author: jdneo

package-lock.json

@@ -1678,7 +1678,7 @@
     "@types/vscode": "^1.77.0",
     "@types/winreg": "^1.2.30",
     "@types/winston": "^2.4.4",
-    "@vscode/test-electron": "^2.1.5",
+    "@vscode/test-electron": "^2.3.8",
     "@typescript-eslint/eslint-plugin": "^5.18.0",
     "@typescript-eslint/parser": "^5.18.0",
     "css-loader": "^6.7.3",

package.json

@@ -5,6 +5,7 @@ import { IBuildTool, getContributedBuildTools } from "./plugin";
 
 export const PICKED_BUILD_FILES = "java.pickedBuildFiles";
 export const BUILD_TOOL_FOR_CONFLICTS = "java.buildToolForConflicts";
+export const IMPORT_METHOD = "java.importMethod";
 export class BuildFileSelector {
 	/**
 	 * The build tools that are contributed/supported by extensions.
@@ -295,4 +296,5 @@ interface IBuildFilePicker extends QuickPickItem {
 export function cleanupProjectPickerCache(context: ExtensionContext) {
 	context.workspaceState.update(PICKED_BUILD_FILES, undefined);
 	context.workspaceState.update(BUILD_TOOL_FOR_CONFLICTS, undefined);
+	context.workspaceState.update(IMPORT_METHOD, undefined);
 }

src/buildFilesSelector.ts

@@ -8,7 +8,7 @@ import { cleanupLombokCache } from './lombokSupport';
 import { ensureExists, getJavaConfiguration } from './utils';
 import { apiManager } from './apiManager';
 import { isActive, setActive, smartSemicolonDetection } from './smartSemicolonDetection';
-import { BuildFileSelector, PICKED_BUILD_FILES } from './buildFilesSelector';
+import { BuildFileSelector, IMPORT_METHOD, PICKED_BUILD_FILES } from './buildFilesSelector';
 
 const DEFAULT_HIDDEN_FILES: string[] = ['**/.classpath', '**/.project', '**/.settings', '**/.factorypath'];
 const IS_WORKSPACE_JDK_ALLOWED = "java.ls.isJdkAllowed";
@@ -370,6 +370,11 @@ export function handleTextDocumentChanges(document: TextDocument, changes: reado
 export async function getImportMode(context: ExtensionContext, selector: BuildFileSelector): Promise<ImportMode> {
 	const mode = getJavaConfiguration().get<string>("import.projectSelection");
 	if (mode === "manual") {
+		// use automatic mode if user has selected "Import All" before.
+		if (context.workspaceState.get(IMPORT_METHOD) === "Import All") {
+			return ImportMode.automatic;
+		}
+
 		// if no selectable build files, use automatic mode
 		const hasBuildFiles = await selector.hasBuildFiles();
 		if (!hasBuildFiles) {
@@ -386,6 +391,7 @@ export async function getImportMode(context: ExtensionContext, selector: BuildFi
 			{ modal: true },
 			"Import All", "Let Me Select...");
 		if (answer === "Import All") {
+			context.workspaceState.update(IMPORT_METHOD, "Import All");
 			return ImportMode.automatic;
 		} else if (answer === "Let Me Select...") {
 			return ImportMode.manual;