From 23cdc0392e352bfe0e597add3553d080764c959b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Vi=CC=81ctor=20Falco=CC=81n?= Date: Fri, 12 Jun 2026 11:23:45 +0100 Subject: [PATCH] test(profile): drop month_start_day from flag-off request payloads Mirror real requests: the settings form only renders (and submits) the field while CustomMonthStartDay is active, so the currency and email tests should not send it with the flag off. --- tests/Feature/Settings/ProfileUpdateTest.php | 24 +++++--------------- 1 file changed, 6 insertions(+), 18 deletions(-) diff --git a/tests/Feature/Settings/ProfileUpdateTest.php b/tests/Feature/Settings/ProfileUpdateTest.php index 3d77e908..f5b9b56f 100644 --- a/tests/Feature/Settings/ProfileUpdateTest.php +++ b/tests/Feature/Settings/ProfileUpdateTest.php @@ -48,9 +48,7 @@ test('profile accepts new latam primary currency', function () { ->patch(route('profile.update'), [ 'name' => 'Test User', 'email' => 'test@example.com', - 'currency_code' => 'ARS', - 'month_start_day' => 1, - ]); + 'currency_code' => 'ARS', ]); $response ->assertSessionHasNoErrors() @@ -67,9 +65,7 @@ test('profile accepts Pakistani rupee as primary currency', function () { ->patch(route('profile.update'), [ 'name' => 'Test User', 'email' => 'test@example.com', - 'currency_code' => 'PKR', - 'month_start_day' => 1, - ]); + 'currency_code' => 'PKR', ]); $response ->assertSessionHasNoErrors() @@ -86,9 +82,7 @@ test('profile accepts Brazilian real as primary currency', function () { ->patch(route('profile.update'), [ 'name' => 'Test User', 'email' => 'test@example.com', - 'currency_code' => 'BRL', - 'month_start_day' => 1, - ]); + 'currency_code' => 'BRL', ]); $response ->assertSessionHasNoErrors() @@ -105,9 +99,7 @@ test('profile accepts Saudi riyal as primary currency', function () { ->patch(route('profile.update'), [ 'name' => 'Test User', 'email' => 'test@example.com', - 'currency_code' => 'SAR', - 'month_start_day' => 1, - ]); + 'currency_code' => 'SAR', ]); $response ->assertSessionHasNoErrors() @@ -124,9 +116,7 @@ test('profile rejects bitcoin as primary currency', function () { ->patch(route('profile.update'), [ 'name' => 'Test User', 'email' => 'test@example.com', - 'currency_code' => 'BTC', - 'month_start_day' => 1, - ]); + 'currency_code' => 'BTC', ]); $response->assertSessionHasErrors(['currency_code']); }); @@ -174,9 +164,7 @@ test('email verification status is unchanged when the email address is unchanged ->patch(route('profile.update'), [ 'name' => 'Test User', 'email' => $user->email, - 'currency_code' => $user->currency_code, - 'month_start_day' => 1, - ]); + 'currency_code' => $user->currency_code, ]); $response ->assertSessionHasNoErrors()