From d2382fee74be03cfc1ef5f79b8500ac524c5ff42 Mon Sep 17 00:00:00 2001
From: jonschlinkert <jon.schlinkert@sellside.com>
Date: Tue, 24 Feb 2015 22:55:23 -0500
Subject: [PATCH] add tests for errors

---
 test.js | 30 +++++++++++++++++++++---------
 1 file changed, 21 insertions(+), 9 deletions(-)

diff --git a/test.js b/test.js
index 72b0959..fb3b183 100644
--- a/test.js
+++ b/test.js
@@ -8,32 +8,44 @@
 'use strict';
 
 var should = require('should');
-var config = require('./');
+var git = require('./');
+
 
 describe('sync:', function () {
   it('should return an object', function () {
-    config.sync().should.have.properties([]);
+    git.sync().should.have.properties(['core']);
+    git.sync().should.not.have.properties(['foo']);
+    git.sync().should.not.throw;
   });
 
   it('should throw an error when .git/config does not exist:', function () {
     (function () {
-      config.sync('foo');
+      git.sync('foo');
     }).should.throw('.git/config does not exist.');
   });
 });
 
 describe('async:', function () {
+  it('should throw a callback is not passed:', function (cb) {
+    (function () {
+      git();
+    }).should.throw('parse-git-config async expects a callback function.')
+    cb();
+  });
+
   it('should throw an error when .git/config does not exist:', function (cb) {
-    config(function (err, data) {
-      console.log(arguments)
+    git(function (err, config) {
+      config.should.have.property('core');
+      config.should.not.have.property('foo');
+      cb();
     });
-    cb();
   });
 
   it('should throw an error when .git/config does not exist:', function (cb) {
-    config('foo', function (err, data) {
-      console.log(arguments)
+    git('foo', function (err, config) {
+      err.should.be.an.instanceof(Error);
+      err.message.should.equal('ENOENT, open \'foo/.git/config\'');
+      cb();
     });
-    cb();
   });
 });