Skip to content

Migrate to PHPUnit 12 attributes and add new helper tests - #177

Merged
john-c-houser merged 2 commits into
mainfrom
fix-phpunit12-attributes
Jul 7, 2026
Merged

Migrate to PHPUnit 12 attributes and add new helper tests#177
john-c-houser merged 2 commits into
mainfrom
fix-phpunit12-attributes

Conversation

@john-c-houser

Copy link
Copy Markdown
Collaborator

No description provided.

@john-c-houser
john-c-houser requested a review from Copilot July 7, 2026 15:52

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.

Comment on lines +148 to +150
// Trigger autoloader to load src/Libilsws.php where APIException is defined
$ilsws = new Libilsws($this->dummyYamlPath);
$exception = new \Libilsws\APIException();
Comment on lines +161 to +165
// 2. Test plain text fallback when JSON is invalid
$plainError = 'Raw database connection timeout';
$result2 = $exception->errorMessage($plainError, 500);
$this->assertEquals('HTTP 500: SirsiDynix Web Services unavailable - Raw database connection timeout', $result2);

Comment thread composer.json
Comment on lines 13 to 23
"php": ">=8.1",
"ext-curl": "*",
"ext-json": "*",
"phpmailer/phpmailer": "^6.8",
"phpmailer/phpmailer": ">=7.0",
"symfony/yaml": "^6.0|^7.0",
"twig/twig": "^3.0"
"twig/twig": ">=3.0"
},
"require-dev": {
"phpunit/phpunit": "^10.5",
"symfony/var-dumper": "^6.0"
"phpunit/phpunit": ">=10.5",
"symfony/var-dumper": ">=6.0"
},
@john-c-houser
john-c-houser merged commit 85fc790 into main Jul 7, 2026
2 of 3 checks passed
@john-c-houser
john-c-houser deleted the fix-phpunit12-attributes branch July 7, 2026 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants