From 895041b2a7c07186cf73116e2ab2fe572ed4728c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Carlos=20Mart=C3=ADn=20Nieto?= <cmn@dwim.me>
Date: Mon, 2 Sep 2013 20:10:38 +0200
Subject: [PATCH] tests: properly setup and teardown

---
 test/object_test.exs | 20 ++++++++++++--------
 test/test_helper.exs | 10 ++++++++++
 2 files changed, 22 insertions(+), 8 deletions(-)

diff --git a/test/object_test.exs b/test/object_test.exs
index b660605..cc6aea8 100644
--- a/test/object_test.exs
+++ b/test/object_test.exs
@@ -3,17 +3,21 @@ Code.require_file "../test_helper.exs", __FILE__
 defmodule ObjectTest do
   use ExUnit.Case
   use Geef
+  import RepoHelpers
 
-  defp tmp_dir do
-    {a, b, c} = :erlang.now()
-    n = node()
-    dir = :io_lib.format("geef-~p~p~p~p.git", [n, a, b, c])
-    Path.join(System.tmp_dir!, dir)
+  setup do
+    {{:ok, repo}, path} = tmp_bare()
+    {:ok, [repo: repo, path: path]}
   end
 
-  test "object OO helpers" do
-    dir = tmp_dir
-    { :ok, repo } = Repository.init(dir, true)
+  teardown meta do
+    Repository.stop(meta[:repo])
+    File.rm_rf!(meta[:path])
+    :ok
+  end
+
+  test "object OO helpers", meta do
+    repo = meta[:repo]
     { :ok, odb } = Repository.odb(repo)
     #{ :ok, index } = :geef_index.new
 
diff --git a/test/test_helper.exs b/test/test_helper.exs
index 4b8b246..57c7766 100644
--- a/test/test_helper.exs
+++ b/test/test_helper.exs
@@ -1 +1,11 @@
 ExUnit.start
+
+defmodule RepoHelpers do
+  def tmp_bare do
+    {a, b, c} = :erlang.now()
+    n = node()
+    dir = :io_lib.format("geef-~p~p~p~p.git", [n, a, b, c])
+    path = Path.join(System.tmp_dir!, dir)
+    {Geef.Repository.init(path, true), path}
+  end
+end