Add ability to update billable rate of existing time entries

tests/Unit/Endpoint/Api/V1/ImportEndpointTest.php

@@ -78,6 +78,26 @@ class ImportEndpointTest extends ApiEndpointTestAbstract
         $response->assertForbidden();
     }
 
+    public function test_import_fails_if_data_can_not_be_base64_decoded(): void
+    {
+        $user = $this->createUserWithPermission([
+            'import',
+        ]);
+        Passport::actingAs($user->user);
+
+        // Act
+        $response = $this->postJson(route('api.v1.import.import', ['organization' => $user->organization->getKey()]), [
+            'type' => 'toggl_time_entries',
+            'data' => 'some invalid data ...',
+        ]);
+
+        // Assert
+        $response->assertStatus(400);
+        $response->assertExactJson([
+            'message' => 'Invalid base64 encoded data',
+        ]);
+    }
+
     public function test_import_return_error_message_if_import_fails(): void
     {
         $user = $this->createUserWithPermission([

tests/Unit/Endpoint/Api/V1/MemberEndpointTest.php

@@ -12,7 +12,9 @@ use App\Models\Project;
 use App\Models\ProjectMember;
 use App\Models\TimeEntry;
 use App\Models\User;
+use App\Service\BillableRateService;
 use Laravel\Passport\Passport;
+use Mockery\MockInterface;
 use PHPUnit\Framework\Attributes\UsesClass;
 
 #[UsesClass(MemberController::class)]
@@ -91,6 +93,7 @@ class MemberEndpointTest extends ApiEndpointTestAbstract
         $data = $this->createUserWithPermission([
             'members:update',
         ]);
+        $this->assertBillableRateServiceIsUnused();
         Passport::actingAs($data->user);
 
         // Act
@@ -107,6 +110,32 @@ class MemberEndpointTest extends ApiEndpointTestAbstract
         $this->assertSame(Role::Employee->value, $member->role);
     }
 
+    public function test_update_member_can_update_billable_rate_of_member_and_update_time_entries(): void
+    {
+        // Arrange
+        $data = $this->createUserWithPermission([
+            'members:update',
+        ]);
+        $this->mock(BillableRateService::class, function (MockInterface $mock) use ($data) {
+            $mock->shouldReceive('updateTimeEntriesBillableRateForMember')
+                ->once()
+                ->withArgs(fn (Member $memberArg) => $memberArg->is($data->member) && $memberArg->billable_rate === 10001);
+        });
+        Passport::actingAs($data->user);
+
+        // Act
+        $response = $this->putJson(route('api.v1.members.update', [$data->organization->getKey(), $data->member]), [
+            'billable_rate' => 10001,
+            'billable_rate_update_time_entries' => 'true',
+        ]);
+
+        // Assert
+        $response->assertStatus(200);
+        $member = $data->member;
+        $member->refresh();
+        $this->assertSame(10001, $member->billable_rate);
+    }
+
     public function test_invite_placeholder_succeeds_if_data_is_valid(): void
     {
         $data = $this->createUserWithPermission([

tests/Unit/Endpoint/Api/V1/OrganizationEndpointTest.php

@@ -6,7 +6,9 @@ namespace Tests\Unit\Endpoint\Api\V1;
 
 use App\Http\Controllers\Api\V1\OrganizationController;
 use App\Models\Organization;
+use App\Service\BillableRateService;
 use Laravel\Passport\Passport;
+use Mockery\MockInterface;
 use PHPUnit\Framework\Attributes\UsesClass;
 
 #[UsesClass(OrganizationController::class)]
@@ -45,8 +47,8 @@ class OrganizationEndpointTest extends ApiEndpointTestAbstract
     public function test_update_endpoint_fails_if_user_has_no_permission_to_update_organizations(): void
     {
         // Arrange
-        $data = $this->createUserWithPermission([
-        ]);
+        $data = $this->createUserWithPermission();
+        $this->assertBillableRateServiceIsUnused();
         $organizationFake = Organization::factory()->make();
         Passport::actingAs($data->user);
 
@@ -65,6 +67,7 @@ class OrganizationEndpointTest extends ApiEndpointTestAbstract
         $data = $this->createUserWithPermission([
             'organizations:update',
         ]);
+        $this->assertBillableRateServiceIsUnused();
         $organizationFake = Organization::factory()->make();
         Passport::actingAs($data->user);
 
@@ -88,6 +91,7 @@ class OrganizationEndpointTest extends ApiEndpointTestAbstract
         $data = $this->createUserWithPermission([
             'organizations:update',
         ]);
+        $this->assertBillableRateServiceIsUnused();
         $organizationFake = Organization::factory()->make();
         Passport::actingAs($data->user);
 
@@ -104,4 +108,34 @@ class OrganizationEndpointTest extends ApiEndpointTestAbstract
             'billable_rate' => $organizationFake->billable_rate,
         ]);
     }
+
+    public function test_update_endpoint_can_update_billable_rate_of_organization_and_update_time_entries(): void
+    {
+        // Arrange
+        $data = $this->createUserWithPermission([
+            'organizations:update',
+        ]);
+        $billableRate = 111;
+        $organizationFake = Organization::factory()->billableRate($billableRate)->make();
+        $this->mock(BillableRateService::class, function (MockInterface $mock) use ($data, $billableRate) {
+            $mock->shouldReceive('updateTimeEntriesBillableRateForOrganization')
+                ->once()
+                ->withArgs(fn (Organization $organization) => $organization->is($data->organization) && $organization->billable_rate === $billableRate);
+        });
+        Passport::actingAs($data->user);
+
+        // Act
+        $response = $this->withoutExceptionHandling()->putJson(route('api.v1.organizations.update', [$data->organization->getKey()]), [
+            'name' => $organizationFake->name,
+            'billable_rate' => $organizationFake->billable_rate,
+            'billable_rate_update_time_entries' => 'true',
+        ]);
+
+        // Assert
+        $response->assertStatus(200);
+        $this->assertDatabaseHas(Organization::class, [
+            'name' => $organizationFake->name,
+            'billable_rate' => $organizationFake->billable_rate,
+        ]);
+    }
 }

tests/Unit/Endpoint/Api/V1/ProjectEndpointTest.php

@@ -11,7 +11,9 @@ use App\Models\Project;
 use App\Models\ProjectMember;
 use App\Models\Task;
 use App\Models\TimeEntry;
+use App\Service\BillableRateService;
 use Laravel\Passport\Passport;
+use Mockery\MockInterface;
 use PHPUnit\Framework\Attributes\UsesClass;
 
 #[UsesClass(ProjectController::class)]
@@ -200,6 +202,7 @@ class ProjectEndpointTest extends ApiEndpointTestAbstract
             'projects:create',
         ]);
         $projectFake = Project::factory()->forOrganization($data->organization)->make();
