diff --git a/routes/console.php b/routes/console.php index 4b1468b0..676fec7f 100644 --- a/routes/console.php +++ b/routes/console.php @@ -1,5 +1,8 @@ comment(Inspiring::quote()); })->purpose('Display an inspiring quote'); -Schedule::command('sync:packagist')->everyTwoHours(); +Schedule::command(SyncPackagistData::class)->everyTwoHours(); // Every two hours at minute 30. -Schedule::command('sync:repo')->cron('30 */2 * * *'); -Schedule::command('purge:abandonedscreenshots')->dailyAt('1:00'); +Schedule::command(SyncRepositoryData::class)->cron('30 */2 * * *'); +Schedule::command(DeleteAbandonedScreenshots::class)->dailyAt('1:00'); // Schedule::command('novapackages:check-package-urls')->weeklyOn(7, '20:00'); // Schedule::command('novapackages:send-unavailable-package-followup')->dailyAt('21:00'); // Schedule::command('novapackages:disable-unavailable-packages')->dailyAt('21:30'); diff --git a/tests/Feature/CheckPackageUrlAvailabilityCommandTest.php b/tests/Feature/CheckPackageUrlAvailabilityCommandTest.php index f1581a53..85fb5525 100644 --- a/tests/Feature/CheckPackageUrlAvailabilityCommandTest.php +++ b/tests/Feature/CheckPackageUrlAvailabilityCommandTest.php @@ -1,5 +1,6 @@ artisan('novapackages:check-package-urls'); + $this->artisan(CheckPackageUrls::class); expect($this->validPackage->marked_as_unavailable_at)->toBeNull(); $this->assertEquals( @@ -59,7 +60,7 @@ $this->packageWithUnavailableUrl->url => Http::response(null, 404), ]); - $this->artisan('novapackages:check-package-urls'); + $this->artisan(CheckPackageUrls::class); Notification::assertNotSentTo( $this->validPackage->author->user, @@ -78,7 +79,7 @@ $this->packageWithUnavailableUrl->marked_as_unavailable_at = now(); $this->packageWithUnavailableUrl->save(); - $this->artisan('novapackages:check-package-urls'); + $this->artisan(CheckPackageUrls::class); Notification::assertNotSentTo( $this->packageWithUnavailableUrl->author->user, diff --git a/tests/Feature/DeleteAbandonedScreenshotsCommandTest.php b/tests/Feature/DeleteAbandonedScreenshotsCommandTest.php index 5049e466..93d99e87 100644 --- a/tests/Feature/DeleteAbandonedScreenshotsCommandTest.php +++ b/tests/Feature/DeleteAbandonedScreenshotsCommandTest.php @@ -1,5 +1,6 @@ Carbon::now(), ]); - $this->artisan('purge:abandonedscreenshots'); + $this->artisan(DeleteAbandonedScreenshots::class); $remainingScreenshots = Screenshot::get(); expect($remainingScreenshots)->toHaveCount(2); diff --git a/tests/Feature/DeleteSelfAuthoredPackageRatingsTest.php b/tests/Feature/DeleteSelfAuthoredPackageRatingsTest.php index b7ab849c..20a2f103 100644 --- a/tests/Feature/DeleteSelfAuthoredPackageRatingsTest.php +++ b/tests/Feature/DeleteSelfAuthoredPackageRatingsTest.php @@ -1,5 +1,6 @@ ratings()->save($validRating); expect($package->fresh()->ratings()->count())->toEqual(2); - $this->artisan('purge:self-authored-package-ratings'); + $this->artisan(DeleteSelfAuthoredPackageRatings::class); tap($package->fresh(), function ($package) use ($otherUser) { expect($package->ratings()->count())->toEqual(1); @@ -57,7 +58,7 @@ $package->ratings()->save($validRating); expect($package->fresh()->ratings()->count())->toEqual(2); - $this->artisan('purge:self-authored-package-ratings'); + $this->artisan(DeleteSelfAuthoredPackageRatings::class); tap($package->fresh(), function ($package) use ($otherUser) { expect($package->ratings()->count())->toEqual(1); diff --git a/tests/Feature/DisableUnavailablePackageCommandTest.php b/tests/Feature/DisableUnavailablePackageCommandTest.php index fef59c10..0ac218f7 100644 --- a/tests/Feature/DisableUnavailablePackageCommandTest.php +++ b/tests/Feature/DisableUnavailablePackageCommandTest.php @@ -1,5 +1,6 @@ id, ]); - $this->artisan('novapackages:disable-unavailable-packages'); + $this->artisan(DisableUnavailablePackages::class); expect($packageThatShouldBeDisabled->refresh()->is_disabled)->toBeTrue(); Notification::assertSentTo( diff --git a/tests/Feature/RemindPackageAuthorOfUnavailablePackageCommandTest.php b/tests/Feature/RemindPackageAuthorOfUnavailablePackageCommandTest.php index 17c498bb..6898a9d2 100644 --- a/tests/Feature/RemindPackageAuthorOfUnavailablePackageCommandTest.php +++ b/tests/Feature/RemindPackageAuthorOfUnavailablePackageCommandTest.php @@ -1,5 +1,6 @@ id, ]); - $this->artisan('novapackages:send-unavailable-package-followup'); + $this->artisan(SendUnavailablePackageFollowUp::class); Notification::assertSentTo( $packageThatShouldReceiveNotification->author->user, diff --git a/tests/Feature/SyncRepositoryDataCommandTest.php b/tests/Feature/SyncRepositoryDataCommandTest.php index 51aee193..2ccff7bf 100644 --- a/tests/Feature/SyncRepositoryDataCommandTest.php +++ b/tests/Feature/SyncRepositoryDataCommandTest.php @@ -1,5 +1,6 @@ fakeRepoFromPackageModel([$repoAttributesA, $repoAttributesB]); - $this->artisan('sync:repo'); + $this->artisan(SyncRepositoryData::class); expect(Package::all())->toHaveCount(2); $packageA->refresh(); @@ -76,7 +77,7 @@ $this->fakeRepoFromPackageModel([$repoAttributesA, $repoAttributesB]); - $this->artisan('sync:repo', ['package' => $packageA->id]); + $this->artisan(SyncRepositoryData::class, ['package' => $packageA->id]); expect(Package::all())->toHaveCount(2); $packageA->refresh(); @@ -110,7 +111,7 @@ 'latest_version' => 'v2.3.4', ]); - $this->artisan('sync:repo', ['package' => $package->id]); + $this->artisan(SyncRepositoryData::class, ['package' => $package->id]); $package->refresh(); expect(Package::all())->toHaveCount(1); expect($package->updated_at->toDateTimeString())->toEqual($updatedAt->toDateTimeString());