Skip to content
This repository was archived by the owner on Apr 8, 2024. It is now read-only.

Commit 37f6110

Browse files
committed
Merge branch 'release/2.1.0'
2 parents c2816c8 + 36236cd commit 37f6110

File tree

3 files changed

+4
-70
lines changed

3 files changed

+4
-70
lines changed

VERSION

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
2.0.2
1+
2.1.0

src/Commands/PurgeCommand.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -169,7 +169,7 @@ protected function purgeExpiredRecordsForModel($model, $days)
169169
return false;
170170
}
171171

172-
if (!method_exists($model, 'onlyTrashed') || !method_exists($model, 'forceDelete')) {
172+
if (!method_exists($model, 'forceDelete')) {
173173
$this->recordMessage(sprintf("The model [%s] does not support soft deleting.", $model), 'error');
174174

175175
return false;

tests/Commands/PurgeCommandTest.php

Lines changed: 2 additions & 68 deletions
Original file line numberDiff line numberDiff line change
@@ -217,7 +217,7 @@ public function it_writes_a_comment_if_there_are_no_models_configured()
217217
* @test
218218
* @group unit
219219
*/
220-
public function it_warns_on_models_in_the_config_that_does_not_exists()
220+
public function it_warns_on_models_in_the_config_that_do_not_exist()
221221
{
222222
$this->config_mock->shouldReceive('get')
223223
->once()
@@ -295,72 +295,7 @@ public function it_warns_on_models_in_the_config_that_does_not_exists()
295295
* @test
296296
* @group unit
297297
*/
298-
public function it_errors_on_models_in_the_config_that_does_not_have_onlyTrashed()
299-
{
300-
$this->config_mock->shouldReceive('get')
301-
->once()
302-
->withArgs(
303-
[
304-
'garbageman.dispatch_purge_events',
305-
false,
306-
]
307-
)
308-
->andReturn(false);
309-
310-
$this->config_mock->shouldReceive('get')
311-
->once()
312-
->withArgs(
313-
[
314-
'garbageman.logging_level',
315-
[
316-
'console' => 6,
317-
'log' => 6,
318-
],
319-
]
320-
)
321-
->andReturn(
322-
[
323-
'console' => 6,
324-
'log' => 6,
325-
]
326-
);
327-
328-
$this->config_mock->shouldReceive('get')
329-
->once()
330-
->withArgs(
331-
[
332-
'garbageman.schedule',
333-
[],
334-
]
335-
)
336-
->andReturn(
337-
[
338-
'NoOnlyTrashed' => 14,
339-
]
340-
);
341-
342-
$this->log_mock->shouldReceive('error')
343-
->once()
344-
->with('The model [NoOnlyTrashed] does not support soft deleting.')
345-
->andReturnNull();
346-
347-
$this->output_mock->shouldReceive('writeln')
348-
->once()
349-
->withArgs(
350-
$this->checkVerbosity(
351-
'<error>The model [NoOnlyTrashed] does not support soft deleting.</error>'
352-
)
353-
)
354-
->andReturnNull();
355-
356-
$this->command->handle();
357-
}
358-
359-
/**
360-
* @test
361-
* @group unit
362-
*/
363-
public function it_errors_on_models_in_the_config_that_does_not_have_forceDelete()
298+
public function it_errors_on_models_in_the_config_that_do_not_have_forceDelete()
364299
{
365300
$this->config_mock->shouldReceive('get')
366301
->once()
@@ -603,7 +538,6 @@ public function it_deletes_all_expired_records_for_models_with_soft_delete_when_
603538
public function it_deletes_each_expired_record_for_models_and_throws_events_with_soft_delete_when_configured_to_dispatch_events(
604539
)
605540
{
606-
607541
$this->config_mock->shouldReceive('get')
608542
->once()
609543
->withArgs(

0 commit comments

Comments
 (0)