Skip to content

Commit 9a30a5b

Browse files
OskarStarkOskarStark
committed
Add lowest, Symfony 7.4 and 8.0 tests for bridges
Dynamically generate matrix includes for both packages and bridges, eliminating hardcoded entries and ensuring new bridges are automatically tested with all PHP/Symfony version combinations. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
1 parent cbda539 commit 9a30a5b

File tree

1 file changed

+30
-58
lines changed

1 file changed

+30
-58
lines changed

‎.github/workflows/unit-tests.yaml‎

Lines changed: 30 additions & 58 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,9 @@ jobs:
2323
runs-on: ubuntu-latest
2424
outputs:
2525
packages: ${{ steps.set-matrix.outputs.packages }}
26+
packages-include: ${{ steps.set-matrix.outputs.packages-include }}
2627
bridges: ${{ steps.set-matrix.outputs.bridges }}
28+
bridges-include: ${{ steps.set-matrix.outputs.bridges-include }}
2729
steps:
2830
- name: Checkout
2931
uses: actions/checkout@v6
@@ -50,6 +52,19 @@ jobs:
5052
done
5153
echo "packages=$PACKAGES" >> $GITHUB_OUTPUT
5254
55+
# Generate package includes (lowest, Symfony 7.4, Symfony 8.0)
56+
PACKAGES_INCLUDE=$(echo "$PACKAGES" | jq -c '
57+
. as $pkgs |
58+
# lowest deps with PHP 8.2
59+
($pkgs | map(. + {"php-version": "8.2", "dependency-version": "lowest"})) +
60+
# Symfony 7.4 LTS with PHP 8.2
61+
($pkgs | map(. + {"php-version": "8.2", "symfony-version": "7.4.*"})) +
62+
# Symfony 8.0 with PHP 8.5
63+
($pkgs | map(. + {"php-version": "8.5", "symfony-version": "8.0.*"}))
64+
| map({package: {path: .path, type: .type, name: .name}} + (. | del(.path, .type, .name)))
65+
')
66+
echo "packages-include=$PACKAGES_INCLUDE" >> $GITHUB_OUTPUT
67+
5368
# Bridges (store and tool)
5469
STORE_BRIDGES=$(ls -1 src/store/src/Bridge/ | sort \
5570
| jq -R -s -c 'split("\n") | map(select(length > 0)) | map({component: "store", type: "Store", bridge: .})')
@@ -58,6 +73,19 @@ jobs:
5873
BRIDGES=$(jq -n -c --argjson store "$STORE_BRIDGES" --argjson tool "$TOOL_BRIDGES" '$store + $tool')
5974
echo "bridges=$BRIDGES" >> $GITHUB_OUTPUT
6075
76+
# Generate bridge includes (lowest, Symfony 7.4, Symfony 8.0)
77+
BRIDGES_INCLUDE=$(echo "$BRIDGES" | jq -c '
78+
. as $bridges |
79+
# lowest deps with PHP 8.2
80+
($bridges | map(. + {"php-version": "8.2", "dependency-version": "lowest"})) +
81+
# Symfony 7.4 LTS with PHP 8.2
82+
($bridges | map(. + {"php-version": "8.2", "symfony-version": "7.4.*"})) +
83+
# Symfony 8.0 with PHP 8.5
84+
($bridges | map(. + {"php-version": "8.5", "symfony-version": "8.0.*"}))
85+
| map({bridge: {component: .component, type: .type, bridge: .bridge}} + (. | del(.component, .type, .bridge)))
86+
')
87+
echo "bridges-include=$BRIDGES_INCLUDE" >> $GITHUB_OUTPUT
88+
6189
# Pretty print for info
6290
echo "### Packages"
6391
echo "$PACKAGES" | jq .
@@ -76,64 +104,7 @@ jobs:
76104
php-version: ['8.2', '8.5']
77105
dependency-version: ['']
78106
symfony-version: ['']
79-
include:
80-
# lowest deps for all packages
81-
- php-version: '8.2'
82-
dependency-version: 'lowest'
83-
package: { path: 'agent', type: 'Component', name: 'Agent' }
84-
- php-version: '8.2'
85-
dependency-version: 'lowest'
86-
package: { path: 'ai-bundle', type: 'Bundle', name: 'AI Bundle' }
87-
- php-version: '8.2'
88-
dependency-version: 'lowest'
89-
package: { path: 'chat', type: 'Component', name: 'Chat' }
90-
- php-version: '8.2'
91-
dependency-version: 'lowest'
92-
package: { path: 'mcp-bundle', type: 'Bundle', name: 'MCP Bundle' }
93-
- php-version: '8.2'
94-
dependency-version: 'lowest'
95-
package: { path: 'platform', type: 'Component', name: 'Platform' }
96-
- php-version: '8.2'
97-
dependency-version: 'lowest'
98-
package: { path: 'store', type: 'Component', name: 'Store' }
99-
# Symfony 7.4 LTS for all packages
100-
- php-version: '8.2'
101-
symfony-version: '7.4.*'
102-
package: { path: 'agent', type: 'Component', name: 'Agent' }
103-
- php-version: '8.2'
104-
symfony-version: '7.4.*'
105-
package: { path: 'ai-bundle', type: 'Bundle', name: 'AI Bundle' }
106-
- php-version: '8.2'
107-
symfony-version: '7.4.*'
108-
package: { path: 'chat', type: 'Component', name: 'Chat' }
109-
- php-version: '8.2'
110-
symfony-version: '7.4.*'
111-
package: { path: 'mcp-bundle', type: 'Bundle', name: 'MCP Bundle' }
112-
- php-version: '8.2'
113-
symfony-version: '7.4.*'
114-
package: { path: 'platform', type: 'Component', name: 'Platform' }
115-
- php-version: '8.2'
116-
symfony-version: '7.4.*'
117-
package: { path: 'store', type: 'Component', name: 'Store' }
118-
# Symfony 8.0 for all packages
119-
- php-version: '8.5'
120-
symfony-version: '8.0.*'
121-
package: { path: 'agent', type: 'Component', name: 'Agent' }
122-
- php-version: '8.5'
123-
symfony-version: '8.0.*'
124-
package: { path: 'ai-bundle', type: 'Bundle', name: 'AI Bundle' }
125-
- php-version: '8.5'
126-
symfony-version: '8.0.*'
127-
package: { path: 'chat', type: 'Component', name: 'Chat' }
128-
- php-version: '8.5'
129-
symfony-version: '8.0.*'
130-
package: { path: 'mcp-bundle', type: 'Bundle', name: 'MCP Bundle' }
131-
- php-version: '8.5'
132-
symfony-version: '8.0.*'
133-
package: { path: 'platform', type: 'Component', name: 'Platform' }
134-
- php-version: '8.5'
135-
symfony-version: '8.0.*'
136-
package: { path: 'store', type: 'Component', name: 'Store' }
107+
include: ${{ fromJson(needs.matrix.outputs.packages-include) }}
137108

138109
env:
139110
SYMFONY_REQUIRE: ${{ matrix.symfony-version || '>=7.4' }}
@@ -174,6 +145,7 @@ jobs:
174145
php-version: ['8.2', '8.5']
175146
dependency-version: ['']
176147
symfony-version: ['']
148+
include: ${{ fromJson(needs.matrix.outputs.bridges-include) }}
177149

178150
env:
179151
SYMFONY_REQUIRE: ${{ matrix.symfony-version || '>=7.4' }}

0 commit comments

Comments
 (0)