Commit 0b825ef
committed
[Server] Fix CI: switch warning-log tests to PHPUnit mocks; baseline new conformance scenario
- StreamableHttpTransportTest: replace the anonymous AbstractLogger
fixture with `$this->createMock(LoggerInterface::class)` so the
test does not declare a `log()` signature at all. That sidesteps
both the psr/log v1 LSP incompatibility (untyped parameter) and
the phpstan `missingType.parameter` complaint in one move, and
matches how the rest of the suite (CallToolHandlerTest etc.)
fakes loggers.
- conformance-baseline.yml: mark `server-sse-multiple-streams` as
an expected failure. The new 2025-11-25 conformance scenario
exercises `GET → SSE` for parallel server→client streams, which
the SDK does not implement yet (tracked as #291); the test
becoming visible is a conformance-suite version bump rather
than a regression from this PR.1 parent 3a40738 commit 0b825ef
2 files changed
Lines changed: 9 additions & 25 deletions
File tree
- tests
- Conformance
- Unit/Server/Transport
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
| 2 | + | |
2 | 3 | | |
3 | 4 | | |
4 | 5 | | |
| |||
Lines changed: 7 additions & 24 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
24 | 24 | | |
25 | 25 | | |
26 | 26 | | |
27 | | - | |
| 27 | + | |
28 | 28 | | |
29 | 29 | | |
30 | 30 | | |
| |||
93 | 93 | | |
94 | 94 | | |
95 | 95 | | |
96 | | - | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
97 | 100 | | |
98 | 101 | | |
99 | 102 | | |
| |||
109 | 112 | | |
110 | 113 | | |
111 | 114 | | |
112 | | - | |
113 | | - | |
114 | | - | |
115 | | - | |
116 | | - | |
117 | 115 | | |
118 | 116 | | |
119 | 117 | | |
| |||
123 | 121 | | |
124 | 122 | | |
125 | 123 | | |
126 | | - | |
| 124 | + | |
| 125 | + | |
127 | 126 | | |
128 | 127 | | |
129 | 128 | | |
130 | | - | |
131 | | - | |
132 | | - | |
133 | 129 | | |
134 | 130 | | |
135 | 131 | | |
| |||
278 | 274 | | |
279 | 275 | | |
280 | 276 | | |
281 | | - | |
282 | | - | |
283 | | - | |
284 | | - | |
285 | | - | |
286 | | - | |
287 | | - | |
288 | | - | |
289 | | - | |
290 | | - | |
291 | | - | |
292 | | - | |
293 | | - | |
294 | 277 | | |
0 commit comments