Update based on review comments

Author: t-rufang

PluginsAndFeatures/azure-toolkit-for-intellij/src/com/microsoft/azure/hdinsight/spark/ui/filesystem/ADLSGen2FileSystem.java

@@ -31,7 +31,6 @@
 import com.microsoft.azure.hdinsight.spark.common.ADLSGen2Deploy;
 import com.microsoft.azuretools.azurecommons.helpers.NotNull;
 import com.microsoft.azuretools.azurecommons.helpers.Nullable;
-import org.apache.commons.lang3.StringUtils;
 import rx.Observable;
 
 import java.io.IOException;
@@ -63,8 +62,8 @@ public String getProtocol() {
     public VirtualFile[] listFiles(AdlsGen2VirtualFile vf) {
         List<AdlsGen2VirtualFile> childrenList = new ArrayList<>();
         if (vf.isDirectory()) {
-            // sample fileSystemRootPath: https://accountName.dfs.core.windows.net/fileSystem
-            String fileSystemRootPath = StringUtils.stripEnd(rootPathUri.resolve("/").getUrl().toString(), "/");
+            // sample fileSystemRootPath: https://accountName.dfs.core.windows.net/fileSystem/
+            String fileSystemRootPath = rootPathUri.resolve("/").getUrl().toString();
             // sample directoryParam: sub/path/to
             String directoryParam = vf.getAbfsUri().getDirectoryParam();
             childrenList = this.op.list(fileSystemRootPath, directoryParam)
@@ -78,7 +77,7 @@ public VirtualFile[] listFiles(AdlsGen2VirtualFile vf) {
                     .onErrorResumeNext(err -> {
                                 String errorMessage = "Failed to list folders and files with error " + err.getMessage() + ". ";
                                 if (err instanceof ForbiddenHttpErrorStatus) {
-                                    errorMessage += ADLSGen2Deploy.getForbiddenErrorHints(fileSystemRootPath);
+                                    errorMessage += ADLSGen2Deploy.getForbiddenErrorHints(vf.toString());
                                 }
                                 return Observable.error(new IOException(errorMessage));
                             }

Utils/hdinsight-node-common/src/com/microsoft/azure/hdinsight/sdk/storage/adlsgen2/ADLSGen2FSOperation.java

@@ -26,6 +26,7 @@
 import com.microsoft.azure.hdinsight.sdk.rest.azure.storageaccounts.RemoteFile;
 import com.microsoft.azure.hdinsight.sdk.rest.azure.storageaccounts.api.GetRemoteFilesResponse;
 import com.microsoft.azuretools.azurecommons.helpers.NotNull;
+import org.apache.commons.lang3.StringUtils;
 import org.apache.http.NameValuePair;
 import org.apache.http.client.methods.HttpPatch;
 import org.apache.http.client.methods.HttpPut;
@@ -99,7 +100,11 @@ public Observable<RemoteFile> list(String rootPath, String relativePath) {
                 .enableRecursive(false)
                 .setResource("filesystem");
 
-        return http.get(rootPath, listReqBuilder.setDirectory(relativePath).build(), null, GetRemoteFilesResponse.class)
+        return http.get(
+                StringUtils.stripEnd(rootPath, "/"),
+                listReqBuilder.setDirectory(relativePath).build(),
+                null,
+                GetRemoteFilesResponse.class)
                 .flatMap(pathList -> Observable.from(pathList.getRemoteFiles()));
     }