membphis commented on PR #13613: URL: https://github.com/apache/apisix/pull/13613#issuecomment-4828563743
P2: `graphql-proxy-cache` subdirectory tests are still not included in CI. This PR fixes the plugin subdirectory coverage gap by explicitly adding `t/plugin/dubbo-proxy` and `t/plugin/proxy-cache`, and it explains why `t/plugin/ext-plugin/` is intentionally left out. However, `t/plugin/graphql-proxy-cache/` also contains runnable `.t` tests (`disk.t`, `graphql.t`, and `memory.t`) and is still not matched by the current matrix: ```yaml - t/plugin/[a-k]*.t t/plugin/dubbo-proxy - t/stream-plugin t/plugin/[l-z]*.t t/plugin/proxy-cache ``` That means this PR would still leave a plugin subdirectory test gap undisclosed. Please either add `t/plugin/graphql-proxy-cache` to the appropriate shard, or document why it is intentionally excluded and should be handled separately. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
