blindspot on RemoveUnusedImportsStep

[Pankraz76]

fix #1261

@iddeepak kindly request your eyes on this.

• execute and debug some test locally on IDE?
• maybe work this out?
  • tc exposed some issues

[ERROR] Failures: 
[ERROR]   RemoveUnusedImportsTest.removeUnused:485 value of:
    removeUnusedImports(...)
expected:
    interface Test { private static void foo() {} }
    
    
but was:
    import java.lang.Foo;
    interface Test { private static void foo() {} }
    
    
[ERROR]   RemoveUnusedImportsTest.removeUnused:485 value of:
    removeUnusedImports(...)
expected:
    …pkg.Constants.FOO;
    
    public class Test …
but was:
    …pkg.Constants.FOO;
    
    
    public class Test …
[ERROR]   RemoveUnusedImportsTest.removeUnused:485 value of:
    removeUnusedImports(...)
diff (-expected +actual):
    @@ -1,3 +1,4 @@
    +import static java.util.Collections.*;
     import static java.util.Collections.emptyList;
     
     class Test {
[ERROR]   RemoveUnusedImportsTest.removeUnused:485 value of:
    removeUnusedImports(...)
expected:
    class Test {
      java.…
but was:
    
    class Test {
      java.…
[ERROR]   RemoveUnusedImportsTest.removeUnused:485 value of:
    removeUnusedImports(...)
diff (-expected +actual):
    @@ -1,3 +1,4 @@
    +import static java.lang.Math.*;
     import static java.lang.Math.PI;
     
     class Test {
[ERROR]   RemoveUnusedImportsTest.removeUnused:485 value of:
    removeUnusedImports(...)
expected:
    // This is a comment mentioning ArrayList
    class Test {}
    
    
but was:
    
    // This is a comment mentioning ArrayList
    class Test {}
    
    
[ERROR]   RemoveUnusedImportsTest.removeUnused:485 value of:
    removeUnusedImports(...)
diff (-expected +actual):
    @@ -1,5 +1,6 @@
     package com.foo;
     import static com.foo.Outer.A;
    +import com.foo.*;
     import com.foo.B;
     import com.bar.C;
     class Test {
[INFO] 
[ERROR] Tests run: 1580, Failures: 7, Errors: 0, Skipped: 0

[google-cla]

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

[Pankraz76]

WIP

[Pankraz76]

item:
java: exporting a package from system module jdk.compiler is not allowed with --release

unable to run in ide, therefore unable to debug:

feel free to take over, thx.

[iddeepak]

yeah, same here individually not able to run

[Pankraz76]

item:
java: exporting a package from system module jdk.compiler is not allowed with --release

unable to run in ide, therefore unable to debug:

feel free to take over, thx.

[iddeepak]

java.lang.Deprecated is in the implicitly imported package.
https://www.jetbrains.com/help/inspectopedia/JavaLangImport.html

[iddeepak]

you writes java.lang.Foo fully qualified.