Open
Conversation
…e matrix Add a `discover` job that finds all directories containing .bru files under tests/bruno/e2e/ and outputs them as a JSON array for the matrix. The e2e-testing job now uses fromJSON() to consume this dynamic list. funding-tx-verification is excluded from the base matrix since it requires specific FUNDING_TX_VERIFICATION_CASE args and is handled entirely through the include entries. This ensures new e2e test suites are automatically included in CI without manually updating the workflow file, solving the problem from #1193. Co-authored-by: doitian <35768+doitian@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] [1193] Solve the problem by finding all e2e test suites dynamically
ci: dynamically discover e2e test suites
Mar 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The e2e workflow matrix was hardcoded, causing new test suites (like
shutdown-force,period-check/force-close-expiry) to be silently skipped in CI until someone noticed and manually added them.discoverjob that finds all directories containing.brufiles undertests/bruno/e2e/and outputs them as a JSON arrayfromJSON(needs.discover.outputs.workflows)so new suites are picked up automaticallyfunding-tx-verificationfrom the base matrix — it requires specificFUNDING_TX_VERIFICATION_CASEargs and is handled entirely throughincludeentriesincludeentries unchanged (udt/xudt-test, 3-nodes-transfer/sha256, 3× funding-tx-verification)Discovery logic:
Uses sparse checkout for the discover job to keep it fast.
💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.