[Tests] Add failing update to-one tests

Author: lindyhopchris

Diff for: tests/dummy/app/Http/Controllers/Api/V1/UserController.php

@@ -26,8 +26,10 @@
 
 class UserController extends Controller
 {
-
     use Actions\FetchOne;
+    use Actions\FetchRelated;
+    use Actions\FetchRelationship;
+    use Actions\UpdateRelationship;
 
     /**
      * Return the current user.

Diff for: tests/dummy/app/JsonApi/V1/Phones/PhoneSchema.php

@@ -0,0 +1,49 @@
+<?php
+/*
+ * Copyright 2022 Cloud Creativity Limited
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+declare(strict_types=1);
+
+namespace App\JsonApi\V1\Phones;
+
+use App\Models\Phone;
+use LaravelJsonApi\Eloquent\Fields\DateTime;
+use LaravelJsonApi\Eloquent\Fields\ID;
+use LaravelJsonApi\Eloquent\Fields\Str;
+use LaravelJsonApi\Eloquent\Schema;
+
+class PhoneSchema extends Schema
+{
+    /**
+     * The model the schema corresponds to.
+     *
+     * @var string
+     */
+    public static string $model = Phone::class;
+
+    /**
+     * @inheritDoc
+     */
+    public function fields(): iterable
+    {
+        return [
+            ID::make(),
+            DateTime::make('createdAt')->readOnly(),
+            Str::make('number'),
+            DateTime::make('updatedAt')->readOnly(),
+        ];
+    }
+}

Diff for: tests/dummy/app/JsonApi/V1/Server.php

@@ -65,6 +65,7 @@ protected function allSchemas(): array
         return [
             Comments\CommentSchema::class,
             Images\ImageSchema::class,
+            Phones\PhoneSchema::class,
             Posts\PostSchema::class,
             Tags\TagSchema::class,
             Users\UserSchema::class,

Diff for: tests/dummy/app/JsonApi/V1/Users/UserRequest.php

@@ -0,0 +1,36 @@
+<?php
+/*
+ * Copyright 2022 Cloud Creativity Limited
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+declare(strict_types=1);
+
+namespace App\JsonApi\V1\Users;
+
+use LaravelJsonApi\Laravel\Http\Requests\ResourceRequest;
+use LaravelJsonApi\Validation\Rule as JsonApiRule;
+
+class UserRequest extends ResourceRequest
+{
+    /**
+     * @return array
+     */
+    public function rules(): array
+    {
+        return [
+            'phone' => JsonApiRule::toOne(),
+        ];
+    }
+}

Diff for: tests/dummy/app/JsonApi/V1/Users/UserSchema.php

@@ -22,6 +22,7 @@
 use App\Models\User;
 use LaravelJsonApi\Eloquent\Fields\DateTime;
 use LaravelJsonApi\Eloquent\Fields\ID;
+use LaravelJsonApi\Eloquent\Fields\Relations\HasOne;
 use LaravelJsonApi\Eloquent\Fields\Str;
 use LaravelJsonApi\Eloquent\Filters\Where;
 use LaravelJsonApi\Eloquent\Filters\WhereIdIn;
@@ -47,6 +48,7 @@ public function fields(): array
             ID::make(),
             DateTime::make('createdAt')->readOnly(),
             Str::make('name'),
+            HasOne::make('phone')->deleteDetachedModel(),
             DateTime::make('updatedAt')->readOnly(),
         ];
     }

Diff for: tests/dummy/app/Models/Phone.php

@@ -0,0 +1,44 @@
+<?php
+/*
+ * Copyright 2022 Cloud Creativity Limited
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+declare(strict_types=1);
+
+namespace App\Models;
+
+use Illuminate\Database\Eloquent\Factories\HasFactory;
+use Illuminate\Database\Eloquent\Model;
+use Illuminate\Database\Eloquent\Relations\BelongsTo;
+
+class Phone extends Model
+{
+    use HasFactory;
+
+    /**
+     * @var string[]
+     */
+    protected $fillable = [
+        'number',
+    ];
+
+    /**
+     * @return BelongsTo
+     */
+    public function user(): BelongsTo
+    {
+        return $this->belongsTo(User::class);
+    }
+}

Diff for: tests/dummy/app/Models/User.php

@@ -18,6 +18,7 @@
 namespace App\Models;
 
 use Illuminate\Database\Eloquent\Factories\HasFactory;
+use Illuminate\Database\Eloquent\Relations\HasOne;
 use Illuminate\Foundation\Auth\User as Authenticatable;
 use Illuminate\Notifications\Notifiable;
 
@@ -60,4 +61,12 @@ public function isAdmin(): bool
     {
         return '[email protected]' === $this->email;
     }
