Fix NPE issues in telemetry

PluginsAndFeatures/azure-toolkit-for-intellij/src/com/microsoft/intellij/actions/functions/DeployFunctionAction.java

@@ -74,6 +74,10 @@ public void update(AnActionEvent event) {
     }
 
     private void runConfiguration(Module module) {
+        // todo: investigate when will module be null
+        if (module == null) {
+            return;
+        }
         final Project project = module.getProject();
         final RunManagerEx manager = RunManagerEx.getInstanceEx(project);
         final ConfigurationFactory factory = new FunctionDeploymentConfigurationFactory(configType);

PluginsAndFeatures/azure-toolkit-for-intellij/src/com/microsoft/intellij/actions/functions/RunFunctionAction.java

@@ -64,6 +64,10 @@ public void update(AnActionEvent event) {
     }
 
     private void runConfiguration(Module module) {
+        // todo: investigate when will module be null
+        if (module == null) {
+            return;
+        }
         final Project project = module.getProject();
         final RunManagerEx manager = RunManagerEx.getInstanceEx(project);
         final ConfigurationFactory factory = new FunctionRunConfigurationFactory(configType);

PluginsAndFeatures/azure-toolkit-for-intellij/src/com/microsoft/intellij/runner/functions/core/FunctionUtils.java

@@ -198,7 +198,7 @@ public static Map<String, FunctionConfiguration> prepareStagingFolder(Path stagi
 
         final List<File> jarFiles = new ArrayList<>();
         OrderEnumerator.orderEntries(module).productionOnly().forEachLibrary(lib -> {
-            if (ArrayUtils.contains(lib.getName().split("\\:"), FUNCTION_JAVA_LIBRARY_ARTIFACT_ID)) {
+            if (StringUtils.isNotEmpty(lib.getName()) && ArrayUtils.contains(lib.getName().split("\\:"), FUNCTION_JAVA_LIBRARY_ARTIFACT_ID)) {
                 return true;
             }
 
@@ -337,6 +337,9 @@ private static Binding getBinding(final Project project, JvmAnnotation annotatio
 
     private static Binding getUserDefinedBinding(final Project project, PsiAnnotation annotation) throws AzureExecutionException {
         PsiJavaCodeReferenceElement referenceElement = annotation.getNameReferenceElement();
+        if (referenceElement == null) {
+            return null;
+        }
         PsiAnnotation customBindingAnnotation =
                 AnnotationUtil.findAnnotation((PsiModifierListOwner) referenceElement.resolve(),
                                               AZURE_FUNCTION_CUSTOM_BINDING_CLASS);
@@ -427,7 +430,7 @@ private static void copyFilesWithDefaultContent(Path sourcePath, File dest, Stri
         if (src != null && src.exists()) {
             FileUtils.copyFile(src, dest);
         } else {
-            FileUtils.write(src, defaultContent, Charset.defaultCharset());
+            FileUtils.write(dest, defaultContent, Charset.defaultCharset());
         }
     }
 

PluginsAndFeatures/azure-toolkit-for-intellij/src/com/microsoft/intellij/runner/functions/deploy/FunctionDeploymentState.java

@@ -51,6 +51,9 @@
 
 public class FunctionDeploymentState extends AzureRunProfileState<WebAppBase> {
 
+    private static final String TARGET_FUNCTION_DOES_NOT_EXIST =
+            "Target function does not exist, please select a valid function in function deployment run configuration.";
+
     private FunctionDeployConfiguration functionDeployConfiguration;
     private final FunctionDeployModel deployModel;
     private File stagingFolder;
@@ -72,6 +75,9 @@ public WebAppBase executeSteps(@NotNull RunProcessHandler processHandler
         // Update run time information by function app
         final FunctionApp functionApp = AzureFunctionMvpModel.getInstance()
                 .getFunctionById(functionDeployConfiguration.getSubscriptionId(), functionDeployConfiguration.getFunctionId());
+        if (functionApp == null) {
+            throw new AzureExecutionException(TARGET_FUNCTION_DOES_NOT_EXIST);
+        }
         final AppServicePlan appServicePlan = AppServiceUtils.getAppServicePlanByAppService(functionApp);
         final IntelliJFunctionRuntimeConfiguration runtimeConfiguration = new IntelliJFunctionRuntimeConfiguration();
         runtimeConfiguration.setOs(appServicePlan.operatingSystem() == OperatingSystem.WINDOWS ? "windows" : "linux");

PluginsAndFeatures/azure-toolkit-for-intellij/src/com/microsoft/intellij/runner/functions/library/function/DeployFunctionHandler.java

@@ -43,7 +43,7 @@
 import com.microsoft.azure.functions.annotation.AuthorizationLevel;
 import com.microsoft.azure.management.appservice.FunctionApp;
 import com.microsoft.azure.management.appservice.FunctionApp.Update;
-import com.microsoft.intellij.runner.functions.library.IAppServiceContext;
+import com.microsoft.intellij.runner.functions.deploy.FunctionDeployModel;
 import com.microsoft.intellij.runner.functions.library.IPrompter;
 import org.apache.commons.collections4.CollectionUtils;
 import org.apache.commons.lang3.StringUtils;
@@ -93,17 +93,16 @@ public class DeployFunctionHandler {
             + "function information (Attempt %d/%d)...";
 
     private static final OperatingSystemEnum DEFAULT_OS = OperatingSystemEnum.Windows;
-    private IAppServiceContext ctx;
+    private FunctionDeployModel ctx;
     private IPrompter prompter;
 
-    public DeployFunctionHandler(IAppServiceContext ctx, IPrompter prompter) {
+    public DeployFunctionHandler(FunctionDeployModel ctx, IPrompter prompter) {
         Preconditions.checkNotNull(ctx);
         this.ctx = ctx;
         this.prompter = prompter;
     }
 
     public FunctionApp execute() throws Exception {
-
         final FunctionApp app = getFunctionApp();
         updateFunctionAppSettings(app);
         final DeployTarget deployTarget = new DeployTarget(app, DeployTargetType.FUNCTION);
@@ -220,14 +219,12 @@ private boolean isDedicatedPricingTier() {
         return AppServiceUtils.getPricingTierFromString(ctx.getPricingTier()) != null;
     }
 
-    private FunctionApp getFunctionApp() {
+    private FunctionApp getFunctionApp() throws AzureExecutionException {
         try {
-            return ctx.getAzureClient().appServices().functionApps().getByResourceGroup(ctx.getResourceGroup(),
-                    ctx.getAppName());
-        } catch (Exception ex) {
-            // Swallow exception for non-existing Azure Functions
+            return ctx.getAzureClient().appServices().functionApps().getById(ctx.getFunctionId());
+        } catch (IOException e) {
+            throw new AzureExecutionException("Failed to get azure client");
         }
-        return null;
     }
 
     // region get App Settings

Utils/azuretools-core/src/com/microsoft/azuretools/sdkmanage/AzureCliAzureManager.java

@@ -182,7 +182,7 @@ private static class LazyLoader {
      * identity/implementation/IdentityClient.java#L366
      */
     private Pair<String, OffsetDateTime> getAccessTokenViaCli(String tid, @Nullable String resource) throws IOException {
-        if (!PATTERN_TENANT.matcher(tid).matches()) {
+        if (StringUtils.isEmpty(tid) || !PATTERN_TENANT.matcher(tid).matches()) {
             throw new InvalidParameterException(String.format("[%s] is not a valid tenant ID", tid));
         } else if (StringUtils.isNotEmpty(resource) && !PATTERN_RESOURCE.matcher(resource).matches()) {
             throw new InvalidParameterException(String.format("[%s] is not a valid resource endpoint", resource));