Merge pull request #190 from olafurpg/189

Handle Importee.Rename in RemoveUnusedImports, fixes #189

Author: olafurpg

scalafix-core/src/main/scala/scalafix/rewrite/RemoveUnusedImports.scala

@@ -11,15 +11,17 @@ case class RemoveUnusedImports(mirror: Mirror)
     case Message(pos, _, "Unused import") =>
       pos
   }.toSet
-  private def isUnused(importee: Importee) = importee match {
-    // NOTE: workaround for https://github.com/scalameta/scalameta/issues/899
-    case Importee.Wildcard() =>
-      val lookupPos =
+  private def isUnused(importee: Importee) = {
+    val pos = importee match {
+      // NOTE: workaround for https://github.com/scalameta/scalameta/issues/899
+      case Importee.Wildcard() =>
         importee.parents
           .collectFirst { case x: Import => x.pos }
           .getOrElse(importee.pos)
-      unusedImports.contains(lookupPos)
-    case _ => unusedImports.contains(importee.pos)
+      case Importee.Rename(name, _) => name.pos
+      case _ => importee.pos
+    }
+    unusedImports.contains(pos)
   }
   override def rewrite(ctx: RewriteCtx): Patch =
     ctx.tree.collect {

scalafix-tests/input/src/main/scala/test/RemoveUnusedImports.scala

@@ -8,6 +8,8 @@ import scala.util.control.{NonFatal, Breaks}
 import scala.concurrent.{Await, Future}
 import scala.util.{Properties, DynamicVariable, Try}
 import scala.util.{Success => Successful, Random}
+import scala.collection.mutable.{Set ⇒ MutableSet, Map ⇒ MutableMap}
+import scala.collection.mutable.{Seq => MutableSeq}
 import scala.sys.process._
 import scala.concurrent.{CancellationException, ExecutionException, ExecutionContext}
 import scala.runtime.{RichBoolean}