Added comments

snowplow · Feb 2, 2014 · 33838aa · 33838aa
1 parent e193ef9
commit 33838aa
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 4 deletions.
diff --git a/2-collectors/thrift-raw-event/project/Dependencies.scala b/2-collectors/thrift-raw-event/project/Dependencies.scala
@@ -16,6 +16,7 @@ import sbt._
 
 object Dependencies {
 
+  // TODO: both of these can be removed
   val resolutionRepos = Seq(
     // For scala-util
     "Snowplow Analytics Maven repo" at "http://maven.snplow.com/releases/",

diff --git a/...rc/test/scala/com.snowplowanalytics.snowplow.collectors.thrift/SnowplowRawEventSpec.scala b/...rc/test/scala/com.snowplowanalytics.snowplow.collectors.thrift/SnowplowRawEventSpec.scala
@@ -12,17 +12,20 @@
  * implied.  See the Apache License Version 2.0 for the specific language
  * governing permissions and limitations there under.
  */
-
 package com.snowplowanalytics.snowplow.collectors.thrift
 
+// Scala
 import scala.collection.JavaConversions._
 
-// Specs2 and ScalaCheck
-import org.specs2.mutable.Specification
+// ScalaCheck
 import org.scalacheck.{Arbitrary,Gen,Properties}
 import org.scalacheck.Prop.forAll
 
-object SnowplowRawEventSpec extends Properties("SnowplowRawEvent"){
+/**
+ * ScalaCheck specification testing all of the properties
+ * of the SnowplowRawEvent POJO.
+ */
+object SnowplowRawEventSpec extends Properties("SnowplowRawEvent") {
   property("timestamp") = forAll { (timestamp: Long) =>
     val event = new SnowplowRawEvent(timestamp, "collector", "encoding",
       "127.0.0.1")