RemoveUnusedImports should handle unused import aliases

[jonas]

Scalafix 0.4 doesn't remove unused imports which are aliased:

import scala.collection.mutable.{Set ⇒ MutableSet, Map ⇒ MutableMap}

The following patch will cause the unused import rewrite test to fail:

diff --git i/scalafix-tests/input/src/main/scala/test/RemoveUnusedImports.scala w/scalafix-tests/input/src/main/scala/test/RemoveUnusedImports.scala
index 9b64c9c..e410565 100644
--- i/scalafix-tests/input/src/main/scala/test/RemoveUnusedImports.scala
+++ w/scalafix-tests/input/src/main/scala/test/RemoveUnusedImports.scala
@@ -11,6 +11,8 @@ import scala.util.{Success => Successful, Random}
 import scala.sys.process._
 import scala.concurrent.{CancellationException, ExecutionException, ExecutionContext}
 import scala.runtime.{RichBoolean}
+import scala.collection.mutable.{Set ⇒ MutableSet, Map ⇒ MutableMap}
+import scala.collection.mutable.{Seq => MutableSeq}
 import scala.concurrent.{ // formatting caveat
   CancellationException,
   ExecutionException,

Observed in sangria-graphql/sangria#250, specifically this line which had to be fixed manually.

[olafurpg]

Thank you for reporting! I opened #190 which I believe fixes this bug.