-
Notifications
You must be signed in to change notification settings - Fork 0
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
Internalize ResqueStatus, fix deprecated tests. #6
Conversation
d240a26
to
9375df3
Compare
9375df3
to
56941dd
Compare
- uses: "ramsey/composer-install@v2" | ||
with: | ||
composer-options: "--ignore-platform-reqs --optimize-autoloader" | ||
- name: Build the stack |
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.
Через docker compose как-то проще за всем следить, и redis теперь нужен,
@@ -622,7 +621,7 @@ public function resume() | |||
$options->allOption = 'Resume all workers'; | |||
$options->selectMessage = 'Worker to resume'; | |||
$options->actionMessage = 'resuming'; | |||
$options->workers = call_user_func([$this->ResqueStatus, 'getPausedWorker']); | |||
$options->workers = call_user_func([$this->ResqueStatus, 'getPausedWorkers']); |
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.
Автозамена, так бы убрал call_user_func
, не знаю зачем он его везде использовал.
public const WORKER_KEY = 'ResqueWorker'; | ||
public const SCHEDULER_WORKER_KEY = 'ResqueSchedulerWorker'; | ||
public const PAUSED_WORKER_KEY = 'PausedWorker'; |
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->stringContains($datas[0]['queue'])], | ||
)->willReturnOnConsecutiveCalls(0, 9, 10, 3); | ||
|
||
$this->output |
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.
С N раза решил взяться и аккуратно переписать его, получилось, теперь нет больше предупреждений.
No description provided.