Closes #469

Kong · Aug 12, 2015 · e302901 · e302901
1 parent ec28b83
commit e302901
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 7 deletions.
diff --git a/kong/plugins/oauth2/api.lua b/kong/plugins/oauth2/api.lua
@@ -43,7 +43,7 @@ return {
     end,
 
     GET = function(self, dao_factory, helpers)
-      return helpers.responses.send_HTTP_OK(self.plugin)
+      return helpers.responses.send_HTTP_OK(self.credential)
     end,
 
     PATCH = function(self, dao_factory)

diff --git a/spec/plugins/basicauth/api_spec.lua b/spec/plugins/basicauth/api_spec.lua
@@ -78,8 +78,10 @@ describe("Basic Auth Credentials API", function()
     describe("GET", function()
 
       it("should retrieve by id", function()
-        local _, status = http_client.get(BASE_URL..credential.id)
+        local response, status = http_client.get(BASE_URL..credential.id)
         assert.equal(200, status)
+        local body = json.decode(response)
+        assert.equals(credential.id, body.id)
       end)
 
     end)

diff --git a/spec/plugins/keyauth/api_spec.lua b/spec/plugins/keyauth/api_spec.lua
@@ -81,8 +81,10 @@ describe("Key Auth Credentials API", function()
     describe("GET", function()
 
       it("should retrieve by id", function()
-        local _, status = http_client.get(BASE_URL..credential.id)
+        local response, status = http_client.get(BASE_URL..credential.id)
         assert.equal(200, status)
+        local body = json.decode(response)
+        assert.equals(credential.id, body.id)
       end)
 
     end)

diff --git a/spec/plugins/oauth2/api_spec.lua b/spec/plugins/oauth2/api_spec.lua
@@ -13,7 +13,7 @@ describe("OAuth 2 Credentials API", function()
   teardown(function()
     spec_helper.stop_kong()
   end)
-
+  
   describe("/consumers/:consumer/oauth2/", function()
 
     setup(function()
@@ -72,14 +72,16 @@ describe("OAuth 2 Credentials API", function()
 
     end)
   end)
-
+  
   describe("/consumers/:consumer/oauth2/:id", function()
 
     describe("GET", function()
 
       it("should retrieve by id", function()
-        local _, status = http_client.get(BASE_URL..credential.id)
+        local response, status = http_client.get(BASE_URL..credential.id)
         assert.equal(200, status)
+        local body = json.decode(response)
+        assert.equals(credential.id, body.id)
       end)
 
     end)
@@ -117,6 +119,6 @@ describe("OAuth 2 Credentials API", function()
       end)
 
     end)
-    
+
   end)
 end)