fix: 🐛 NPE when finishing Snyk Wizard in empty workspace

SnykWizard was triggering authentication and trust on finishing without checking
the workspace for projects. This leads to a NPE, as our language server client
is still uninitialized. LSP4e only initializes the language client and starts the
language server, when a project / file is associated with the corresponding
mime/media type.

This fix wraps authentication and trust into a condition, checking, if the workspace
contains projects.

Author: bastiandoetsch

plugin/src/main/java/io/snyk/eclipse/plugin/wizards/SnykWizard.java

@@ -1,5 +1,9 @@
 package io.snyk.eclipse.plugin.wizards;
 
+import java.util.Arrays;
+
+import org.eclipse.core.resources.IProject;
+import org.eclipse.core.resources.ResourcesPlugin;
 import org.eclipse.jface.viewers.IStructuredSelection;
 import org.eclipse.jface.wizard.Wizard;
 import org.eclipse.ui.INewWizard;
@@ -9,55 +13,59 @@
 import io.snyk.languageserver.protocolextension.SnykExtendedLanguageClient;
 
 public class SnykWizard extends Wizard implements INewWizard {
-  protected SnykWizardConfigureAPIPage configureAPIPage;
-  protected SnykWizardAuthenticatePage authenticatePage;
-
-  protected SnykWizardModel model;
-
-  protected IWorkbench workbench;
-  protected IStructuredSelection selection;
-
-  public SnykWizard() {
-    super();
-    model = new SnykWizardModel();
-    setNeedsProgressMonitor(true);
-  }
-
-  @Override
-  public String getWindowTitle() {
-    return "Snyk Wizard";
-  }
-
-  @Override
-  public void addPages() {
-    configureAPIPage = new SnykWizardConfigureAPIPage();
-    addPage(configureAPIPage);
-
-    authenticatePage = new SnykWizardAuthenticatePage();
-    addPage(authenticatePage);
-  }
-
-  public void init(IWorkbench workbench, IStructuredSelection selection) {
-    this.workbench = workbench;
-    this.selection = selection;
-  }
-
-  public boolean canFinish() {
-    if (this.getContainer().getCurrentPage() == authenticatePage) {
-      return true;
-    }
-    return false;
-  }
-
-  public boolean performCancel() {
-    model.resetPreferences();
-    return true;
-  }
-
-  public boolean performFinish() {
-    new LsConfigurationUpdater().configurationChanged();
-    SnykExtendedLanguageClient.getInstance().triggerAuthentication();
-    SnykExtendedLanguageClient.getInstance().trustWorkspaceFolders();
-    return true;
-  }
+	protected SnykWizardConfigureAPIPage configureAPIPage;
+	protected SnykWizardAuthenticatePage authenticatePage;
+
+	protected SnykWizardModel model;
+
+	protected IWorkbench workbench;
+	protected IStructuredSelection selection;
+
+	public SnykWizard() {
+		super();
+		model = new SnykWizardModel();
+		setNeedsProgressMonitor(true);
+	}
+
+	@Override
+	public String getWindowTitle() {
+		return "Snyk Wizard";
+	}
+
+	@Override
+	public void addPages() {
+		configureAPIPage = new SnykWizardConfigureAPIPage();
+		addPage(configureAPIPage);
+
+		authenticatePage = new SnykWizardAuthenticatePage();
+		addPage(authenticatePage);
+	}
+
+	public void init(IWorkbench workbench, IStructuredSelection selection) {
+		this.workbench = workbench;
+		this.selection = selection;
+	}
+
+	public boolean canFinish() {
+		if (this.getContainer().getCurrentPage() == authenticatePage) {
+			return true;
+		}
+		return false;
+	}
+
+	public boolean performCancel() {
+		model.resetPreferences();
+		return true;
+	}
+
+	public boolean performFinish() {
+		new LsConfigurationUpdater().configurationChanged();
+		var projects = ResourcesPlugin.getWorkspace().getRoot().getProjects();
+		if (projects != null && projects.length > 0
+				&& Arrays.stream(projects).filter(p -> p.isAccessible()).count() > 0) {
+			SnykExtendedLanguageClient.getInstance().triggerAuthentication();
+			SnykExtendedLanguageClient.getInstance().trustWorkspaceFolders();
+		}
+		return true;
+	}
 }