[ALL][ISSUE-61]: fixing issue, refactoring (#62)

akka-http/src/test/scala/com/github/fsanaulla/chronicler/akka/utils/TestHelper.scala

@@ -1,9 +1,9 @@
 package com.github.fsanaulla.chronicler.akka.utils
 
 import akka.http.scaladsl.model.Uri
+import com.github.fsanaulla.chronicler.macros.Macros
+import com.github.fsanaulla.chronicler.macros.annotations.{field, tag}
 import com.github.fsanaulla.core.model._
-import com.github.fsanaulla.macros.Macros
-import com.github.fsanaulla.macros.annotations.{field, tag}
 
 /**
   * Created by fayaz on 11.07.17.

async-http/src/test/scala/com/github/fsanaulla/chronicler/async/utils/TestHelper.scala

@@ -2,9 +2,9 @@ package com.github.fsanaulla.chronicler.async.utils
 
 import java.net.URLEncoder
 
+import com.github.fsanaulla.chronicler.macros.Macros
+import com.github.fsanaulla.chronicler.macros.annotations.{field, tag}
 import com.github.fsanaulla.core.model._
-import com.github.fsanaulla.macros.Macros
-import com.github.fsanaulla.macros.annotations.{field, tag}
 
 /**
   * Created by fayaz on 11.07.17.

core/src/main/scala/com/github/fsanaulla/core/model/InfluxPoint.scala

@@ -64,7 +64,7 @@ final case class Point(
     sb.append(measurement).append(",")
     sb.append(tags.map(tag => tag.key + "=" + tag.value).mkString(","))
     sb.append(" ")
-    sb.append(fields.map(_.toString).mkString(" "))
+    sb.append(fields.map(_.toString).mkString(","))
 
     if (time != -1L) {
       sb.append(" ").append(time)

core/src/test/scala/com/gtihub/fsanaulla/chronicler/core/PointSpec.scala

@@ -17,6 +17,6 @@ class PointSpec extends FlatSpec with Matchers {
       .addField("weight", 75.6)
       .addTimestamp(ts)
 
-    p.serialize shouldEqual "test,city=London name=\"Jivi\" age=22i adult=true weight=75.6" + s" $ts"
+    p.serialize shouldEqual "test,city=London name=\"Jivi\",age=22i,adult=true,weight=75.6" + s" $ts"
   }
 }

macros/src/main/scala/com/github/fsanaulla/macros/Macros.scala → macros/src/main/scala/com/github/fsanaulla/chronicler/macros/Macros.scala

@@ -1,4 +1,4 @@
-package com.github.fsanaulla.macros
+package com.github.fsanaulla.chronicler.macros
 
 import com.github.fsanaulla.core.model.{InfluxFormatter, InfluxReader, InfluxWriter}
 

macros/src/main/scala/com/github/fsanaulla/macros/MacrosImpl.scala → macros/src/main/scala/com/github/fsanaulla/chronicler/macros/MacrosImpl.scala

@@ -1,6 +1,6 @@
-package com.github.fsanaulla.macros
+package com.github.fsanaulla.chronicler.macros
 
-import com.github.fsanaulla.macros.annotations.{field, tag, timestamp}
+import com.github.fsanaulla.chronicler.macros.annotations.{field, tag, timestamp}
 
 import scala.language.experimental.macros
 import scala.reflect.macros.blackbox
@@ -205,7 +205,7 @@ private[macros] class MacrosImpl(val c: blackbox.Context) {
                   case (k, v: String) => k + "=" + "\"" + v + "\""
                   case (k, v: Int) => k + "=" + v + "i"
                   case (k, v) => k + "=" + v
-                } mkString(" ")
+                } mkString(",")
 
                 val nonOptTagsMap: Map[String, String] = Map(..$nonOptTags)
                 val nonOptTags: String = nonOptTagsMap map {
@@ -228,7 +228,7 @@ private[macros] class MacrosImpl(val c: blackbox.Context) {
                case (k, v: String) => k + "=" + "\"" + v + "\""
                case (k, v: Int) => k + "=" + v + "i"
                case (k, v) => k + "=" + v
-            } mkString(" ")
+            } mkString(",")
 
             val nonOptTagsMap: Map[String, String] = Map(..$nonOptTags)
             val nonOptTags: String = nonOptTagsMap map {

macros/src/main/scala/com/github/fsanaulla/macros/annotations/field.scala → macros/src/main/scala/com/github/fsanaulla/chronicler/macros/annotations/field.scala

@@ -1,4 +1,4 @@
-package com.github.fsanaulla.macros.annotations
+package com.github.fsanaulla.chronicler.macros.annotations
 
 /***
   * Marker for declaring field value for influx point.

macros/src/main/scala/com/github/fsanaulla/macros/annotations/tag.scala → macros/src/main/scala/com/github/fsanaulla/chronicler/macros/annotations/tag.scala

@@ -1,4 +1,4 @@
-package com.github.fsanaulla.macros.annotations
+package com.github.fsanaulla.chronicler.macros.annotations
 
 /***
   * Marker for declaring influx point tag's. Can be only String.

macros/src/main/scala/com/github/fsanaulla/macros/annotations/timestamp.scala → macros/src/main/scala/com/github/fsanaulla/chronicler/macros/annotations/timestamp.scala

@@ -1,4 +1,4 @@
-package com.github.fsanaulla.macros.annotations
+package com.github.fsanaulla.chronicler.macros.annotations
 
 /** Mark field that will be used as timestamp in InfluxDB.
   * It can be optional, in this case will be used default

macros/src/test/scala/com/github/fsanaulla/macros/MacroFormatterSpec.scala → macros/src/test/scala/com/github/fsanaulla/chronicler/macros/MacroFormatterSpec.scala

@@ -1,8 +1,8 @@
-package com.github.fsanaulla.macros
+package com.github.fsanaulla.chronicler.macros
 
+import com.github.fsanaulla.chronicler.macros.annotations.{field, tag, timestamp}
 import com.github.fsanaulla.chronicler.testing.FlatSpecWithMatchers
 import com.github.fsanaulla.core.model.InfluxFormatter
-import com.github.fsanaulla.macros.annotations.{field, tag, timestamp}
 import jawn.ast.{JArray, JNull, JNum, JString}
 
 class MacroFormatterSpec extends FlatSpecWithMatchers {

macros/src/test/scala/com/github/fsanaulla/macros/MacroReaderSpec.scala → macros/src/test/scala/com/github/fsanaulla/chronicler/macros/MacroReaderSpec.scala

@@ -1,8 +1,8 @@
-package com.github.fsanaulla.macros
+package com.github.fsanaulla.chronicler.macros
 
+import com.github.fsanaulla.chronicler.macros.annotations.{field, tag, timestamp}
 import com.github.fsanaulla.chronicler.testing.FlatSpecWithMatchers
 import com.github.fsanaulla.core.model.InfluxReader
-import com.github.fsanaulla.macros.annotations.{field, tag, timestamp}
 import jawn.ast._
 
 class MacroReaderSpec extends FlatSpecWithMatchers {

macros/src/test/scala/com/github/fsanaulla/macros/MacroWriterSpec.scala → macros/src/test/scala/com/github/fsanaulla/chronicler/macros/MacroWriterSpec.scala

@@ -1,8 +1,8 @@
-package com.github.fsanaulla.macros
+package com.github.fsanaulla.chronicler.macros
 
+import com.github.fsanaulla.chronicler.macros.annotations.{field, tag, timestamp}
 import com.github.fsanaulla.chronicler.testing.FlatSpecWithMatchers
 import com.github.fsanaulla.core.model.InfluxWriter
-import com.github.fsanaulla.macros.annotations.{field, tag, timestamp}
 
 class MacroWriterSpec extends FlatSpecWithMatchers {
 
@@ -21,14 +21,14 @@ class MacroWriterSpec extends FlatSpecWithMatchers {
   val wr1: InfluxWriter[Test1] = Macros.writer[Test1]
 
   "Macros.writer" should "write with None" in {
-    wr.write(Test("nm", None, "Berkly", 65)) shouldEqual "name=nm school=\"Berkly\" age=65i"
+    wr.write(Test("nm", None, "Berkly", 65)) shouldEqual "name=nm school=\"Berkly\",age=65i"
   }
 
   it should "write with Some" in {
-    wr.write(Test("nm", Some("sn"), "Berkly", 65)) shouldEqual "name=nm,surname=sn school=\"Berkly\" age=65i"
+    wr.write(Test("nm", Some("sn"), "Berkly", 65)) shouldEqual "name=nm,surname=sn school=\"Berkly\",age=65i"
   }
 
   it should "write with timestamp" in {
-    wr1.write(Test1("nm", Some("sn"), 65, "Berkly", 1438715114318570484L)) shouldEqual "name=nm,surname=sn age=65i school=\"Berkly\" 1438715114318570484"
+    wr1.write(Test1("nm", Some("sn"), 65, "Berkly", 1438715114318570484L)) shouldEqual "name=nm,surname=sn age=65i,school=\"Berkly\" 1438715114318570484"
   }
 }

udp/src/test/scala/com/github/fsanaulla/chronicler/udp/UdpClientSpec.scala

@@ -4,11 +4,11 @@ import java.io.File
 
 import com.github.fsanaulla.chronicler.async.InfluxAsyncHttpClient
 import com.github.fsanaulla.chronicler.async.api.Database
+import com.github.fsanaulla.chronicler.macros.Macros
+import com.github.fsanaulla.chronicler.macros.annotations.{field, tag}
 import com.github.fsanaulla.chronicler.testing.{FutureHandler, TestSpec}
 import com.github.fsanaulla.core.model.{InfluxFormatter, Point}
 import com.github.fsanaulla.core.testing.configurations.InfluxUDPConf
-import com.github.fsanaulla.macros.Macros
-import com.github.fsanaulla.macros.annotations.{field, tag}
 import com.github.fsanaulla.scalatest.EmbeddedInfluxDB
 
 import scala.concurrent.ExecutionContext.Implicits.global

url-http/src/test/scala/com/github/fsanaulla/chronicler/urlhttp/utils/TestHelper.scala

@@ -2,9 +2,9 @@ package com.github.fsanaulla.chronicler.urlhttp.utils
 
 import java.net.URLEncoder
 
+import com.github.fsanaulla.chronicler.macros.Macros
+import com.github.fsanaulla.chronicler.macros.annotations.{field, tag}
 import com.github.fsanaulla.core.model._
-import com.github.fsanaulla.macros.Macros
-import com.github.fsanaulla.macros.annotations.{field, tag}
 
 /**
   * Created by fayaz on 11.07.17.

version.sbt

@@ -1 +1 @@
-version in ThisBuild := "0.2.4"
+version in ThisBuild := "0.2.5"