Restore ignored tests (#5826)

* Restore ignored tests

* debug

* debug

* debug

* debug

* debug

* debug

* debug

* debug

* debug

* debug

* debug

* debug

* debug

* debug

* debug

* debug

* debug

* debug

* debug

* debug

* debug

* debug

Author: stevemessick

.gitignore

@@ -20,7 +20,6 @@ lib/jxbrowser/*
 !lib/java-string-similarity-2.0.0.jar
 material-design-icons/
 out/
-packages
 pubspec.lock
 releases/
 resources/jxbrowser/jxbrowser.properties

build.gradle

@@ -83,6 +83,9 @@ dependencies {
 }
 
 sourceSets {
+  test.resources.srcDirs = [
+    "flutter-idea/testData"
+  ]
 }
 
 //noinspection GroovyAssignabilityCheck

docs/building.md

@@ -36,7 +36,8 @@ project in IntelliJ, then delete .idea/modules.xml. You may need to delete other
 files in ~/Library/Application Support/Caches/JetBrains/<IDE>/. Look in `conversion`
 and `external_build_systrm`. It may take some experimentation. When the project
 is opened in IntelliJ, ensure that there are `flutter-idea` and `flutter-studio`
-modules in the project structure.
+modules in the project structure. Also make sure the project sdk is set to a
+JDK-equivalent, not a JRE-equivalent.
 
 To re-open the project as an IntelliJ (not Gradle) project, close it in IntelliJ,
 then restore .idea/modules.xml and delete .idea/gradle.xml. Again, you may need to

flutter-idea/testData

@@ -6,11 +6,13 @@
 package io.flutter.editor;
 
 import com.intellij.codeInsight.lookup.LookupElementBuilder;
+import com.intellij.openapi.application.ApplicationManager;
 import com.intellij.openapi.project.Project;
 import com.intellij.util.ui.ColorIcon;
 import com.intellij.util.ui.EmptyIcon;
 import com.jetbrains.lang.dart.ide.completion.DartCompletionExtension;
 import com.jetbrains.lang.dart.ide.completion.DartServerCompletionContributor;
+import io.flutter.sdk.FlutterSdk;
 import org.apache.commons.lang.StringUtils;
 import org.dartlang.analysis.server.protocol.CompletionSuggestion;
 import org.dartlang.analysis.server.protocol.Element;
@@ -60,12 +62,12 @@ else if (Objects.equals(declaringType, "CupertinoColors")) {
             }
           }
           else if (Objects.equals(declaringType, "Icons")) {
-            final Icon icon = FlutterIconLineMarkerProvider.getMaterialIconByName(project, name);
+            final Icon icon = FlutterIconLineMarkerProvider.getMaterialIconByName(project, getSdkHomePath(project), name);
             // If we have no icon, show an empty node (which is preferable to the default "IconData" text).
             return icon != null ? icon : EMPTY_ICON;
           }
           else if (Objects.equals(declaringType, "CupertinoIcons")) {
-            final Icon icon = FlutterIconLineMarkerProvider.getCupertinoIconByName(project, name);
+            final Icon icon = FlutterIconLineMarkerProvider.getCupertinoIconByName(project, getSdkHomePath(project), name);
             // If we have no icon, show an empty node (which is preferable to the default "IconData" text).
             return icon != null ? icon : EMPTY_ICON;
           }
@@ -75,4 +77,17 @@ else if (Objects.equals(declaringType, "CupertinoIcons")) {
 
     return null;
   }
+
+  @NotNull
+  private static String getSdkHomePath(@NotNull Project project) {
+    final FlutterSdk sdk = FlutterSdk.getFlutterSdk(project);
+    if (sdk == null) {
+      assert ApplicationManager.getApplication() != null;
+      if (ApplicationManager.getApplication().isUnitTestMode()) {
+        return "testData/sdk";
+      }
+      throw new NullPointerException("Flutter SDK not found");
+    }
+    return sdk.getHomePath();
+  }
 }

testData/coverage/lcov.info renamed to flutter-idea/testData/coverage/lcov.info

@@ -42,6 +42,7 @@
 import org.jetbrains.yaml.psi.YamlRecursivePsiElementVisitor;
 
 import javax.swing.*;
+import java.io.FileNotFoundException;
 import java.util.*;
 
 import static io.flutter.dart.DartPsiUtil.*;
@@ -75,28 +76,24 @@ public static void initialize() {
   }
 
   @Nullable
-  public static Icon getCupertinoIconByName(@Nullable Project project, @NotNull String iconName) {
+  public static Icon getCupertinoIconByName(@Nullable Project project, @NotNull String sdkHomePath, @NotNull String iconName) {
     if (project == null) return null;
-    final FlutterSdk sdk = FlutterSdk.getFlutterSdk(project);
-    if (sdk == null) return null;
     final IconInfo iconDef =
-      findStandardDefinition("CupertinoIcons", iconName, project, sdk.getHomePath() + BuiltInPaths.get("CupertinoIcons"), sdk);
+      findStandardDefinition("CupertinoIcons", iconName, project, sdkHomePath + BuiltInPaths.get("CupertinoIcons"), sdkHomePath);
     if (iconDef == null) return null;
     final String path = FlutterSdkUtil.getPathToCupertinoIconsPackage(project);
     // <pub_cache>/hosted/pub.dartlang.org/cupertino_icons-v.m.n/assets/CupertinoIcons.ttf
     return findStandardIconFromDef(iconName, iconDef, path + CupertinoRelativeAssetPath);
   }
 
   @Nullable
-  public static Icon getMaterialIconByName(@Nullable Project project, @NotNull String iconName) {
+  public static Icon getMaterialIconByName(@Nullable Project project, @NotNull String sdkHomePath, @NotNull String iconName) {
     if (project == null) return null;
-    final FlutterSdk sdk = FlutterSdk.getFlutterSdk(project);
-    if (sdk == null) return null;
     final IconInfo iconDef =
-      findStandardDefinition("Icons", iconName, project, sdk.getHomePath() + BuiltInPaths.get("Icons"), sdk);
+      findStandardDefinition("Icons", iconName, project, sdkHomePath + BuiltInPaths.get("Icons"), sdkHomePath);
     if (iconDef == null) return null;
     // <flutter-sdk>/bin/cache/artifacts/material_fonts/MaterialIcons-Regular.otf
-    return findStandardIconFromDef(iconName, iconDef, sdk.getHomePath() + MaterialRelativeAssetPath);
+    return findStandardIconFromDef(iconName, iconDef, sdkHomePath + MaterialRelativeAssetPath);
   }
 
   @Nullable
@@ -198,10 +195,10 @@ else if (parentNode.getElementType() == DartTokenTypes.SIMPLE_TYPE) {
         final String selector = AstBufferUtil.getTextSkippingWhitespaceComments(selectorNode.getNode());
         final Icon icon;
         if (name.equals("Icons")) {
-          icon = getMaterialIconByName(element.getProject(), selector);
+          icon = getMaterialIconByName(element.getProject(), sdk.getHomePath(), selector);
         }
         else if (name.equals("CupertinoIcons")) {
-          icon = getCupertinoIconByName(element.getProject(), selector);
+          icon = getCupertinoIconByName(element.getProject(), sdk.getHomePath(), selector);
         }
         else {
           // Note: I want to keep this code until I'm sure we won't use pubspec.yaml.
@@ -254,12 +251,12 @@ private static IconInfo findStandardDefinition(@NotNull String className,
                                                  @NotNull String iconName,
                                                  @NotNull Project project,
                                                  @Nullable String path,
-                                                 @NotNull FlutterSdk sdk) {
+                                                 @NotNull String sdkHomePath) {
     if (path != null) {
       return findDefinition(className, iconName, project, path);
     }
     assert Objects.requireNonNull(ApplicationManager.getApplication()).isUnitTestMode();
-    return findDefinition(className, iconName, project, sdk.getHomePath() + BuiltInPaths.get(className));
+    return findDefinition(className, iconName, project, sdkHomePath + BuiltInPaths.get(className));
   }
 
   @Nullable
@@ -313,10 +310,10 @@ private static IconInfo findDefinition(@NotNull String className,
                                          @NotNull String path) {
     assert LocalFileSystem.getInstance() != null;
     final VirtualFile virtualFile = LocalFileSystem.getInstance().findFileByPath(path);
-    if (virtualFile == null) return null;
+    if (virtualFile == null) throw new Error("FILE NOT FOUND: " + path);//return null;
     final PsiFile psiFile = PsiManager.getInstance(project).findFile(virtualFile);
     if (psiFile == null) {
-      return null;
+      throw new Error("CANNOT CREATE PSI FILE");//return null;
     }
     final IconInfoVisitor visitor = new IconInfoVisitor(iconName);
     psiFile.accept(visitor);
@@ -327,7 +324,7 @@ private static IconInfo findDefinition(@NotNull String className,
   private static Icon findIconFromDef(@NotNull String iconClassName, @NotNull IconInfo iconDef, @NotNull String path) {
     assert LocalFileSystem.getInstance() != null;
     final VirtualFile virtualFile = LocalFileSystem.getInstance().findFileByPath(path);
-    if (virtualFile == null) return null;
+    if (virtualFile == null) throw new Error("FILE NOT FOUND: " + path);//return null;
     VirtualFile parent = virtualFile;
     while (parent != null && !parent.getName().equals("lib")) {
       parent = parent.getParent();

testData/sample_tests/bin/pretty_print.dart renamed to flutter-idea/testData/sample_tests/bin/pretty_print.dart

@@ -295,6 +295,11 @@ public static String guessFlutterSdkFromPackagesFile(@NotNull Module module) {
 
   @Nullable
   public static String getPathToCupertinoIconsPackage(@NotNull Project project) {
+    //noinspection ConstantConditions
+    if (ApplicationManager.getApplication().isUnitTestMode()) {
+      // TODO(messick): Configure the test framework to have proper pub data so we don't need this.
+      return "testData/sdk";
+    }
     final JsonArray packages = getPackagesFromPackageConfig(PubRoots.forProject(project));
     for (int i = 0; i < packages.size(); i++) {
       final JsonObject pack = packages.get(i).getAsJsonObject();

testData/sample_tests/pubspec.yaml renamed to flutter-idea/testData/sample_tests/pubspec.yaml

@@ -38,6 +38,10 @@ public static ColorKey getEditorNotificationBackgroundColor() {
     return EditorColors.GUTTER_BACKGROUND;
   }
 
+  /**
+   * Unstable API warning: only use this in the deprecated inspector.
+   * @return best-guess for the current project based on the top-most window.
+   */
   @Nullable
   public static Project findVisibleProject() {
     final WindowManager wm = WindowManager.getInstance();

testData/sample_tests/test/custom_outline.txt renamed to flutter-idea/testData/sample_tests/test/custom_outline.txt

@@ -0,0 +1 @@
+flutter-idea/testData

testData/sample_tests/test/custom_test.dart renamed to flutter-idea/testData/sample_tests/test/custom_test.dart

@@ -15,6 +15,8 @@
 import io.flutter.sdk.FlutterSdk;
 import org.junit.Test;
 
+import java.io.IOException;
+
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
 
@@ -27,17 +29,17 @@ private FlutterSdk getSdk() {
     return mockSdk;
   }
 
-  //@Test
-  public void xtestLocatesIconsReference() throws Exception {
+  @Test
+  public void testLocatesIconsReference() throws Exception {
     final PsiElement testIdentifier = setUpDartElement("main() { Icons.access_alarm; }", "Icons", LeafPsiElement.class);
     final LineMarkerInfo<?> marker = new FlutterIconLineMarkerProvider().getLineMarkerInfo(testIdentifier, getSdk());
     assertNotNull(marker);
     final DartReferenceExpression element = DartSyntax.findEnclosingReferenceExpression(testIdentifier);
     assertNotNull(element);
   }
 
-  //@Test
-  public void xtestLocatesIconCtor() throws Exception {
+  @Test
+  public void testLocatesIconCtor() throws Exception {
     final PsiElement testIdentifier =
       setUpDartElement("main() { IconData(0xe190, fontFamily: 'MaterialIcons'); }", "IconData", LeafPsiElement.class);
     final LineMarkerInfo<?> marker = new FlutterIconLineMarkerProvider().getLineMarkerInfo(testIdentifier, getSdk());
@@ -46,8 +48,8 @@ public void xtestLocatesIconCtor() throws Exception {
     assertNotNull(element);
   }
 
-  //@Test
-  public void xtestLocatesCupertinoIconCtor() throws Exception {
+  @Test
+  public void testLocatesCupertinoIconCtor() throws Exception {
     final PsiElement testIdentifier =
       setUpDartElement("main() { IconData(0xe190, fontFamily: 'CupertinoIcons'); }", "IconData", LeafPsiElement.class);
     final LineMarkerInfo<?> marker = new FlutterIconLineMarkerProvider().getLineMarkerInfo(testIdentifier, getSdk());
@@ -56,8 +58,8 @@ public void xtestLocatesCupertinoIconCtor() throws Exception {
     assertNotNull(element);
   }
 
-  //@Test
-  public void xtestLocatesConstIconCtor() throws Exception {
+  @Test
+  public void testLocatesConstIconCtor() throws Exception {
     final PsiElement testIdentifier =
       setUpDartElement("main() { const IconData(0xe190, fontFamily: 'MaterialIcons'); }", "IconData", LeafPsiElement.class);
     final LineMarkerInfo<?> marker = new FlutterIconLineMarkerProvider().getLineMarkerInfo(testIdentifier, getSdk());
@@ -66,17 +68,17 @@ public void xtestLocatesConstIconCtor() throws Exception {
     assertNotNull(element);
   }
 
-  //@Test
-  public void xtestLocatesCupertinoIconsReference() throws Exception {
+  @Test
+  public void testLocatesCupertinoIconsReference() throws Exception {
     final PsiElement testIdentifier = setUpDartElement("main() { CupertinoIcons.book; }", "CupertinoIcons", LeafPsiElement.class);
     final LineMarkerInfo<?> marker = new FlutterIconLineMarkerProvider().getLineMarkerInfo(testIdentifier, getSdk());
     assertNotNull(marker);
     final DartReferenceExpression element = DartSyntax.findEnclosingReferenceExpression(testIdentifier);
     assertNotNull(element);
   }
 
-  //@Test
-  public void xtestLocatesCupertinoIconsReferenceWithComment() throws Exception {
+  @Test
+  public void testLocatesCupertinoIconsReferenceWithComment() throws Exception {
     final PsiElement testIdentifier =
       setUpDartElement("main() { CupertinoIcons . /* a book */ book; }", "CupertinoIcons", LeafPsiElement.class);
     final LineMarkerInfo<?> marker = new FlutterIconLineMarkerProvider().getLineMarkerInfo(testIdentifier, getSdk());

testData/sample_tests/test/no_tests.dart renamed to flutter-idea/testData/sample_tests/test/no_tests.dart

@@ -67,6 +67,11 @@ elif [ "CHECK_BOT" = "$BOT" ] ; then
 
 elif [ "UNIT_TEST_BOT" = "$BOT" ] ; then
 
+echo `pwd`
+echo "ls -lR"
+ls -lR > dir-list.txt
+cat dir-list.txt
+
   # Run unit tests.
   ./bin/plugin test
 

testData/sample_tests/test/no_tests_outline.txt renamed to flutter-idea/testData/sample_tests/test/no_tests_outline.txt

@@ -28,6 +28,7 @@ setup() {
   export FLUTTER_KEYSTORE_ID=74840
   export FLUTTER_KEYSTORE_NAME=flutter-intellij-plugin-auth-token
   export FLUTTER_KEYSTORE_JXBROWSER_KEY_NAME=flutter-intellij-plugin-jxbrowser-license-key
+  export NO_FS_ROOTS_ACCESS_CHECK=true
 
   (cd tool/plugin; echo "pub get `pwd`"; pub get --no-precompile)
   ./gradlew --version

testData/sample_tests/test/simple_outline.txt renamed to flutter-idea/testData/sample_tests/test/simple_outline.txt

@@ -3,6 +3,12 @@
 source ./tool/kokoro/setup.sh
 setup
 
+curl https://download.java.net/java/GA/jdk11/9/GPL/openjdk-11.0.2_osx-x64_bin.tar.gz > ../java.tar.gz
+(cd ..; tar fx java.tar.gz)
+export JAVA_HOME=`pwd`/../jdk-11.0.2.jdk/Contents/Home
+export PATH=$PATH:$JAVA_HOME/bin
+echo "JAVA_HOME=$JAVA_HOME"
+
 (cd testData/sample_tests; echo "pub get `pwd`"; pub get --no-precompile)
 
 echo "kokoro test start"