Skip to content

Commit

Permalink
style(tests): reindex Test::Nginx tests (#13187)
Browse files Browse the repository at this point in the history
### Summary

Just reindex Test::Nginx tests with `reindex` found at:
https://github.com/openresty/openresty-devel-utils/blob/master/reindex

Signed-off-by: Aapo Talvensaari <aapo.talvensaari@gmail.com>
(cherry picked from commit fd994be)
  • Loading branch information
bungle committed Jun 10, 2024
1 parent 3e39c98 commit a89b4d1
Show file tree
Hide file tree
Showing 11 changed files with 19 additions and 12 deletions.
2 changes: 0 additions & 2 deletions t/01-pdk/02-log/00-phase_checks.t
Original file line number Diff line number Diff line change
Expand Up @@ -237,5 +237,3 @@ qq{
GET /t
--- no_error_log
[error]
10 changes: 8 additions & 2 deletions t/01-pdk/02-log/05-set_serialize_value.t
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ mode false mode must be 'set', 'add' or 'replace'
[error]
=== TEST 2: kong.log.serialize() rejects invalid values, including self-referencial tables
--- http_config eval: $t::Util::HttpConfig
--- config
Expand Down Expand Up @@ -64,6 +65,7 @@ self_ref false value must be nil, a number, string, boolean or a non-self-refere
[error]
=== TEST 3: kong.log.set_serialize_value stores changes on ngx.ctx.serialize_values
--- http_config eval: $t::Util::HttpConfig
--- config
Expand Down Expand Up @@ -91,6 +93,7 @@ GET /t
[error]
=== TEST 4: kong.log.set_serialize_value() sets, adds and replaces values with simple keys
--- http_config eval: $t::Util::HttpConfig
--- config
Expand Down Expand Up @@ -139,6 +142,7 @@ add existing value does not set it true
[error]
=== TEST 5: kong.log.set_serialize_value sets, adds and replaces values with keys with dots
--- http_config eval: $t::Util::HttpConfig
--- config
Expand Down Expand Up @@ -248,6 +252,8 @@ complex true
--- no_error_log
[error]
=== TEST 7: kong.log.set_serialize_value() setting values to nil
--- http_config eval: $t::Util::HttpConfig
--- config
Expand Down Expand Up @@ -287,6 +293,8 @@ request.headers.authorization true
--- no_error_log
[error]
=== TEST 8: kong.log.serialize() redactes authorization headers by default
--- http_config eval: $t::Util::HttpConfig
--- config
Expand Down Expand Up @@ -325,5 +333,3 @@ Proxy-Authorization REDACTED
PROXY_AUTHORIZATION REDACTED
--- no_error_log
[error]
2 changes: 0 additions & 2 deletions t/01-pdk/04-request/21-get_uri_captures.t
Original file line number Diff line number Diff line change
Expand Up @@ -39,5 +39,3 @@ GET /t/01/ok
uri_captures: tag: /ok, 0: /t/01/ok, 1: /01/ok, 2: /01, 3: /ok
--- no_error_log
[error]
3 changes: 2 additions & 1 deletion t/01-pdk/08-response/05-set_header.t
Original file line number Diff line number Diff line change
Expand Up @@ -280,7 +280,8 @@ Transfer-Encoding: chunked
manually setting Transfer-Encoding. Ignored.
=== TEST 8: response.set_header() with header table
=== TEST 9: response.set_header() with header table
--- http_config eval: $t::Util::HttpConfig
--- config
location = /t {
Expand Down
2 changes: 0 additions & 2 deletions t/01-pdk/08-response/11-exit.t
Original file line number Diff line number Diff line change
Expand Up @@ -1178,5 +1178,3 @@ Content-Type: application/json; charset=utf-8
{"n":9007199254740992}
--- no_error_log
[error]
1 change: 1 addition & 0 deletions t/01-pdk/08-response/13-error.t
Original file line number Diff line number Diff line change
Expand Up @@ -454,6 +454,7 @@ grpc-message: ResourceExhausted
[error]
=== TEST 15: service.response.error() honors values of multiple Accept headers
--- http_config eval: $t::Util::HttpConfig
--- config
Expand Down
3 changes: 1 addition & 2 deletions t/01-pdk/10-nginx/02-get_statistics.t
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ run_tests();

__DATA__
=== TEST 1: nginx.get_statistics()
returns Nginx connections and requests states
--- http_config eval: $t::Util::HttpConfig
Expand Down Expand Up @@ -54,4 +53,4 @@ connections_waiting: \d+
connections_accepted: \d+
connections_handled: \d+/
--- no_error_log
[error]
[error]
2 changes: 2 additions & 0 deletions t/01-pdk/15-tracing/01-context.t
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,7 @@ nil
[error]
=== TEST 2: tracer.set_active_span() sets active span
--- http_config eval: $t::Util::HttpConfig
--- config
Expand All @@ -57,6 +58,7 @@ access
[error]
=== TEST 3: tracer.active_span() get tracer from active span
--- http_config eval: $t::Util::HttpConfig
--- config
Expand Down
1 change: 1 addition & 0 deletions t/03-dns-client/02-timer-usage.t
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@ no_shuffle();
run_tests();

__DATA__

=== TEST 1: stale result triggers async timer
--- config
location = /t {
Expand Down
1 change: 1 addition & 0 deletions t/04-patch/01-ngx-buf-double-free.t
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ repeat_each(2);
run_tests();

__DATA__
=== TEST 1: one buf was linked to multiple ngx_chain_t nodes
--- config
location /t {
Expand Down
4 changes: 3 additions & 1 deletion t/04-patch/02-ngx-read-body-block.t
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,8 @@ true err: nil
[error]
[alert]



=== TEST 2: ngx.req.read_body() should work for HTTP2 POST requests that doesn't carry the content-length header
--- config
location = /test {
Expand All @@ -46,4 +48,4 @@ Content-Length:
true err: nil
--- no_error_log
[error]
[alert]
[alert]

0 comments on commit a89b4d1

Please sign in to comment.