-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Magento#25669: fixed issue "health_check.php fails if any database cache engine configured" #25722
Magento#25669: fixed issue "health_check.php fails if any database cache engine configured" #25722
Conversation
…nfigured - fixed wrong "@return description" of the remove method in class Magento\Framework\Cache\Backend\Database - fixed wrong returned type of the method "unlock" in class Magento\Framework\Lock\Backend\Cache - fixed health_check to create instances into block "check cache storage availability"
Hi @andrewbess. Thank you for your contribution
For more details, please, review the Magento Contributor Guide documentation. |
…nfigured - fixed PHPDocs according to requirements static tests for CE
4d4e755
to
3905038
Compare
https://prnt.sc/q1rujl |
@@ -45,7 +45,7 @@ public function lock(string $name, int $timeout = -1): bool | |||
*/ | |||
public function unlock(string $name): bool | |||
{ | |||
return $this->cache->remove($this->getIdentifier($name)); | |||
return (bool)$this->cache->remove($this->getIdentifier($name)); |
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.
Could I kindly ask you to cover this part using a simple unit test? Just mock a behavior that caused an error previously for $this->cache->remove($this->getIdentifier($name))
and make sure that there's no error after the proposed change.
Thank you!
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.
Hello @rogyar
Tests have already created.
Thank you @engcom-Echo
I will take care of test coverage. |
Hi @rogyar, thank you for the review. |
✔️ QA passed |
…ase cache engine configured" #25722
Hi @andrewbess, thank you for your contribution! |
Description (*)
This PR fixes the issue /issues/25669: health_check.php fails if any database cache engine configured.
Fixed Issues (if relevant)
Manual testing scenarios (*)
Questions or comments
Contribution checklist (*)