+        $this->assertBillableRateServiceIsUnused();
         Passport::actingAs($data->user);
 
         // Act
@@ -230,6 +233,7 @@ class ProjectEndpointTest extends ApiEndpointTestAbstract
         $otherOrganization = Organization::factory()->create();
         $project = Project::factory()->forOrganization($otherOrganization)->create();
         $projectFake = Project::factory()->make();
+        $this->assertBillableRateServiceIsUnused();
         Passport::actingAs($data->user);
 
         // Act
@@ -246,10 +250,10 @@ class ProjectEndpointTest extends ApiEndpointTestAbstract
     public function test_update_endpoint_fails_if_user_has_no_permission_to_update_projects(): void
     {
         // Arrange
-        $data = $this->createUserWithPermission([
-        ]);
+        $data = $this->createUserWithPermission();
         $project = Project::factory()->forOrganization($data->organization)->create();
         $projectFake = Project::factory()->make();
+        $this->assertBillableRateServiceIsUnused();
         Passport::actingAs($data->user);
 
         // Act
@@ -272,6 +276,7 @@ class ProjectEndpointTest extends ApiEndpointTestAbstract
         $project = Project::factory()->forOrganization($data->organization)->create();
         $projectFake = Project::factory()->make();
         $client = Client::factory()->forOrganization($data->organization)->create();
+        $this->assertBillableRateServiceIsUnused();
         Passport::actingAs($data->user);
 
         // Act
@@ -299,6 +304,7 @@ class ProjectEndpointTest extends ApiEndpointTestAbstract
         ]);
         $project = Project::factory()->forOrganization($data->organization)->create();
         $projectFake = Project::factory()->make();
+        $this->assertBillableRateServiceIsUnused();
         Passport::actingAs($data->user);
 
         // Act
@@ -318,6 +324,39 @@ class ProjectEndpointTest extends ApiEndpointTestAbstract
         ]);
     }
 
