fix config.getString and tests

jdgarcia · jdgarcia · commit 2e3f21abfa3a · 2015-08-13T11:35:34.000-07:00
diff --git a/lib/config.js b/lib/config.js
@@ -3,6 +3,6 @@ var NodeGit = require("../");
 var Config = NodeGit.Config;
 
 // Backwards compatibility.
-Config.getString = function() {
-  return Config.getStringBuf.apply(this, arguments);
+Config.prototype.getString = function() {
+  return this.getStringBuf.apply(this, arguments);
 };
diff --git a/test/tests/config.js b/test/tests/config.js
@@ -36,7 +36,13 @@ describe("Config", function() {
       .then(function(userNameFromNodeGit) {
         assert.equal(savedUserName + "-test", userNameFromNodeGit);
       })
-      .then(finallyFn, finallyFn);
+      .then(finallyFn)
+      .catch(function(e) {
+        return finallyFn()
+          .then(function() {
+            throw e;
+          });
+      });
   });
 
   it("can get and set a repo config value", function() {
@@ -70,6 +76,12 @@ describe("Config", function() {
     .then(function(userNameFromNodeGit) {
       assert.equal(savedUserName + "-test", userNameFromNodeGit);
     })
-    .then(finallyFn, finallyFn);
+    .then(finallyFn)
+    .catch(function(e) {
+      return finallyFn()
+        .then(function() {
+          throw e;
+        });
+    });
   });
 });
