From 650376066b187819dbb69d8d30c667cafb2bfad3 Mon Sep 17 00:00:00 2001 From: Nic Date: Fri, 26 Jun 2026 15:01:53 +0800 Subject: [PATCH 1/2] ci: run proxy-cache and dubbo-proxy plugin subdirectory tests The `t/plugin/[a-k]*.t` / `t/plugin/[l-z]*.t` matrix globs only match top-level `.t` files, so plugin tests that live in subdirectories were never executed by CI. This silently dropped coverage for: t/plugin/proxy-cache/{disk,memory}.t t/plugin/dubbo-proxy/{route,upstream}.t The dubbo backend is already started in build.yml, so the dubbo-proxy tests were prepared to run but the glob never reached them. Add the two directories to the matrix so `prove -r` recurses them. The ext-plugin subdirectory tests are intentionally left out for now: several of them require a real external plugin runner that CI does not currently set up, so enabling them needs separate work. --- .github/workflows/build.yml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index f600c983faac..6ba9126cf0f8 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -29,8 +29,8 @@ jobs: os_name: - linux_openresty test_dir: - - t/plugin/[a-k]*.t - - t/stream-plugin t/plugin/[l-z]*.t + - t/plugin/[a-k]*.t t/plugin/dubbo-proxy + - t/stream-plugin t/plugin/[l-z]*.t t/plugin/proxy-cache - t/admin t/cli t/config-center-yaml t/control t/core t/debug t/discovery t/error_page t/http3/admin t/misc - t/node t/pubsub t/router t/script t/secret t/stream-node t/utils t/wasm t/xds-library t/xrpc From 414a9355cfe62333cd39e247eaa89d90ca595518 Mon Sep 17 00:00:00 2001 From: Nic Date: Fri, 26 Jun 2026 16:30:59 +0800 Subject: [PATCH 2/2] ci: keep this change to proxy-cache only Enabling dubbo-proxy surfaced a stale test: dubbo-proxy/route.t TEST 1 fails because current apisix logs `config_yaml.lua:339: unknown plugin [dubbo-proxy]` when dubbo is disabled globally, and the test's implicit no_error_log check rejects it. The route still works (returns "hello world"); only the test's error_log expectation is out of date from never being run. That's a separate test fix, so scope this PR to the proxy-cache directory, which passes as-is. --- .github/workflows/build.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 6ba9126cf0f8..a8cef92eacbb 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -29,7 +29,7 @@ jobs: os_name: - linux_openresty test_dir: - - t/plugin/[a-k]*.t t/plugin/dubbo-proxy + - t/plugin/[a-k]*.t - t/stream-plugin t/plugin/[l-z]*.t t/plugin/proxy-cache - t/admin t/cli t/config-center-yaml t/control t/core t/debug t/discovery t/error_page t/http3/admin t/misc - t/node t/pubsub t/router t/script t/secret t/stream-node t/utils t/wasm t/xds-library t/xrpc