Permalink
Browse files

Added some additional notes

  • Loading branch information...
1 parent 7d1c1e8 commit c973bb01f6e4e47e1f69182da68a66dcde793456 @rlegendi committed May 27, 2012
Showing with 20 additions and 3 deletions.
  1. +15 −0 TODO.md
  2. +5 −3 src/main/scala/org/specs2/Specs2Bridge.scala
View
15 TODO.md
@@ -21,3 +21,18 @@ Things to read/watch
* Esp. High Wizardry in the Land of Scala - sounds a great title :-)
http://hacking-scala.posterous.com/scalaz-resources-for-beginners
+Features I miss
+---------------
+
+- Unimplemented method declaration for Ctrl + Space
+- Companion objects: show apply() signature
+
+
+Scala stuff I found confusing first
+-----------------------------------
+
+- No type definition: Without the IDE, source is unreadable
+- Everything is changing quickly, builds are volatile and shatter quickly
+ (scala, sbt version-pending builds)
+- Mixing 4 build tools (sbt/ivy/mvn/tychoo)
+
@@ -92,9 +92,11 @@ object Specs2Bridge {
* ScopeOpened ---
* ScopeClosed ---
*/
-
+
// TODO Ask Eric: When do the DecoratedResults are used?
-
+
+ // TODO Indenting?
+
result match {
case f @ Failure(m, e, st, d) =>
// notifier.fireTestFailure(new notification.Failure(desc, junitFailure(f)))
@@ -132,7 +134,7 @@ object Specs2Bridge {
//notifier.fireTestFinished(desc)
//reporter()
}
- case ExecutedSpecStart(_, _, _) => //notifier.fireTestRunStarted(desc)
+ case ExecutedSpecStart(_, _, _) => //notifier.fireTestRunStarted(desc) // TODO: Should we care?
case ExecutedSpecEnd(_, _, _) => //notifier.fireTestRunFinished(new org.junit.runner.Result)
case _ => // don't do anything otherwise too many tests will be counted
}

0 comments on commit c973bb0

Please sign in to comment.