Skip to content

Commit fdf656b

Browse files
committed
Fix CI
1 parent ef7e06a commit fdf656b

File tree

2 files changed

+16
-8
lines changed

2 files changed

+16
-8
lines changed

.github/workflows/php.yml

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,8 @@ jobs:
5353
uses: shivammathur/setup-php@v2
5454
with:
5555
php-version: ${{ matrix.php-versions }}
56-
extensions: ctype, date, dom, fileinfo, filter, hash, intl, mbstring, openssl, pcre, posix, spl, xml
56+
extensions: ctype, date, dom, fileinfo, filter, hash, intl, mbstring, openssl, pcre, posix,\
57+
session, sodium, spl, xml
5758
tools: composer
5859
ini-values: error_reporting=E_ALL
5960
coverage: pcov
@@ -115,7 +116,8 @@ jobs:
115116
uses: shivammathur/setup-php@v2
116117
with:
117118
php-version: ${{ matrix.php-versions }}
118-
extensions: ctype, date, dom, fileinfo, filter, hash, intl, mbstring, openssl, pcre, posix, spl, xml, zip
119+
extensions: ctype, date, dom, fileinfo, filter, hash, intl, mbstring, openssl, pcre, posix,\
120+
session, sodium, spl, xml, zip
119121
tools: composer
120122
ini-values: error_reporting=E_ALL
121123
coverage: none
@@ -162,8 +164,9 @@ jobs:
162164
with:
163165
# Should be the higest supported version, so we can use the newest tools
164166
php-version: '8.5'
165-
tools: composer, composer-require-checker, composer-unused, phpcs, phpstan
166-
extensions: ctype, date, dom, fileinfo, filter, hash, intl, mbstring, openssl, pcre, posix, spl, xml
167+
tools: composer, composer-require-checker, composer-unused
168+
extensions: ctype, date, dom, fileinfo, filter, hash, intl, mbstring, openssl, pcre, posix,\
169+
session, sodium, spl, xml
167170

168171
- name: Setup problem matchers for PHP
169172
run: echo "::add-matcher::${{ runner.tool_cache }}/php.json"
@@ -193,7 +196,7 @@ jobs:
193196
run: composer-unused
194197

195198
- name: PHP Code Sniffer
196-
run: phpcs
199+
run: vendor/bin/phpcs
197200

198201
- name: PHPStan
199202
run: |
@@ -215,7 +218,8 @@ jobs:
215218
with:
216219
# Should be the lowest supported version
217220
php-version: '8.3'
218-
extensions: ctype, date, dom, fileinfo, filter, hash, intl, mbstring, openssl, pcre, posix, spl, xml
221+
extensions: ctype, date, dom, fileinfo, filter, hash, intl, mbstring, openssl, pcre, posix,\
222+
session, sodium, spl, xml
219223
tools: composer
220224
coverage: none
221225

tests/src/Controller/CDCTest.php

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,9 @@ public function testResumeNoDomain(): void
117117
$c = new Controller\CDC(self::$config, self::$session);
118118

119119
$this->expectException(Error\BadRequest::class);
120-
$this->expectExceptionMessage("BADREQUEST('%REASON%' => 'Missing domain to CDC resume handler.')");
120+
$this->expectExceptionMessage(
121+
'{"errorCode":"BADREQUEST","%REASON%":"Missing domain to CDC resume handler."}',
122+
);
121123

122124
$c->resume($request);
123125
}
@@ -178,7 +180,9 @@ public function testResumeDomain(): void
178180

179181
// @TODO: Inject Server & Client objects and test entire workflow
180182
$this->expectException(Error\BadRequest::class);
181-
$this->expectExceptionMessage("BADREQUEST('%REASON%' => 'Missing CDC response to CDC resume handler.')");
183+
$this->expectExceptionMessage(
184+
'{"errorCode":"BADREQUEST","%REASON%":"Missing CDC response to CDC resume handler."}',
185+
);
182186
// $this->assertTrue($response->isSuccessful());
183187
// $this->assertInstanceOf(RunnableResponse::class, $response);
184188

0 commit comments

Comments
 (0)