Fix issues with the new jsonschema

With the update, exclusiveM* functions are now checked and some unit tests failed due to invalid data. Also set window to min 0, because test: `TEST 22: Window is set to 0 and default is 1.` is already checking this. Signed-off-by: Eloy Coto <[email protected]>
3scale · Apr 2, 2020 · 378b601 · 378b601
1 parent 3b73569
commit 378b601
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/gateway/src/apicast/policy/rate_limit/apicast-policy.json b/gateway/src/apicast/policy/rate_limit/apicast-policy.json
@@ -197,7 +197,7 @@
             "window": {
               "type": "integer",
               "description": "The time window in seconds before the request count is reset",
-              "exclusiveMinimum": 0,
+              "minimum": 0,
               "default": 1
             }
           }

diff --git a/spec/policy/rate_limit/rate_limit_spec.lua b/spec/policy/rate_limit/rate_limit_spec.lua
@@ -66,13 +66,13 @@ describe('Rate limit policy', function()
       it('does not crash', function()
         local rate_limit_policy = RateLimitPolicy.new({
           connection_limiters = {
-            { key = { name = 'test1', scope = 'global' }, conn = 0, burst = 0, delay = 0 }
+            { key = { name = 'test1', scope = 'global' }, conn = 1, burst = 0, delay = 0.5 }
           },
           leaky_bucket_limiters = {
-            { key = { name = 'test2', scope = 'global' }, rate = 0, burst = 0 }
+            { key = { name = 'test2', scope = 'global' }, rate = 1, burst = 0 }
           },
           fixed_window_limiters = {
-            { key = { name = 'test3', scope = 'global' }, count = 0, window = 0 }
+            { key = { name = 'test3', scope = 'global' }, count = 1, window = 1 }
           },
         })
 
@@ -98,7 +98,7 @@ describe('Rate limit policy', function()
       it('works with multiple limiters', function()
         local rate_limit_policy = RateLimitPolicy.new({
           connection_limiters = {
-            { key = { name = 'test1', scope = 'global' }, conn = 20, burst = 10, delay = 0.5 }
+            { key = { name = 'test1', scope = 'global' }, conn = 20, burst = 10, delay = 0.4 }
           },
           leaky_bucket_limiters = {
             { key = { name = 'test2', scope = 'global' }, rate = 18, burst = 9 }