Start implementing test suite for save actions

This contains tests for the save action RemoveTrailingWhitespace.

org.scala-ide.sdt.core.tests/src/org/scalaide/TestsSuite.scala

@@ -42,6 +42,7 @@ import org.scalaide.core.project.DirectoryScalaInstallationTest
 import org.scalaide.core.classpath.DesiredScalaInstallationTests
 import org.scalaide.ui.wizards.WizardTests
 import org.scalaide.core.text.TextTestSuite
+import org.scalaide.extensions.saveactions.SaveActionTestSuite
 
 @RunWith(classOf[Suite])
 @Suite.SuiteClasses(
@@ -87,6 +88,7 @@ import org.scalaide.core.text.TextTestSuite
     classOf[DirectoryScalaInstallationTest],
     classOf[CompletionTestSuite],
     classOf[WizardTests],
-    classOf[TextTestSuite]
+    classOf[TextTestSuite],
+    classOf[SaveActionTestSuite]
 ))
 class TestsSuite

org.scala-ide.sdt.core.tests/src/org/scalaide/extensions/saveactions/RemoveTrailingWhitespaceTest.scala

@@ -0,0 +1,35 @@
+package org.scalaide.extensions.saveactions
+
+import org.junit.Test
+
+class RemoveTrailingWhitespaceTest extends SaveActionTests { self =>
+
+  override def saveAction = new RemoveTrailingWhitespace {
+    override val document = self.document
+  }
+
+  @Test
+  def trailing_whitespace_is_removed() = """|^
+    |class X {
+    |  def hello = {    $
+    |    val x = 0   $
+    |    $
+    |    val y = 1   $
+    |    $
+    |    x + y
+    |  }
+    |}
+    |   $
+    |""".stripMargin becomes """|^
+    |class X {
+    |  def hello = {
+    |    val x = 0
+    |
+    |    val y = 1
+    |
+    |    x + y
+    |  }
+    |}
+    |
+    |""".stripMargin after SaveEvent
+}

org.scala-ide.sdt.core.tests/src/org/scalaide/extensions/saveactions/SaveActionTestSuite.scala

@@ -0,0 +1,10 @@
+package org.scalaide.extensions.saveactions
+
+import org.junit.runner.RunWith
+import org.junit.runners.Suite
+
+@RunWith(classOf[Suite])
+@Suite.SuiteClasses(Array(
+  classOf[RemoveTrailingWhitespaceTest]
+))
+class SaveActionTestSuite

org.scala-ide.sdt.core.tests/src/org/scalaide/extensions/saveactions/SaveActionTests.scala

@@ -0,0 +1,40 @@
+package org.scalaide.extensions
+package saveactions
+
+import org.eclipse.jface.text.{Document => EDocument}
+import org.scalaide.core.internal.text.TextDocument
+import org.scalaide.core.text.Document
+import org.scalaide.core.text.TextChange
+import org.scalaide.core.ui.TextEditTests
+import org.scalaide.extensions.SaveAction
+
+abstract class SaveActionTests extends TextEditTests {
+
+  def saveAction: SaveAction
+
+  var document: Document = _
+
+  private var udoc: EDocument = _
+
+  override def runTest(source: String, operation: Operation) = {
+    udoc = new EDocument(source)
+    document = new TextDocument(udoc)
+    operation.execute()
+  }
+
+  override def source = udoc.get()
+
+  case object SaveEvent extends Operation {
+    override def execute() = {
+      val changes = saveAction.perform()
+      val sorted = changes.sortBy {
+        case TextChange(start, _, _) => -start
+      }
+      sorted foreach {
+        case TextChange(start, end, text) =>
+          udoc.replace(start, end-start, text)
+      }
+    }
+  }
+
+}

org.scala-ide.sdt.core/src/org/scalaide/extensions/saveactions/RemoveTrailingWhitespace.scala

@@ -20,7 +20,7 @@ trait RemoveTrailingWhitespace extends SaveAction with DocumentSupport {
 
   override def setting = RemoveTrailingWhitespaceSetting
 
-  def perform() = {
+  override def perform() = {
     document.lines flatMap { line =>
       val trimmed = line.trimRight
 
@@ -30,4 +30,4 @@ trait RemoveTrailingWhitespace extends SaveAction with DocumentSupport {
         Seq()
     }
   }
-}
+}