t/apicast-policy-3scale-batcher.t: convert to APIcast::Blackbox #1349
Conversation
| location /transactions.xml { | ||
| content_by_lua_block { | ||
| ngx.req.read_body() | ||
| local post_args = ngx.req.get_post_args() | ||
| local app_id_match, usage_match | ||
| for k, v in pairs(post_args) do | ||
| if k == 'transactions[0][app_key]' then | ||
| ngx.exit(500) | ||
| elseif k == 'transactions[0][usage][hits]' then | ||
| usage_match = v == '3' | ||
| elseif k == 'transactions[0][app_id]' then | ||
| app_id_match = v == 'appid' | ||
| end | ||
| end | ||
| ngx.shared.result = usage_match and app_id_match | ||
| } |
There was a problem hiding this comment.
this is executed when /force_report_to_backend is called, when the backend_client is instantiated, so it includes the usage (hits) from 3 calls now instead of 2, because the /force_report_to_backend endpoint is an upstream too so it increases the hits as well. It used to be 2 in the old test because that endpoint was accessed without going through APIcast.
| location /transactions.xml { | ||
| content_by_lua_block { | ||
| ngx.req.read_body() | ||
| local post_args = ngx.req.get_post_args() | ||
| local app_id_match, usage_match | ||
| for k, v in pairs(post_args) do | ||
| if k == 'transactions[0][app_key]' then | ||
| ngx.exit(500) | ||
| elseif k == 'transactions[0][usage][hits]' then | ||
| usage_match = v == '3' | ||
| elseif k == 'transactions[0][app_id]' then | ||
| app_id_match = v == 'appid' | ||
| end | ||
| end | ||
| ngx.shared.result = usage_match and app_id_match | ||
| } |
There was a problem hiding this comment.
this is executed when /force_report_to_backend is called, when the backend_client is instantiated, so it includes the usage (hits) from 3 calls now instead of 2, because the /force_report_to_backend endpoint is an upstream too so it increases the hits as well. It used to be 2 in the old test because that endpoint was accessed without going through APIcast.
samugi
force-pushed
the
blackbox-batcher-policy-test
branch
from
3e74c4f to
35ca9b5
Compare
samugi
force-pushed
the
blackbox-batcher-policy-test
branch
from
35ca9b5 to
04a3054
Compare
2 participants
No description provided.