From 6c83233264843cecf2f53083ad61d3a120f004af Mon Sep 17 00:00:00 2001
From: Euan Torano <euantorano@gmail.com>
Date: Mon, 3 Jul 2017 22:23:32 +0100
Subject: [PATCH] Added some basic unit tests, and Circle CI

Signed-off-by: Euan Torano <euantorano@gmail.com>
---
 .circleci/config.yml | 28 +++++++++++++
 redis.nimble         | 33 +++++++++-------
 tests/main.nim       | 93 ++++++++++++++++++++++++++++++++++++++++++++
 tests/nim.cfg        |  8 ++++
 4 files changed, 147 insertions(+), 15 deletions(-)
 create mode 100644 .circleci/config.yml
 create mode 100644 tests/main.nim
 create mode 100644 tests/nim.cfg

diff --git a/.circleci/config.yml b/.circleci/config.yml
new file mode 100644
index 0000000..4ca2cb2
--- /dev/null
+++ b/.circleci/config.yml
@@ -0,0 +1,28 @@
+version: 2
+jobs:
+  build:
+    working_directory: /usr/src/redis
+    docker:
+      - image: nimlang/nim
+      - image: redis:3
+    steps:
+      - checkout
+      - run:
+          name: test
+          command: nim c -r tests/main.nim
+  build_alpine:
+    working_directory: /usr/src/redis
+    docker:
+      - image: nimlang/nim:alpine
+      - image: redis:3
+    steps:
+      - checkout
+      - run:
+          name: test
+          command: nim c -r tests/main.nim
+workflows:
+  version: 2
+  build_and_test:
+    jobs:
+      - build
+      - build_alpine
diff --git a/redis.nimble b/redis.nimble
index 64efc5a..b454b72 100644
--- a/redis.nimble
+++ b/redis.nimble
@@ -1,15 +1,18 @@
-# Package
-
-version       = "0.2.0"
-author        = "Dominik Picheta"
-description   = "Official redis client for Nim"
-license       = "MIT"
-
-srcDir = "src"
-
-# Dependencies
-
-requires "nim >= 0.11.0"
-
-task docs, "Build documentation":
-    exec "nim doc --index:on -o:docs/redis.html src/redis.nim"
\ No newline at end of file
+# Package
+
+version       = "0.2.0"
+author        = "Dominik Picheta"
+description   = "Official redis client for Nim"
+license       = "MIT"
+
+srcDir = "src"
+
+# Dependencies
+
+requires "nim >= 0.11.0"
+
+task docs, "Build documentation":
+  exec "nim doc --index:on -o:docs/redis.html src/redis.nim"
+
+task test, "Run tests":
+  exec "nim c -r tests/main.nim"
diff --git a/tests/main.nim b/tests/main.nim
new file mode 100644
index 0000000..9b83443
--- /dev/null
+++ b/tests/main.nim
@@ -0,0 +1,93 @@
+import redis, unittest
+
+suite "redis tests":
+  let r = redis.open("localhost")
+
+  test "simple set and get":
+    const expected = "Hello, World!"
+
+    r.setk("redisTests:simpleSetAndGet", expected)
+    let actual = r.get("redisTests:simpleSetAndGet")
+
+    check actual == expected
+
+  test "increment key by one":
+    const expected = 3
+
+    r.setk("redisTests:incrementKeyByOne", "2")
+    let actual = r.incr("redisTests:incrementKeyByOne")
+
+    check actual == expected
+
+  test "increment key by five":
+    const expected = 10
+
+    r.setk("redisTests:incrementKeyByFive", "5")
+    let actual = r.incrBy("redisTests:incrementKeyByFive", 5)
+
+    check actual == expected
+
+  test "decrement key by one":
+    const expected = 2
+
+    r.setk("redisTest:decrementKeyByOne", "3")
+    let actual = r.decr("redisTest:decrementKeyByOne")
+
+    check actual == expected
+
+  test "decrement key by three":
+    const expected = 7
+
+    r.setk("redisTest:decrementKeyByThree", "10")
+    let actual = r.decrBy("redisTest:decrementKeyByThree", 3)
+
+    check actual == expected
+
+  test "append string to key":
+    const expected = "hello world"
+
+    r.setk("redisTest:appendStringToKey", "hello")
+    let keyLength = r.append("redisTest:appendStringToKey", " world")
+
+    check keyLength == len(expected)
+    check r.get("redisTest:appendStringToKey") == expected
+
+  test "check key exists":
+    r.setk("redisTest:checkKeyExists", "foo")
+    check r.exists("redisTest:checkKeyExists") == true
+
+  test "delete key":
+    r.setk("redisTest:deleteKey", "bar")
+    check r.exists("redisTest:deleteKey") == true
+
+    check r.del(@["redisTest:deleteKey"]) == 1
+    check r.exists("redisTest:deleteKey") == false
+
+  test "rename key":
+    const expected = "42"
+
+    r.setk("redisTest:renameKey", expected)
+    discard r.rename("redisTest:renameKey", "redisTest:meaningOfLife")
+
+    check r.exists("redisTest:renameKey") == false
+    check r.get("redisTest:meaningOfLife") == expected
+
+  test "get key length":
+    const expected = 5
+
+    r.setk("redisTest:getKeyLength", "hello")
+    let actual = r.strlen("redisTest:getKeyLength")
+
+    check actual == expected
+
+  test "push entries to list":
+    for i in 1..5:
+      check r.lPush("redisTest:pushEntriesToList", $i) == i
+
+    check r.llen("redisTest:pushEntriesToList") == 5
+
+  # TODO: Ideally tests for all other procedures, will add these in the future
+
+  # delete all keys in the DB at the end of the tests
+  discard r.flushdb()
+  r.quit()
diff --git a/tests/nim.cfg b/tests/nim.cfg
new file mode 100644
index 0000000..86c5998
--- /dev/null
+++ b/tests/nim.cfg
@@ -0,0 +1,8 @@
+path: "$projectPath/../src"
+hints: off
+linedir: on
+debuginfo
+stacktrace: on
+linetrace: on
+
+define: test