Commit 662f989
refactor: Make ext-redis optional with @requires annotations
Remove ext-redis from require-dev and add @requires extension redis to test classes that need it:
- Remove ext-redis from composer.json require-dev
- Add @requires extension redis to DonutCommandRedisCacheTest
- Add @requires extension redis to StorageRedisDsnModuleMarshallerTest
- Add @requires extension redis to StorageRedisDsnModuleTest
This allows developers to run composer install without Redis extension installed.
Tests requiring Redis will be automatically skipped by PHPUnit when the extension is not available.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>1 parent 7e7a055 commit 662f989
6 files changed
Lines changed: 9 additions & 22 deletions
File tree
- tests-pecl-ext
- tests
- vendor-bin/tool
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
27 | 27 | | |
28 | 28 | | |
29 | 29 | | |
30 | | - | |
31 | 30 | | |
32 | 31 | | |
33 | 32 | | |
| |||
71 | 70 | | |
72 | 71 | | |
73 | 72 | | |
74 | | - | |
75 | | - | |
76 | 73 | | |
77 | 74 | | |
78 | 75 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
23 | 26 | | |
24 | 27 | | |
25 | 28 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
14 | 14 | | |
15 | 15 | | |
16 | 16 | | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
17 | 20 | | |
18 | 21 | | |
19 | 22 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
12 | 15 | | |
13 | 16 | | |
14 | 17 | | |
| |||
This file was deleted.
This file was deleted.
0 commit comments