fix dataset v3 call CBOE/VXEEM fixes normanjoyner#6

Victory · Jan 17, 2016 · c57e76a · c57e76a
1 parent 6d4430d
commit c57e76a
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/lib/api.js b/lib/api.js
@@ -15,8 +15,15 @@ module.exports = {
 
         var qs = options;
         qs.auth_token = this.auth_token;
+        if (this.api_version == 1) {
+            this.last_uri_called = ["api", this.api_version, "datasets", code.source, code.table].join("/");
+        } else {
+            this.last_uri_called = "api/" + this.api_version + "/datasets/" + code.source;
+            if (code.table) {
+                this.last_uri_called += "/" + code.table;
+            }
+        }
 
-        this.last_uri_called = ["api", this.api_version, "datasets", code.source, code.table].join("/");
         var config = {
             uri: this.last_uri_called, 
             format: format,

diff --git a/test/quandl.js b/test/quandl.js
@@ -48,7 +48,7 @@ describe("quandl", function(){
             assert.equal(quandl.last_uri_called, "");
         });
 
-        it("has correct api call for v3", function() {
+        it("has correct api call for dataset v3", function() {
             quandl.configure({
                 api_version: 3,
                 auth_token: "dsahFHUiewjjd"
@@ -60,7 +60,7 @@ describe("quandl", function(){
 
             assert.equal(
                 quandl.last_uri_called, 
-                "api/v1/datasets/CBOE/VXEEM.json");
+                "api/v3/datasets/CBOE/VXEEM");
         });
     });