Add updater for bank account

database/factories/Member/MemberFactory.php

@@ -93,6 +93,11 @@ class MemberFactory extends Factory
         return $this->state(['nami_id' => $namiId]);
     }
 
+    public function notInNami(): self
+    {
+        return $this->state(['nami_id' => null]);
+    }
+
     public function withBankAccount(BankAccountFactory $factory): self
     {
         return $this->afterCreating(function ($member) use ($factory) {

tests/Feature/Member/UpdateTest.php

@@ -6,269 +6,239 @@ use App\Actions\PullMemberAction;
 use App\Actions\PullMembershipsAction;
 use App\Activity;
 use App\Confession;
-use App\Country;
-use App\Fee;
 use App\Group;
 use App\Member\Actions\NamiPutMemberAction;
 use App\Member\Member;
-use App\Nationality;
-use App\Payment\Subscription;
 use App\Subactivity;
 use Illuminate\Foundation\Testing\DatabaseTransactions;
 use Illuminate\Support\Facades\Http;
-use Tests\TestCase;
+use Tests\RequestFactories\MemberUpdateRequestFactory;
 use Zoomyboy\LaravelNami\Fakes\MemberFake;
 
-class UpdateTest extends TestCase
+uses(DatabaseTransactions::class);
+
+function singleMemberUrl(int $gruppierungId, int $memberId): string
 {
-    use DatabaseTransactions;
+    return "https://nami.dpsg.de/ica/rest/nami/mitglied/filtered-for-navigation/gruppierung/gruppierung/{$gruppierungId}/{$memberId}";
+}
 
-    public function testItRedirectsToMemberOverview(): void
+beforeEach(function () {
-    {
+    Confession::factory()->create(['is_null' => true]);
-        $this->withoutExceptionHandling()->login()->loginNami();
+});
-        $member = $this->member();
-        $this->fakeRequest();
-        NamiPutMemberAction::allowToRun();
 
-        $response = $this
+function fakeRequest(): void
-            ->from("/member/{$member->id}")
+{
-            ->patch("/member/{$member->id}", array_merge($member->getAttributes(), ['has_nami' => true, 'bank_account' => []]));
+    Http::fake(function ($request) {
+        if ($request->url() === singleMemberUrl(10, 135) && 'GET' === $request->method()) {
+            return Http::response('{ "success": true, "data": {"missingkey": "missingvalue", "kontoverbindung": {"a": "b"} } }', 200);
+        }
 
-        $response->assertRedirect('/member');
+        if ($request->url() === singleMemberUrl(10, 135) && 'PUT' === $request->method() && 43 === $request['version']) {
-        NamiPutMemberAction::spy()->shouldHaveReceived('handle')->withArgs(
+            return Http::response('{ "success": false, "message": "Update nicht möglich. Der Datensatz wurde zwischenzeitlich verändert." }', 200);
-            fn (Member $memberParam, ?Activity $activityParam, ?Subactivity $subactivityParam) => $memberParam->is($member)
+        }
-                && null === $activityParam
-                && null === $subactivityParam
-        )->once();
-    }
 
-    public function testItChecksVersion(): void
+        if ($request->url() === singleMemberUrl(10, 135) && 'PUT' === $request->method()) {
-    {
+            return Http::response('{ "success": true, "data": { "version": 44 } }', 200);
-        $this->login()->loginNami()->withoutExceptionHandling();
+        }
-        $member = $this->member();
+    });
-        $member->update(['version' => 43]);
+}
-        $this->fakeRequest();
 
-        $response = $this
+function factory()
-            ->from("/member/{$member->id}")
+{
-            ->patch("/member/{$member->id}", array_merge($member->getAttributes(), ['has_nami' => true, 'firstname' => '::firstname::', 'bank_account' => []]));
+    return Member::factory()
+        ->defaults()
+        ->for(Group::factory()->state(['nami_id' => 10]))
+        ->state(['nami_id' => 135]);
+}
 
-        $response->assertRedirect("/member/{$member->id}/edit?conflict=1");
+it('calls put action', function () {
-    }
+    $this->withoutExceptionHandling()->login()->loginNami();
+    $member = factory()->create();
+    fakeRequest();
+    NamiPutMemberAction::allowToRun();
 
-    public function testItUpdatesPhoneNumber(): void
+    $this->patch("/member/{$member->id}", MemberUpdateRequestFactory::new()->create());
-    {
-        $this->withoutExceptionHandling()->login()->loginNami();
-        $member = $this->member();
-        $this->fakeRequest();
-        NamiPutMemberAction::allowToRun();
 
-        $this->patch("/member/{$member->id}", array_merge($member->getAttributes(), [
+    NamiPutMemberAction::spy()->shouldHaveReceived('handle')->withArgs(
-            'main_phone' => '02103 4455129',
+        fn (Member $memberParam, ?Activity $activityParam, ?Subactivity $subactivityParam) => $memberParam->is($member)
-            'fax' => '02103 4455130',
+            && null === $activityParam
-            'children_phone' => '02103 4455130',
+            && null === $subactivityParam
-            'has_nami' => true,
+    )->once();
-            'bank_account' => []
+});
-        ]));
 
-        $this->assertDatabaseHas('members', [
+it('redirects to member overview', function () {
-            'main_phone' => '+49 2103 4455129',
+    $this->withoutExceptionHandling()->login()->loginNami();
-            'fax' => '+49 2103 4455130',
+    $member = factory()->create();
-            'children_phone' => '+49 2103 4455130',
+    fakeRequest();
-        ]);
+    NamiPutMemberAction::allowToRun();
-    }
 
-    public function testItUpdatesBankAccount(): void
+    $this->patch("/member/{$member->id}", MemberUpdateRequestFactory::new()->create())
-    {
+        ->assertRedirect('/member');
-        $this->withoutExceptionHandling()->login()->loginNami();
+});
-        $member = $this->member();
-        $this->fakeRequest();
-        NamiPutMemberAction::allowToRun();
 
-        $this->patch("/member/{$member->id}", array_merge($member->getAttributes(), [
+it('testItChecksVersion', function () {
-            'has_nami' => true,
+    $this->login()->loginNami()->withoutExceptionHandling();
-            'bank_account' => [
+    $member = tap(factory()->create(), fn ($member) => $member->update(['version' => 43]));
-                'iban' => 'DE1122',
+    fakeRequest();
-                'bic' => 'SOLSDE',
-                'person' => 'Max'
-            ]
-        ]));
 
-        $this->assertDatabaseHas('bank_accounts', [
+    $this->patch("/member/{$member->id}", array_merge($member->getAttributes(), MemberUpdateRequestFactory::new()->create()))
+        ->assertRedirect("/member/{$member->id}/edit?conflict=1");
+});
+
+it('testItUpdatesPhoneNumber', function () {
+    $this->withoutExceptionHandling()->login()->loginNami();
+    $member = factory()->create();
+    fakeRequest();
+    NamiPutMemberAction::allowToRun();
+
+    $this->patch("/member/{$member->id}", MemberUpdateRequestFactory::new()->create([
+        'main_phone' => '02103 4455129',
+        'fax' => '02103 4455130',
+        'children_phone' => '02103 4455130',
+    ]));
+
+    $this->assertDatabaseHas('members', [
+        'main_phone' => '+49 2103 4455129',
+        'fax' => '+49 2103 4455130',
+        'children_phone' => '+49 2103 4455130',
+    ]);
+});
+
+it('testItUpdatesBankAccount', function () {
+    $this->withoutExceptionHandling()->login()->loginNami();
+    $member = factory()->create();
+    fakeRequest();
+    NamiPutMemberAction::allowToRun();
+
+    $this->patch("/member/{$member->id}", MemberUpdateRequestFactory::new()->create([
+        'bank_account' => [
             'iban' => 'DE1122',
             'bic' => 'SOLSDE',
-            'person' => 'Max',
+            'person' => 'Max'
-            'member_id' => $member->id,
+        ]
-        ]);
+    ]));
-    }
 
-    public function testItUpdatesWiederverwendenFlag(): void
+    $this->assertDatabaseHas('bank_accounts', [
-    {
+        'iban' => 'DE1122',
-        $this->withoutExceptionHandling()->login()->loginNami();
+        'bic' => 'SOLSDE',
-        $member = $this->member();
+        'person' => 'Max',
-        $this->fakeRequest();
+        'member_id' => $member->id,
-        NamiPutMemberAction::allowToRun();
+    ]);
+});
 
-        $this->patch("/member/{$member->id}", array_merge($member->getAttributes(), [
+it('testItUpdatesWiederverwendenFlag', function () {
-            'keepdata' => true,
+    $this->withoutExceptionHandling()->login()->loginNami();
-            'has_nami' => true,
+    $member = factory()->create();
+    fakeRequest();
+    NamiPutMemberAction::allowToRun();
+
+    $this->patch("/member/{$member->id}", MemberUpdateRequestFactory::new()->create([
+        'keepdata' => true,
+    ]));
+
+    $this->assertDatabaseHas('members', [
+        'keepdata' => true,
+    ]);
+});
+
+it('testItSetsLocationToNull', function () {
+    $this->withoutExceptionHandling()->login()->loginNami();
+    $member = factory()->notInNami()->create(['location' => 'Hilden']);
+    fakeRequest();
+    NamiPutMemberAction::allowToRun();
+
+    $this->patch("/member/{$member->id}", MemberUpdateRequestFactory::new()->noNami()->create([
+        'location' => null,
+        'bank_account' => []
+    ]));
+
+    $this->assertDatabaseHas('members', [
+        'location' => null,
+    ]);
+});
+
+it('testItUpdatesContact', function () {
+    $this->withoutExceptionHandling()->login()->loginNami();
+    $member = factory()->notInNami()->create();
+    fakeRequest();
+
+    $this->patch("/member/{$member->id}", MemberUpdateRequestFactory::new()->noNami()->create([
+        'other_country' => 'englisch',
+        'bank_account' => []
+    ]));
+
+    $this->assertEquals('englisch', $member->fresh()->other_country);
+});
+
+it('testItCreatesMemberWithFirstActivityId', function () {
+    $this->login()->loginNami()->withoutExceptionHandling();
+    $member = factory()->new()->defaults()->create();
+    app(MemberFake::class)->stores($member->group->nami_id, 103);
+    $activity = Activity::factory()->inNami(89)->create();
+    $subactivity = Subactivity::factory()->inNami(90)->create();
+    Confession::factory()->create(['is_null' => true]);
+    PullMemberAction::shouldRun();
+    PullMembershipsAction::shouldRun();
+
+    $this->patch("/member/{$member->id}", MemberUpdateRequestFactory::new()->create([
+        'first_activity_id' => $activity->id,
+        'first_subactivity_id' => $subactivity->id,
+    ]))->assertSessionHasNoErrors();
+
+    app(MemberFake::class)->assertStored($member->group->nami_id, [
+        'ersteTaetigkeitId' => 89,
+        'ersteUntergliederungId' => 90,
+    ]);
+});
+
+it('testItRequiresFirstActivityId', function () {
+    $this->login()->loginNami();
+    $member = factory()->new()->defaults()->create();
+    app(MemberFake::class)->stores($member->group->nami_id, 103);
+    Confession::factory()->create(['is_null' => true]);
+    PullMemberAction::shouldRun();
+    PullMembershipsAction::shouldRun();
+
+    $this->patch("/member/{$member->id}", MemberUpdateRequestFactory::new()->create([
+        'first_activity_id' => null,
+        'first_subactivity_id' => null,
+    ]))->assertSessionHasErrors([
+        'first_activity_id' => 'Erste Tätigkeit ist erforderlich.',
+        'first_subactivity_id' => 'Erste Untergliederung ist erforderlich.',
+    ]);
+});
+
+it('testItUpdatesCriminalRecord', function () {
+    $this->withoutExceptionHandling()->login()->loginNami();
+    $member = factory()->notInNami()->create();
+    fakeRequest();
+
+    $this
+        ->patch("/member/{$member->id}", MemberUpdateRequestFactory::new()->noNami()->create([
+            'ps_at' => '2021-02-01',
+            'more_ps_at' => '2021-02-02',
+            'has_svk' => true,
+            'has_vk' => true,
+            'efz' => '2021-02-03',
+            'without_education_at' => '2021-02-04',
+            'without_efz_at' => '2021-02-05',
+            'recertified_at' => '2021-02-08',
+            'has_nami' => false,
+            'multiply_pv' => true,
+            'multiply_more_pv' => true,
+            'salutation' => 'Doktor',
             'bank_account' => []
         ]));
 
-        $this->assertDatabaseHas('members', [
+    $this->assertEquals('2021-02-01', $member->fresh()->ps_at->format('Y-m-d'));
-            'keepdata' => true,
+    $this->assertEquals('2021-02-02', $member->fresh()->more_ps_at->format('Y-m-d'));
-        ]);
+    $this->assertTrue($member->fresh()->has_svk);
-    }
+    $this->assertTrue($member->fresh()->has_vk);
-
+    $this->assertTrue($member->fresh()->multiply_pv);
-    public function testItSetsLocationToNull(): void
+    $this->assertTrue($member->fresh()->multiply_more_pv);
-    {
+    $this->assertEquals('2021-02-03', $member->fresh()->efz->format('Y-m-d'));
-        $this->withoutExceptionHandling()->login()->loginNami();
+    $this->assertEquals('2021-02-04', $member->fresh()->without_education_at->format('Y-m-d'));
-        $member = $this->member(['location' => 'Hilden', 'nami_id' => null]);
+    $this->assertEquals('2021-02-05', $member->fresh()->without_efz_at->format('Y-m-d'));
-        $this->fakeRequest();
+    $this->assertEquals('2021-02-08', $member->fresh()->recertified_at->format('Y-m-d'));
-        NamiPutMemberAction::allowToRun();
+    $this->assertEquals('Doktor', $member->fresh()->salutation);
-
+});
-        $this->patch("/member/{$member->id}", array_merge($member->getAttributes(), [
-            'location' => null,
-            'bank_account' => []
-        ]));
-
-        $this->assertDatabaseHas('members', [
-            'location' => null,
-        ]);
-    }
-
-    public function testItUpdatesContact(): void
-    {
-        $this->withoutExceptionHandling()->login()->loginNami();
-        $member = $this->member(['nami_id' => null]);
-        $this->fakeRequest();
-
-        $response = $this
-            ->from("/member/{$member->id}")
-            ->patch("/member/{$member->id}", array_merge($member->getAttributes(), [
-                'other_country' => 'englisch',
-                'bank_account' => []
-            ]));
-
-        $this->assertEquals('englisch', $member->fresh()->other_country);
-    }
-
-    public function testItCreatesMemberWithFirstActivityId(): void
-    {
-        $this->login()->loginNami()->withoutExceptionHandling();
-        $member = Member::factory()->defaults()->create();
-        app(MemberFake::class)->stores($member->group->nami_id, 103);
-        $activity = Activity::factory()->inNami(89)->create();
-        $subactivity = Subactivity::factory()->inNami(90)->create();
-        Confession::factory()->create(['is_null' => true]);
-        PullMemberAction::shouldRun();
-        PullMembershipsAction::shouldRun();
-
-        $this->patch("/member/{$member->id}", [
-            ...$member->getAttributes(),
-            'has_nami' => true,
-            'first_activity_id' => $activity->id,
-            'first_subactivity_id' => $subactivity->id,
-            'bank_account' => []
-        ])->assertSessionHasNoErrors();
-
-        app(MemberFake::class)->assertStored($member->group->nami_id, [
-            'ersteTaetigkeitId' => 89,
-            'ersteUntergliederungId' => 90,
-        ]);
-    }
-
-    public function testItRequiresFirstActivityId(): void
-    {
-        $this->login()->loginNami();
-        $member = Member::factory()->defaults()->create();
-        app(MemberFake::class)->stores($member->group->nami_id, 103);
-        Confession::factory()->create(['is_null' => true]);
-        PullMemberAction::shouldRun();
-        PullMembershipsAction::shouldRun();
-
-        $this->patch("/member/{$member->id}", [
-            ...$member->getAttributes(),
-            'has_nami' => true,
-            'first_activity_id' => null,
-            'first_subactivity_id' => null,
-            'bank_account' => []
-        ])->assertSessionHasErrors([
-            'first_activity_id' => 'Erste Tätigkeit ist erforderlich.',
-            'first_subactivity_id' => 'Erste Untergliederung ist erforderlich.',
-        ]);
-    }
-
-    public function testItUpdatesCriminalRecord(): void
-    {
-        $this->withoutExceptionHandling()->login()->loginNami();
-        $member = $this->member(['nami_id' => null]);
-        $this->fakeRequest();
-
-        $response = $this
-            ->from("/member/{$member->id}")
-            ->patch("/member/{$member->id}", array_merge($member->getAttributes(), [
-                'ps_at' => '2021-02-01',
-                'more_ps_at' => '2021-02-02',
-                'has_svk' => true,
-                'has_vk' => true,
-                'efz' => '2021-02-03',
-                'without_education_at' => '2021-02-04',
-                'without_efz_at' => '2021-02-05',
-                'recertified_at' => '2021-02-08',
-                'has_nami' => false,
-                'multiply_pv' => true,
-                'multiply_more_pv' => true,
-                'salutation' => 'Doktor',
-                'bank_account' => []
-            ]));
-
-        $this->assertEquals('2021-02-01', $member->fresh()->ps_at->format('Y-m-d'));
-        $this->assertEquals('2021-02-02', $member->fresh()->more_ps_at->format('Y-m-d'));
-        $this->assertTrue($member->fresh()->has_svk);
-        $this->assertTrue($member->fresh()->has_vk);
-        $this->assertTrue($member->fresh()->multiply_pv);
-        $this->assertTrue($member->fresh()->multiply_more_pv);
-        $this->assertEquals('2021-02-03', $member->fresh()->efz->format('Y-m-d'));
-        $this->assertEquals('2021-02-04', $member->fresh()->without_education_at->format('Y-m-d'));
-        $this->assertEquals('2021-02-05', $member->fresh()->without_efz_at->format('Y-m-d'));
-        $this->assertEquals('2021-02-08', $member->fresh()->recertified_at->format('Y-m-d'));
-        $this->assertEquals('Doktor', $member->fresh()->salutation);
-    }
-
-    /**
-     * @param array<string, string|Activity|null> $overwrites
-     */
-    private function member(array $overwrites = []): Member
-    {
-        return Member::factory()
-            ->for(Group::factory()->state(['nami_id' => 10]))
-            ->for(Confession::factory())
-            ->for(Nationality::factory())
-            ->for(Subscription::factory()->forFee())
-            ->for(Country::factory())
-            ->create(['nami_id' => 135, ...$overwrites]);
-    }
-
-    private function fakeRequest(): void
-    {
-        Http::fake(function ($request) {
-            if ($request->url() === $this->singleMemberUrl(10, 135) && 'GET' === $request->method()) {
-                return Http::response('{ "success": true, "data": {"missingkey": "missingvalue", "kontoverbindung": {"a": "b"} } }', 200);
-            }
-
-            if ($request->url() === $this->singleMemberUrl(10, 135) && 'PUT' === $request->method() && 43 === $request['version']) {
-                return Http::response('{ "success": false, "message": "Update nicht möglich. Der Datensatz wurde zwischenzeitlich verändert." }', 200);
-            }
-
-            if ($request->url() === $this->singleMemberUrl(10, 135) && 'PUT' === $request->method()) {
-                return Http::response('{ "success": true, "data": { "version": 44 } }', 200);
-            }
-        });
-    }
-
-    private function singleMemberUrl(int $gruppierungId, int $memberId): string
-    {
-        return "https://nami.dpsg.de/ica/rest/nami/mitglied/filtered-for-navigation/gruppierung/gruppierung/{$gruppierungId}/{$memberId}";
-    }
-}

tests/RequestFactories/MemberRequestFactory.php

@@ -2,6 +2,9 @@
 
 namespace Tests\RequestFactories;
 
+use App\Country;
+use App\Nationality;
+use App\Payment\Subscription;
 use Worksome\RequestFactories\RequestFactory;
 
 class MemberRequestFactory extends RequestFactory
@@ -11,14 +14,18 @@ class MemberRequestFactory extends RequestFactory
      */
     public function definition(): array
     {
+        $country = Country::factory()->create();
+        $nationality = Nationality::factory()->create();
+        $subscription = Subscription::factory()->forFee()->create();
+
         return [
             'address' => 'Bavert 50',
             'birthday' => '2013-02-19',
-            'children_phone' => '+49 123 44444',
+            'children_phone' => '+49 176 70512778',
             'efz' => '',
             'email' => '',
             'email_parents' => 'osloot@aol.com',
-            'fax' => '+49 666',
+            'fax' => '+49 212 4732223',
             'firstname' => 'Joe',
             'further_address' => '',
             'has_nami' => true,
@@ -28,8 +35,8 @@ class MemberRequestFactory extends RequestFactory
             'lastname' => 'Muster',
             'letter_address' => '',
             'location' => 'Solingen',
-            'main_phone' => '+49 212 2334322',
+            'main_phone' => '+49 212 337056',
-            'mobile_phone' => '+49 157 53180451',
+            'mobile_phone' => '+49 176 70512774',
             'more_ps_at' => '',
             'multiply_more_pv' => false,
             'multiply_pv' => false,
@@ -40,6 +47,13 @@ class MemberRequestFactory extends RequestFactory
             'without_efz_at' => '',
             'work_phone' => '',
             'zip' => '42719',
+            'bank_account' => [
+                'iban' => '',
+                'bic' => '',
+            ],
+            'country_id' => $country->id,
+            'nationality_id' => $nationality->id,
+            'subscription_id' => $subscription->id,
         ];
     }
 }

tests/RequestFactories/MemberStoreRequestFactory.php

@@ -3,58 +3,17 @@
 namespace Tests\RequestFactories;
 
 use App\Activity;
-use App\Country;
-use App\Nationality;
-use App\Payment\Subscription;
 use App\Subactivity;
-use Worksome\RequestFactories\RequestFactory;
 
-class MemberStoreRequestFactory extends RequestFactory
+class MemberStoreRequestFactory extends MemberRequestFactory
 {
     public function definition(): array
     {
-        $country = Country::factory()->create();
-        $nationality = Nationality::factory()->create();
-        $subscription = Subscription::factory()->forFee()->create();
         $activity = Activity::factory()->inNami(89)->create();
         $subactivity = Subactivity::factory()->inNami(90)->create();
 
         return [
-            'bank_account' => [
+            ...parent::definition(),
-                'iban' => '',
-                'bic' => '',
-            ],
-            'address' => 'Bavert 50',
-            'birthday' => '2013-02-19',
-            'children_phone' => '+49 176 70512778',
-            'efz' => '',
-            'email' => '',
-            'email_parents' => 'osloot@aol.com',
-            'fax' => '+49 212 4732223',
-            'firstname' => 'Joe',
-            'further_address' => '',
-            'has_nami' => true,
-            'has_svk' => false,
-            'has_vk' => false,
-            'joined_at' => '2022-08-12',
-            'lastname' => 'Muster',
-            'letter_address' => '',
-            'location' => 'Solingen',
-            'main_phone' => '+49 212 337056',
-            'mobile_phone' => '+49 176 70512774',
-            'more_ps_at' => '',
-            'multiply_more_pv' => false,
-            'multiply_pv' => false,
-            'other_country' => '',
-            'ps_at' => '',
-            'send_newspaper' => true,
-            'without_education_at' => '',
-            'without_efz_at' => '',
-            'work_phone' => '',
-            'zip' => '42719',
-            'country_id' => $country->id,
-            'nationality_id' => $nationality->id,
-            'subscription_id' => $subscription->id,
             'first_activity_id' => $activity->id,
             'first_subactivity_id' => $subactivity->id,
         ];

tests/RequestFactories/MemberUpdateRequestFactory.php

@@ -0,0 +1,11 @@
+<?php
+
+namespace Tests\RequestFactories;
+
+class MemberUpdateRequestFactory extends MemberRequestFactory
+{
+    public function noNami(): self
+    {
+        return $this->state(['has_nami' => false]);
+    }
+}