+    public function test_update_endpoint_can_update_projects_billable_rate_and_update_time_entries(): void
+    {
+        // Arrange
+        $data = $this->createUserWithPermission([
+            'projects:update',
+        ]);
+        $project = Project::factory()->forOrganization($data->organization)->create();
+        $projectFake = Project::factory()->make();
+        $this->mock(BillableRateService::class, function (MockInterface $mock) use ($project): void {
+            $mock->shouldReceive('updateTimeEntriesBillableRateForProject')
+                ->once()
+                ->withArgs(fn (Project $projectArg) => $projectArg->is($project) && $projectArg->billable_rate === 10003);
+        });
+        Passport::actingAs($data->user);
+
+        // Act
+        $response = $this->putJson(route('api.v1.projects.update', [$data->organization->getKey(), $project->getKey()]), [
+            'name' => $projectFake->name,
+            'color' => $projectFake->color,
+            'is_billable' => $projectFake->is_billable,
+            'billable_rate' => 10003,
+            'billable_rate_update_time_entries' => 'true',
+        ]);
+
+        // Assert
+        $response->assertStatus(200);
+        $this->assertDatabaseHas(Project::class, [
+            'name' => $projectFake->name,
+            'color' => $projectFake->color,
+            'billable_rate' => 10003,
+        ]);
+    }
+
     public function test_destroy_endpoint_fails_if_user_is_not_part_of_project_organization(): void
     {
         // Arrange

tests/Unit/Endpoint/Api/V1/ProjectMemberEndpointTest.php

@@ -9,7 +9,9 @@ use App\Models\Member;
 use App\Models\Project;
 use App\Models\ProjectMember;
 use App\Models\User;
+use App\Service\BillableRateService;
 use Laravel\Passport\Passport;
+use Mockery\MockInterface;
 use PHPUnit\Framework\Attributes\UsesClass;
 
 #[UsesClass(ProjectMemberController::class)]
@@ -289,20 +291,52 @@ class ProjectMemberEndpointTest extends ApiEndpointTestAbstract
             'project-members:update',
         ]);
         $project = Project::factory()->forOrganization($data->organization)->create();
+        $billableRate = 1001;
         $projectMember = ProjectMember::factory()->forProject($project)->create();
-        $projectMemberFake = ProjectMember::factory()->make();
+        $this->assertBillableRateServiceIsUnused();
         Passport::actingAs($data->user);
 
         // Act
         $response = $this->putJson(route('api.v1.project-members.update', [$data->organization->getKey(), $projectMember->getKey()]), [
-            'billable_rate' => $projectMemberFake->billable_rate,
+            'billable_rate' => $billableRate,
         ]);
 
         // Assert
         $response->assertStatus(200);
         $this->assertDatabaseHas(ProjectMember::class, [
             'id' => $projectMember->getKey(),
-            'billable_rate' => $projectMemberFake->billable_rate,
+            'billable_rate' => $billableRate,
+            'member_id' => $projectMember->member_id,
+        ]);
+    }
+
+    public function test_update_endpoints_can_update_billable_rate_and_update_time_entries(): void
+    {
+        // Arrange
+        $data = $this->createUserWithPermission([
+            'project-members:update',
+        ]);
+        $project = Project::factory()->forOrganization($data->organization)->create();
+        $billableRate = 1001;
+        $projectMember = ProjectMember::factory()->forProject($project)->create();
+        $this->mock(BillableRateService::class, function (MockInterface $mock) use ($projectMember, $billableRate): void {
+            $mock->shouldReceive('updateTimeEntriesBillableRateForProjectMember')
+                ->once()
+                ->withArgs(fn (ProjectMember $projectMemberArg) => $projectMemberArg->is($projectMember) && $projectMemberArg->billable_rate === $billableRate);
+        });
+        Passport::actingAs($data->user);
+
+        // Act
+        $response = $this->putJson(route('api.v1.project-members.update', [$data->organization->getKey(), $projectMember->getKey()]), [
+            'billable_rate' => $billableRate,
+            'billable_rate_update_time_entries' => 'true',
+        ]);
+
+        // Assert
+        $response->assertStatus(200);
+        $this->assertDatabaseHas(ProjectMember::class, [
+            'id' => $projectMember->getKey(),
+            'billable_rate' => $billableRate,
             'member_id' => $projectMember->member_id,
         ]);
     }