CLI-1273: Cryptic errors for 500 responses #732
danepowellmt.yml
on: pull_request
Mutation Testing Code Review Annotations
16s
Annotations
3 warnings
|
Mutation Testing Code Review Annotations
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Mutation Testing Code Review Annotations:
src/Connector/Client.php#L155
Escaped Mutant for Mutator "IncrementInteger":
--- Original
+++ New
@@ @@
public function processResponse(ResponseInterface $response): mixed
{
// Internal server errors return HTML instead of JSON, breaking our parsing.
- if ($response->getStatusCode() === 500) {
+ if ($response->getStatusCode() === 501) {
throw new RuntimeException('Cloud API internal server error. Status ' . $response->getStatusCode() . '. Request ID ' . $response->getHeaderLine('X-Request-Id'));
}
$body_json = $response->getBody();
|
|
Mutation Testing Code Review Annotations:
src/Connector/Client.php#L155
Escaped Mutant for Mutator "DecrementInteger":
--- Original
+++ New
@@ @@
public function processResponse(ResponseInterface $response): mixed
{
// Internal server errors return HTML instead of JSON, breaking our parsing.
- if ($response->getStatusCode() === 500) {
+ if ($response->getStatusCode() === 499) {
throw new RuntimeException('Cloud API internal server error. Status ' . $response->getStatusCode() . '. Request ID ' . $response->getHeaderLine('X-Request-Id'));
}
$body_json = $response->getBody();
|