fix RemoveUnusedImportsStep leftovers

[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

[iddeepak]

@Pankraz76
I have pushed a few small tweaks—feel free to pull them into your branch and let me know what you think or if anything needs further adjustment PR #1266
Thank you!

[Pankraz76]

this seems to be an issue now imposed by the new fixes done by me.

[Pankraz76]

items:

so the formatter test failing on the previous version is fixed now, but still failing other tests as well.

At least only related to the class, except of the IndexOutOfBounds.

[ERROR] Failures: 
[ERROR]   RemoveUnusedImportsTest.removeUnused:577 value of:
    removeUnusedImports(...)
diff (-expected +actual):
    @@ -1,5 +1,4 @@
     package p;
    -import java.lang.Foo;
     import java.lang.Foo.Bar;
     import p.Baz;
     import p.Baz.Bork;
[ERROR]   RemoveUnusedImportsTest.removeUnused:577 value of:
    removeUnusedImports(...)
expected:
    …pkg.Constants.FOO;
    
    public class Test …
but was:
    …pkg.Constants.FOO;
    
    
    public class Test …
[ERROR]   RemoveUnusedImportsTest.removeUnused:577 value of:
    removeUnusedImports(...)
expected:
    class Test {
      java.…
but was:
    
    class Test {
      java.…
[ERROR] Errors: 
[ERROR]   MainTest.optimizeImportsDoesNotLeaveEmptyLines:246 » IndexOutOfBounds No group...
[INFO] 
[ERROR] Tests run: 1580, Failures: 3, Errors: 1, Skipped: 0

[Pankraz76]

• fix RemoveUnusedImportsStep leftovers palantir/palantir-java-format#1337

[Pankraz76]

can not fix remaining tests, ether disable or fix.

Kindly request some feedback. Thanks.