Laravel 10.x Shift #49

Merged
nomadjimbob merged 17 commits from shift-91885 into main 2023-05-24 21:47:14 +00:00
67 changed files with 118 additions and 114 deletions
Showing only changes of commit 979b9f704c - Show all commits

View File

@@ -45,7 +45,7 @@ class MediaMigrate extends Command
*
* @return void
*/
public function handle()
public function handle(): void
{
$replace = $this->option('replace');

View File

@@ -51,7 +51,7 @@ class MediaRebuild extends Command
*
* @return void
*/
public function handle()
public function handle(): void
{
$replace = $this->option('replace');
$all = $this->option('replace');

View File

@@ -13,7 +13,7 @@ class Kernel extends ConsoleKernel
* @param \Illuminate\Console\Scheduling\Schedule $schedule The schedule.
* @return void
*/
protected function schedule(Schedule $schedule)
protected function schedule(Schedule $schedule): void
{
// $schedule->command('inspire')->hourly();
}
@@ -23,7 +23,7 @@ class Kernel extends ConsoleKernel
*
* @return void
*/
protected function commands()
protected function commands(): void
{
$this->load(__DIR__ . '/Commands');

View File

@@ -29,7 +29,7 @@ class Handler extends ExceptionHandler
*
* @return void
*/
public function register()
public function register(): void
{
// $this->renderable(function (HttpException $e, $request) {
// if ($request->is('api/*')) {

View File

@@ -12,7 +12,7 @@ class Authenticate extends Middleware
* @param mixed $request Request.
* @return string|null
*/
protected function redirectTo(mixed $request)
protected function redirectTo(mixed $request): ?string
{
if ($request->expectsJson() === false) {
return route('login');

View File

@@ -2,6 +2,7 @@
namespace App\Http\Middleware;
use Symfony\Component\HttpFoundation\Response;
use Closure;
use Illuminate\Http\Request;
@@ -14,7 +15,7 @@ class ForceJsonResponse
* @param \Closure(\Illuminate\Http\Request): (\Illuminate\Http\Response|\Illuminate\Http\RedirectResponse) $next
* @return \Illuminate\Http\Response|\Illuminate\Http\RedirectResponse
*/
public function handle(Request $request, Closure $next)
public function handle(Request $request, Closure $next): Response
{
$request->headers->set('Accept', 'application/json');
return $next($request);

View File

@@ -2,6 +2,7 @@
namespace App\Http\Middleware;
use Symfony\Component\HttpFoundation\Response;
use Closure;
use Illuminate\Http\Request;
use App\Models\Analytics;
@@ -15,7 +16,7 @@ class LogRequest
* @param \Closure(\Illuminate\Http\Request): (\Illuminate\Http\Response|\Illuminate\Http\RedirectResponse) $next
* @return \Illuminate\Http\Response|\Illuminate\Http\RedirectResponse
*/
public function handle(Request $request, Closure $next)
public function handle(Request $request, Closure $next): Response
{
// Make it an after middleware
$response = $next($request);

View File

@@ -2,6 +2,7 @@
namespace App\Http\Middleware;
use Symfony\Component\HttpFoundation\Response;
use App\Providers\RouteServiceProvider;
use Closure;
use Illuminate\Http\Request;
@@ -17,7 +18,7 @@ class RedirectIfAuthenticated
* @param string|null ...$guards Guards.
* @return Response|RedirectResponse
*/
public function handle(Request $request, Closure $next, ...$guards)
public function handle(Request $request, Closure $next, string ...$guards): Response
{
$guards = empty($guards) === true ? [null] : $guards;

View File

@@ -2,6 +2,7 @@
namespace App\Http\Middleware;
use Symfony\Component\HttpFoundation\Response;
use Closure;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
@@ -15,7 +16,7 @@ class UseSanctumGuard
* @param \Closure(\Illuminate\Http\Request): (\Illuminate\Http\Response|\Illuminate\Http\RedirectResponse) $next
* @return \Illuminate\Http\Response|\Illuminate\Http\RedirectResponse
*/
public function handle(Request $request, Closure $next)
public function handle(Request $request, Closure $next): Response
{
Auth::shouldUse('sanctum');
return $next($request);

View File

@@ -11,7 +11,7 @@ class AuthLoginRequest extends FormRequest
*
* @return array<string, mixed>
*/
public function rules()
public function rules(): array
{
return [
'email' => 'required|string|min:6|max:255',

View File

@@ -12,7 +12,7 @@ class BaseRequest extends FormRequest
*
* @return boolean
*/
public function authorize()
public function authorize(): bool
{
if (request()->isMethod('post') === true && method_exists($this, 'postAuthorize') === true) {
return $this->postAuthorize();
@@ -30,7 +30,7 @@ class BaseRequest extends FormRequest
*
* @return array<string, mixed>
*/
public function rules()
public function rules(): array
{
$rules = [];

View File

@@ -12,7 +12,7 @@ class ContactSendRequest extends FormRequest
*
* @return array<string, mixed>
*/
public function rules()
public function rules(): array
{
return [
'name' => 'required|max:255',

View File

@@ -37,7 +37,7 @@ class SubscriptionRequest extends BaseRequest
*
* @return array
*/
public function messages()
public function messages(): array
{
return [
'email.unique' => 'This email address has already subscribed',

View File

@@ -12,7 +12,7 @@ class UserForgotPasswordRequest extends FormRequest
*
* @return array<string, mixed>
*/
public function rules()
public function rules(): array
{
return [
'email' => 'required|exists:users,email',

View File

@@ -12,7 +12,7 @@ class UserRegisterRequest extends FormRequest
*
* @return array<string, mixed>
*/
public function rules()
public function rules(): array
{
return [
'display_name' => ['required','string','max:255', new Uniqueish('users')],

View File

@@ -12,7 +12,7 @@ class UserResendVerifyEmailRequest extends FormRequest
*
* @return array<string, mixed>
*/
public function rules()
public function rules(): array
{
return [
'email' => 'required|exists:users,email',

View File

@@ -12,7 +12,7 @@ class UserResetPasswordRequest extends FormRequest
*
* @return array<string, mixed>
*/
public function rules()
public function rules(): array
{
return [
'code' => 'required|digits:6',

View File

@@ -12,7 +12,7 @@ class UserVerifyEmailRequest extends FormRequest
*
* @return array<string, mixed>
*/
public function rules()
public function rules(): array
{
return [
'code' => 'required|digits:6',

View File

@@ -51,7 +51,7 @@ class MoveMediaJob implements ShouldQueue
*
* @return void
*/
public function handle()
public function handle(): void
{
// Don't continue if the media is already on the new storage disk
if ($this->media->storage === $this->newStorage) {

View File

@@ -50,7 +50,7 @@ class SendEmailJob implements ShouldQueue
*
* @return void
*/
public function handle()
public function handle(): void
{
Mail::to($this->to)->send($this->mailable);
}

View File

@@ -65,7 +65,7 @@ class StoreUploadedFileJob implements ShouldQueue
*
* @return void
*/
public function handle()
public function handle(): void
{
$storageDisk = $this->media->storage;
$fileName = $this->media->name;

View File

@@ -19,7 +19,7 @@ class AppServiceProvider extends ServiceProvider
*
* @return void
*/
public function register()
public function register(): void
{
//
}
@@ -29,7 +29,7 @@ class AppServiceProvider extends ServiceProvider
*
* @return void
*/
public function boot()
public function boot(): void
{
Storage::macro('public', function ($diskName) {
$public = config("filesystems.disks.{$diskName}.public", false);

View File

@@ -22,7 +22,7 @@ class AuthServiceProvider extends ServiceProvider
*
* @return void
*/
public function boot()
public function boot(): void
{
//
}

View File

@@ -29,7 +29,7 @@ class EventServiceProvider extends ServiceProvider
*
* @return void
*/
public function boot()
public function boot(): void
{
//
}
@@ -39,7 +39,7 @@ class EventServiceProvider extends ServiceProvider
*
* @return boolean
*/
public function shouldDiscoverEvents()
public function shouldDiscoverEvents(): bool
{
return false;
}

View File

@@ -26,7 +26,7 @@ class RouteServiceProvider extends ServiceProvider
*
* @return void
*/
public function boot()
public function boot(): void
{
// RateLimiter::for('api', function (Request $request) {
// return Limit::perMinute(60)->by($request->user()?->id !== null ?: $request->ip());

View File

@@ -15,7 +15,7 @@ class ArticleFactory extends Factory
*
* @return array<string, mixed>
*/
public function definition()
public function definition(): array
{
$publishDate = Carbon::parse($this->faker->dateTimeBetween('-1 month', '+1 month'));

View File

@@ -15,7 +15,7 @@ class EventFactory extends Factory
*
* @return array<string, mixed>
*/
public function definition()
public function definition(): array
{
$startDate = Carbon::parse($this->faker->dateTimeBetween('now', '+1 year'));
$endDate = Carbon::parse($this->faker->dateTimeBetween($startDate, '+1 year'));

View File

@@ -15,7 +15,7 @@ class MediaFactory extends Factory
*
* @return array<string, mixed>
*/
public function definition()
public function definition(): array
{
return [
'title' => $this->faker->sentence(),

View File

@@ -15,7 +15,7 @@ class UserFactory extends Factory
*
* @return array<string, mixed>
*/
public function definition()
public function definition(): array
{
$faker = \Faker\Factory::create();
$faker->addProvider(new \Faker\Provider\CustomInternetProvider($faker));

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::create('users', function (Blueprint $table) {
$table->uuid('id')->primary();
@@ -33,7 +33,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::dropIfExists('users');
}

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::create('password_resets', function (Blueprint $table) {
$table->string('email')->index();
@@ -25,7 +25,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::dropIfExists('password_resets');
}

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::create('failed_jobs', function (Blueprint $table) {
$table->id();
@@ -29,7 +29,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::dropIfExists('failed_jobs');
}

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::create('personal_access_tokens', function (Blueprint $table) {
$table->id();
@@ -30,7 +30,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::dropIfExists('personal_access_tokens');
}

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::create('posts', function (Blueprint $table) {
$table->uuid('id')->primary();
@@ -30,7 +30,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::dropIfExists('posts');
}

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::create('media', function (Blueprint $table) {
$table->uuid('id')->primary();
@@ -30,7 +30,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::dropIfExists('media');
}

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::create('permissions', function (Blueprint $table) {
$table->uuid('id')->primary();
@@ -28,7 +28,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::dropIfExists('permissions');
}

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::create('events', function (Blueprint $table) {
$table->uuid('id')->primary();
@@ -35,7 +35,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::dropIfExists('events');
}

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::create('subscriptions', function (Blueprint $table) {
$table->uuid('id')->primary();
@@ -26,7 +26,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::dropIfExists('subscriptions');
}

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::create('jobs', function (Blueprint $table) {
$table->bigIncrements('id');
@@ -29,7 +29,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::dropIfExists('jobs');
}

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::create('user_codes', function (Blueprint $table) {
$table->id();
@@ -30,7 +30,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::dropIfExists('user_codes');
}

View File

@@ -12,7 +12,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::connection(config('audit.drivers.database.connection', config('database.default')))->create('audits', function (Blueprint $table) {
@@ -40,7 +40,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::connection(config('audit.drivers.database.connection', config('database.default')))->drop('audits');
}

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::create('user_logins', function (Blueprint $table) {
$table->uuid('id')->primary();
@@ -32,7 +32,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::dropIfExists('user_logins');
}

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::create('analytics', function (Blueprint $table) {
$table->id();
@@ -28,7 +28,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::dropIfExists('analytics');
}

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::create('attachments', function (Blueprint $table) {
$table->id();
@@ -28,7 +28,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::dropIfExists('attachments');
}

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::table('events', function (Blueprint $table) {
$table->string('price')->default("");
@@ -23,7 +23,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::table('events', function (Blueprint $table) {
$table->dropColumn('price');

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::table('events', function (Blueprint $table) {
$table->string('ages')->default("");
@@ -23,7 +23,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::table('events', function (Blueprint $table) {
$table->dropColumn('ages');

View File

@@ -12,7 +12,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
DB::table('media')->whereNull('mime')->update(['mime' => '']);
DB::table('media')->whereNull('permission')->update(['permission' => '']);
@@ -40,7 +40,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::table('media', function (Blueprint $table) {
$table->bigInteger('size')->change();

View File

@@ -12,7 +12,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
DB::table('users')->whereNull('phone')->update(['phone' => '']);
@@ -26,7 +26,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::table('users', function (Blueprint $table) {
$table->string('phone')->nullable(true)->change();

View File

@@ -12,7 +12,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::table('users', function (Blueprint $table) {
$table->string('display_name')->default("");
@@ -31,7 +31,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::table('users', function (Blueprint $table) {
$table->dropColumn('display_name');

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::dropIfExists('subscriptions');
}
@@ -21,7 +21,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::create('subscriptions', function (Blueprint $table) {
$table->uuid('id')->primary();

View File

@@ -12,7 +12,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::rename('posts', 'articles');
@@ -25,7 +25,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::rename('articles', 'posts');

View File

@@ -12,7 +12,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::table('analytics', function (Blueprint $table) {
$table->bigInteger('session')->nullable(false);
@@ -100,7 +100,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::table('analytics', function (Blueprint $table) {
$table->dropColumn('session');

View File

@@ -12,7 +12,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::table('users', function (Blueprint $table) {
$table->dropColumn('username');
@@ -24,7 +24,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::table('users', function (Blueprint $table) {
$table->string('username')->unique();

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::create('shortlinks', function (Blueprint $table) {
$table->id();
@@ -26,7 +26,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::dropIfExists('shortlinks');
}

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::table('shortlinks', function (Blueprint $table) {
$table->bigInteger('used')->default(0);
@@ -23,7 +23,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::table('shortlinks', function (Blueprint $table) {
//

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::table('users', function (Blueprint $table) {
$table->string('first_name')->default('')->change();
@@ -24,7 +24,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::table('users', function (Blueprint $table) {
$table->string('first_name')->nullable(false)->change();

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::table('events', function (Blueprint $table) {
$table->string('location_url')->default('');
@@ -23,7 +23,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::table('events', function (Blueprint $table) {
$table->dropColumn('location_url');

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::table('attachments', function (Blueprint $table) {
$table->boolean('private')->default(false);
@@ -23,7 +23,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::table('attachments', function (Blueprint $table) {
$table->dropColumn('private');

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::create('event_users', function (Blueprint $table) {
$table->id();
@@ -29,7 +29,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::dropIfExists('event_users');
}

View File

@@ -11,7 +11,7 @@ return new class extends Migration
*
* @return void
*/
public function up()
public function up(): void
{
Schema::rename('password_resets', 'password_reset_tokens');
}
@@ -21,7 +21,7 @@ return new class extends Migration
*
* @return void
*/
public function down()
public function down(): void
{
Schema::rename('password_reset_tokens', 'password_resets');
}

View File

@@ -15,7 +15,7 @@ class DatabaseSeeder extends Seeder
*
* @return void
*/
public function run()
public function run(): void
{
\App\Models\User::factory(40)->create();

View File

@@ -20,7 +20,7 @@ class ArticlesApiTest extends TestCase
$this->faker = FakerFactory::create();
}
public function testAnyUserCanViewArticle()
public function testAnyUserCanViewArticle(): void
{
// Create an event
$article = Article::factory()->create([
@@ -51,7 +51,7 @@ class ArticlesApiTest extends TestCase
]);
}
public function testAdminCanCreateUpdateDeleteArticle()
public function testAdminCanCreateUpdateDeleteArticle(): void
{
// Create a user with the admin/events permission
$adminUser = User::factory()->create();
@@ -102,7 +102,7 @@ class ArticlesApiTest extends TestCase
]);
}
public function testNonAdminCannotCreateUpdateDeleteArticle()
public function testNonAdminCannotCreateUpdateDeleteArticle(): void
{
// Create a user without admin/events permission
$user = User::factory()->create();

View File

@@ -9,7 +9,7 @@ class AuthApiTest extends TestCase
use RefreshDatabase;
public function testLogin()
public function testLogin(): void
{
$user = User::factory()->create([
'password' => bcrypt('password'),

View File

@@ -8,7 +8,7 @@ class ContactFormTest extends TestCase
use RefreshDatabase;
public function testContactForm()
public function testContactForm(): void
{
$formData = [
'name' => 'John Doe',

View File

@@ -21,7 +21,7 @@ class EventsApiTest extends TestCase
$this->faker = FakerFactory::create();
}
public function testAnyUserCanViewEvent()
public function testAnyUserCanViewEvent(): void
{
// Create an event
$event = Event::factory()->create([
@@ -52,7 +52,7 @@ class EventsApiTest extends TestCase
]);
}
public function testAnyUserCannotSeeDraftEvent()
public function testAnyUserCannotSeeDraftEvent(): void
{
// Create a draft event
$draftEvent = Event::factory()->create([
@@ -85,7 +85,7 @@ class EventsApiTest extends TestCase
]);
}
public function testAdminCanCreateUpdateDeleteEvent()
public function testAdminCanCreateUpdateDeleteEvent(): void
{
// Create a user with the admin/events permission
$adminUser = User::factory()->create();
@@ -139,7 +139,7 @@ class EventsApiTest extends TestCase
]);
}
public function testNonAdminCannotCreateUpdateDeleteEvent()
public function testNonAdminCannotCreateUpdateDeleteEvent(): void
{
// Create a user without admin/events permission
$user = User::factory()->create();

View File

@@ -10,7 +10,7 @@ class UsersApiTest extends TestCase
use RefreshDatabase;
public function testNonAdminUsersCanOnlyViewBasicUserInfo()
public function testNonAdminUsersCanOnlyViewBasicUserInfo(): void
{
// create a non-admin user
$nonAdminUser = User::factory()->create();
@@ -71,7 +71,7 @@ class UsersApiTest extends TestCase
]);
}
public function testGuestCannotCreateUser()
public function testGuestCannotCreateUser(): void
{
$userData = [
'email' => 'johndoe@example.com',
@@ -85,7 +85,7 @@ class UsersApiTest extends TestCase
]);
}
public function testGuestCanRegisterUser()
public function testGuestCanRegisterUser(): void
{
$userData = [
'first_name' => 'John',
@@ -102,7 +102,7 @@ class UsersApiTest extends TestCase
]);
}
public function testCannotCreateDuplicateEmailOrDisplayName()
public function testCannotCreateDuplicateEmailOrDisplayName(): void
{
$userData = [
'display_name' => 'JackDoe',
@@ -125,7 +125,7 @@ class UsersApiTest extends TestCase
$response->assertJsonValidationErrors(['display_name', 'email']);
}
public function testUserCanOnlyUpdateOwnUser()
public function testUserCanOnlyUpdateOwnUser(): void
{
$user = User::factory()->create();
@@ -153,7 +153,7 @@ class UsersApiTest extends TestCase
$response->assertStatus(403);
}
public function testUserCannotDeleteUsers()
public function testUserCannotDeleteUsers(): void
{
$user = User::factory()->create();
@@ -169,7 +169,7 @@ class UsersApiTest extends TestCase
$this->assertDatabaseHas('users', ['id' => $otherUser->id]);
}
public function testAdminCanUpdateAnyUser()
public function testAdminCanUpdateAnyUser(): void
{
$admin = User::factory()->create();
$admin->givePermission('admin/users');
@@ -204,7 +204,7 @@ class UsersApiTest extends TestCase
]);
}
public function testAdminCanDeleteAnyUser()
public function testAdminCanDeleteAnyUser(): void
{
$admin = User::factory()->create();
$admin->givePermission('admin/users');

View File

@@ -11,7 +11,7 @@ class ExampleTest extends TestCase
*
* @return void
*/
public function test_that_true_is_true()
public function test_that_true_is_true(): void
{
$this->assertTrue(true);
}