rename http_codes to http_statuses

Signed-off-by: Eloy Coto <eloy.coto@acalustra.com>
3scale · Nov 30, 2020 · fb1e622 · fb1e622
1 parent 1021e53
commit fb1e622
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 11 deletions.
diff --git a/gateway/src/apicast/policy/statuscode_overwrite/apicast-policy.json b/gateway/src/apicast/policy/statuscode_overwrite/apicast-policy.json
@@ -37,7 +37,7 @@
       }
     },
     "properties": {
-      "http_codes": {
+      "http_statuses": {
         "title": "HTTP status codes map",
         "$ref": "#/definitions/codes" }
     }

diff --git a/gateway/src/apicast/policy/statuscode_overwrite/code_overwrite.lua b/gateway/src/apicast/policy/statuscode_overwrite/code_overwrite.lua
@@ -5,15 +5,15 @@ local new = _M.new
 
 function _M.new(config)
     local self = new(config)
-    self.http_codes = {}
-    for _, code in ipairs(config.http_codes or {}) do
-        self.http_codes[code.upstream] = code.apicast
+    self.http_statuses = {}
+    for _, code in ipairs(config.http_statuses or {}) do
+        self.http_statuses[code.upstream] = code.apicast
     end
     return self
   end
 
 function _M:header_filter()
-  ngx.status = self.http_codes[ngx.status] or ngx.status
+  ngx.status = self.http_statuses[ngx.status] or ngx.status
 end
 
 return _M
diff --git a/spec/policy/statuscode_overwrite/statuscode_overwrite_spec.lua b/spec/policy/statuscode_overwrite/statuscode_overwrite_spec.lua
@@ -4,7 +4,7 @@ describe('Status Codes overwrite policy', function()
 
   it('mapped correctly valid status code', function()
       local policy = StatusCodeOverwrite.new({
-        http_codes = {
+        http_statuses = {
           { upstream=200, apicast=201 }
         }
       })
@@ -22,7 +22,7 @@ describe('Status Codes overwrite policy', function()
   it('invalid code fails correctly', function()
     assert.has_error(function()
       return StatusCodeOverwrite.new({
-        http_codes = {
+        http_statuses = {
           { upstream=1, apicast=201 }
         }})
       end)
@@ -33,7 +33,7 @@ describe('Status Codes overwrite policy', function()
   it('Upstream code is not defined', function()
     assert.has_error(function()
       return StatusCodeOverwrite.new({
-        http_codes = {
+        http_statuses = {
           { apicast=201 }
         }})
       end)
@@ -42,7 +42,7 @@ describe('Status Codes overwrite policy', function()
   it('Apicast code is not defined', function()
     assert.has_error(function()
       return StatusCodeOverwrite.new({
-        http_codes = {
+        http_statuses = {
           { upstream=201 }
         }})
       end)

diff --git a/t/apicast-policy-statuscode_overwrite.t b/t/apicast-policy-statuscode_overwrite.t
@@ -20,7 +20,7 @@ __DATA__
             "name": "statuscode_overwrite",
             "version": "builtin",
             "configuration": {
-              "http_codes": [
+              "http_statuses": [
                 {
                   "upstream": 200,
                   "apicast": 201
@@ -91,7 +91,7 @@ __DATA__
             "name": "statuscode_overwrite",
             "version": "builtin",
             "configuration": {
-              "http_codes": [
+              "http_statuses": [
                 {
                   "upstream": 200,
                   "apicast": 201