-
Notifications
You must be signed in to change notification settings - Fork 47
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CLI-1302: [api:applications:environment-create] databases must be array #1715
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1715 +/- ##
============================================
+ Coverage 91.37% 91.40% +0.02%
+ Complexity 1801 1800 -1
============================================
Files 121 121
Lines 6460 6458 -2
============================================
Hits 5903 5903
+ Misses 557 555 -2 ☔ View full report in Codecov by Sentry. |
// @todo Remove this! It is a workaround for CLI-769. | ||
elseif ($parameterSpecification['in'] === 'header') { | ||
$command->addPostParameter($parameterDefinition->getName(), $parameterSpecification); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tested to confirm this is no longer necessary
@@ -21,7 +21,6 @@ class ApiCommandTest extends CommandTestBase { | |||
|
|||
public function setUp(): void { | |||
parent::setUp(); | |||
$this->clientProphecy->addOption('headers', ['Accept' => 'application/json']); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is necessary to kill an escaped mutant. We need to check that this method is called, but it isn't actually called on every test case.
No description provided.