+
+    /**
+     * @return HasOne
+     */
+    public function phone(): HasOne
+    {
+        return $this->hasOne(Phone::class);
+    }
 }

Diff for: tests/dummy/app/Policies/UserPolicy.php

@@ -23,7 +23,6 @@
 
 class UserPolicy
 {
-
     /**
      * Determine if the user can view the other user.
      *
@@ -35,4 +34,28 @@ public function view(User $user, User $other): bool
     {
         return true;
     }
+
+    /**
+     * Determine if the user can view the other user's phone.
+     *
+     * @param User $user
+     * @param User $other
+     * @return bool
+     */
+    public function viewPhone(User $user, User $other): bool
+    {
+        return $user->is($other);
+    }
+
+    /**
+     * Determine if the user can update the other user's phone.
+     *
+     * @param User $user
+     * @param User $other
+     * @return bool
+     */
+    public function updatePhone(User $user, User $other): bool
+    {
+        return $user->is($other);
+    }
 }

Diff for: tests/dummy/database/factories/PhoneFactory.php

@@ -0,0 +1,47 @@
+<?php
+/*
+ * Copyright 2022 Cloud Creativity Limited
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+declare(strict_types=1);
+
+namespace Database\Factories;
+
+use App\Models\Phone;
+use App\Models\User;
+use Illuminate\Database\Eloquent\Factories\Factory;
+
+class PhoneFactory extends Factory
+{
+    /**
+     * The name of the factory's corresponding model.
+     *
+     * @var string
+     */
+    protected $model = Phone::class;
+
+    /**
+     * Define the model's default state.
+     *
+     * @return array
+     */
+    public function definition(): array
+    {
+        return [
+            'number' => $this->faker->phoneNumber(),
+            'user_id' => User::factory(),
+        ];
+    }
+}

Diff for: tests/dummy/database/migrations/2020_06_13_143800_create_post_and_video_tables.php

@@ -15,11 +15,13 @@
  * limitations under the License.
  */
 
+declare(strict_types=1);
+
 use Illuminate\Database\Migrations\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-class CreatePostAndVideoTables extends Migration
+return new class extends Migration
 {
 
     /**
@@ -29,7 +31,7 @@ class CreatePostAndVideoTables extends Migration
      */
     public function up(): void
     {
-        Schema::create('posts', function (Blueprint $table) {
+        Schema::create('posts', function (Blueprint $table): void {
             $table->id();
             $table->timestamps();
             $table->softDeletes();
@@ -47,13 +49,13 @@ public function up(): void
                 ->onUpdate('cascade');
         });
 
-        Schema::create('images', function (Blueprint $table) {
+        Schema::create('images', function (Blueprint $table): void {
             $table->uuid('uuid')->primary();
             $table->timestamps();
             $table->string('url');
         });
 
-        Schema::create('videos', function (Blueprint $table) {
+        Schema::create('videos', function (Blueprint $table): void {
             $table->uuid('uuid')->primary();
             $table->timestamps();
             $table->unsignedBigInteger('owner_id');
@@ -67,7 +69,7 @@ public function up(): void
                 ->onUpdate('cascade');
         });
 
-        Schema::create('comments', function (Blueprint $table) {
+        Schema::create('comments', function (Blueprint $table): void {
             $table->id();
             $table->timestamps();
             $table->unsignedBigInteger('post_id');
@@ -87,13 +89,13 @@ public function up(): void
                 ->cascadeOnUpdate();
         });
 
-        Schema::create('tags', function (Blueprint $table) {
+        Schema::create('tags', function (Blueprint $table): void {
             $table->id();
             $table->timestamps();
             $table->string('name');
         });
 
-        Schema::create('taggables', function (Blueprint $table) {
+        Schema::create('taggables', function (Blueprint $table): void {
             $table->unsignedBigInteger('tag_id');
             $table->morphs('taggable');
             $table->primary([
@@ -109,7 +111,7 @@ public function up(): void
                 ->cascadeOnDelete();
         });
 
-        Schema::create('image_post', function (Blueprint $table) {
+        Schema::create('image_post', function (Blueprint $table): void {
             $table->uuid('image_uuid');
             $table->unsignedBigInteger('post_id');
             $table->primary(['image_uuid', 'post_id']);
@@ -127,7 +129,7 @@ public function up(): void
                 ->cascadeOnUpdate();
         });
 
-        Schema::create('post_video', function (Blueprint $table) {
+        Schema::create('post_video', function (Blueprint $table): void {
             $table->unsignedBigInteger('post_id');
             $table->uuid('video_uuid');
             $table->primary(['post_id', 'video_uuid']);
@@ -159,4 +161,4 @@ public function down(): void
         Schema::dropIfExists('videos');
         Schema::dropIfExists('posts');
     }
-}
+};