diff --git a/src/tests/Browser/DistlistTest.php b/src/tests/Browser/DistlistTest.php --- a/src/tests/Browser/DistlistTest.php +++ b/src/tests/Browser/DistlistTest.php @@ -68,7 +68,7 @@ // Log on the user $this->browse(function (Browser $browser) { $browser->visit(new Home()) - ->submitLogon('john@kolab.org', 'simple123', true) + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase(), true) ->on(new Dashboard()) ->assertMissing('@links .link-distlists'); }); diff --git a/src/tests/Browser/DomainTest.php b/src/tests/Browser/DomainTest.php --- a/src/tests/Browser/DomainTest.php +++ b/src/tests/Browser/DomainTest.php @@ -40,7 +40,7 @@ // is passed to logon form and then "redirected" to the requested page $browser->visit('/domain/123') ->on(new Home()) - ->submitLogon('john@kolab.org', 'simple123') + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase()) ->assertErrorPage(404); }); } @@ -150,7 +150,7 @@ // Login the user $browser->visit('/login') ->on(new Home()) - ->submitLogon('john@kolab.org', 'simple123', true) + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase(), true) // On dashboard click the "Domains" link ->on(new Dashboard()) ->assertSeeIn('@links a.link-domains', 'Domains') @@ -182,7 +182,7 @@ // Login the user $browser->visit('/login') ->on(new Home()) - ->submitLogon('jack@kolab.org', 'simple123', true) + ->submitLogon('jack@kolab.org', \App\Utils::generatePassphrase(), true) ->on(new Dashboard()) ->assertVisible('@links a.link-profile') ->assertMissing('@links a.link-domains') diff --git a/src/tests/Browser/LogonTest.php b/src/tests/Browser/LogonTest.php --- a/src/tests/Browser/LogonTest.php +++ b/src/tests/Browser/LogonTest.php @@ -127,7 +127,7 @@ { $this->browse(function (Browser $browser) { $browser->visit(new Home()) - ->submitLogon('john@kolab.org', 'simple123', true) + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase(), true) // Checks if we're really on Dashboard page ->on(new Dashboard()) ->assertVisible('@links a.link-profile') @@ -203,7 +203,7 @@ { $this->browse(function (Browser $browser) { $browser->visit(new Home()) - ->submitLogon('john@kolab.org', 'simple123', true); + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase(), true); // Checks if we're really on Dashboard page $browser->on(new Dashboard()); @@ -234,7 +234,7 @@ // Test missing 2fa code $browser->on(new Home()) ->type('@email-input', 'ned@kolab.org') - ->type('@password-input', 'simple123') + ->type('@password-input', \App\Utils::generatePassphrase()) ->press('form button') ->waitFor('@second-factor-input.is-invalid + .invalid-feedback') ->assertSeeIn( @@ -282,7 +282,7 @@ $browser->script("localStorage.setItem('token', '123')"); $browser->refresh() ->on(new Home()) - ->submitLogon('john@kolab.org', 'simple123', false) + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase(), false) ->waitForLocation('/profile'); }); } diff --git a/src/tests/Browser/Meet/RoomControlsTest.php b/src/tests/Browser/Meet/RoomControlsTest.php --- a/src/tests/Browser/Meet/RoomControlsTest.php +++ b/src/tests/Browser/Meet/RoomControlsTest.php @@ -44,7 +44,7 @@ ->assertMissing('@chat') ->assertMissing('@setup-form') ->assertVisible('@login-form') - ->submitLogon('john@kolab.org', 'simple123') + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase()) ->waitFor('@setup-form') ->assertMissing('@login-form') ->waitUntilMissing('@setup-status-message.loading') @@ -84,7 +84,7 @@ // Join the room as an owner (authenticate) $owner->visit(new RoomPage('john')) ->click('@setup-button') - ->submitLogon('john@kolab.org', 'simple123') + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase()) ->waitFor('@setup-form') ->waitUntilMissing('@setup-status-message.loading') ->type('@setup-nickname-input', 'john') diff --git a/src/tests/Browser/Meet/RoomInterpretersTest.php b/src/tests/Browser/Meet/RoomInterpretersTest.php --- a/src/tests/Browser/Meet/RoomInterpretersTest.php +++ b/src/tests/Browser/Meet/RoomInterpretersTest.php @@ -36,7 +36,7 @@ // Join the room as an owner (authenticate) $owner->visit($page) ->click('@setup-button') - ->submitLogon('john@kolab.org', 'simple123') + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase()) ->waitFor('@setup-form') ->waitUntilMissing('@setup-status-message.loading') ->type('@setup-nickname-input', 'John') diff --git a/src/tests/Browser/Meet/RoomModeratorTest.php b/src/tests/Browser/Meet/RoomModeratorTest.php --- a/src/tests/Browser/Meet/RoomModeratorTest.php +++ b/src/tests/Browser/Meet/RoomModeratorTest.php @@ -37,7 +37,7 @@ // In one browser window join as a room owner $browser->visit(new RoomPage('john')) ->click('@setup-button') - ->submitLogon('john@kolab.org', 'simple123') + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase()) ->waitFor('@setup-form') ->waitUntilMissing('@setup-status-message.loading') ->type('@setup-nickname-input', 'John') diff --git a/src/tests/Browser/Meet/RoomOptionsTest.php b/src/tests/Browser/Meet/RoomOptionsTest.php --- a/src/tests/Browser/Meet/RoomOptionsTest.php +++ b/src/tests/Browser/Meet/RoomOptionsTest.php @@ -39,7 +39,7 @@ // Join the room as an owner (authenticate) $owner->visit(new RoomPage('john')) ->click('@setup-button') - ->submitLogon('john@kolab.org', 'simple123') + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase()) ->waitFor('@setup-form') ->waitUntilMissing('@setup-status-message.loading') ->assertMissing('@setup-password-input') @@ -127,7 +127,7 @@ // Join the room as an owner (authenticate) $owner->visit(new RoomPage('john')) // ->click('@setup-button') - // ->submitLogon('john@kolab.org', 'simple123') + // ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase()) ->waitFor('@setup-form') ->waitUntilMissing('@setup-status-message.loading') ->type('@setup-nickname-input', 'John') @@ -201,7 +201,7 @@ // Join the room as an owner (authenticate) $owner->visit(new RoomPage('john')) // ->click('@setup-button') - // ->submitLogon('john@kolab.org', 'simple123') + // ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase()) ->waitFor('@setup-form') ->waitUntilMissing('@setup-status-message.loading') ->type('@setup-nickname-input', 'John') @@ -279,7 +279,7 @@ // Join the room as an owner (authenticate) $owner->visit(new RoomPage('john')) // ->click('@setup-button') - // ->submitLogon('john@kolab.org', 'simple123') + // ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase()) ->waitFor('@setup-form') ->waitUntilMissing('@setup-status-message.loading') ->type('@setup-nickname-input', 'John') diff --git a/src/tests/Browser/Meet/RoomQATest.php b/src/tests/Browser/Meet/RoomQATest.php --- a/src/tests/Browser/Meet/RoomQATest.php +++ b/src/tests/Browser/Meet/RoomQATest.php @@ -35,7 +35,7 @@ // Join the room as an owner (authenticate) $owner->visit(new RoomPage('john')) ->click('@setup-button') - ->submitLogon('john@kolab.org', 'simple123') + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase()) ->waitFor('@setup-form') ->waitUntilMissing('@setup-status-message.loading') ->select('@setup-mic-select', '') diff --git a/src/tests/Browser/Meet/RoomSetupTest.php b/src/tests/Browser/Meet/RoomSetupTest.php --- a/src/tests/Browser/Meet/RoomSetupTest.php +++ b/src/tests/Browser/Meet/RoomSetupTest.php @@ -147,7 +147,7 @@ ->assertMissing('@chat') ->assertMissing('@setup-form') ->assertVisible('@login-form') - ->submitLogon('john@kolab.org', 'simple123') + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase()) ->waitFor('@setup-form') ->within(new Menu(), function ($browser) { $browser->assertMenuItems(['explore', 'blog', 'support', 'dashboard', 'logout', 'lang']); diff --git a/src/tests/Browser/Meet/RoomsTest.php b/src/tests/Browser/Meet/RoomsTest.php --- a/src/tests/Browser/Meet/RoomsTest.php +++ b/src/tests/Browser/Meet/RoomsTest.php @@ -43,7 +43,7 @@ $browser->visit('/rooms') ->on(new Home()) // User has no 'meet' entitlement yet, expect redirect to error page - ->submitLogon('john@kolab.org', 'simple123', false) + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase(), false) ->waitFor('#app > #error-page') ->assertSeeIn('#error-page .code', '403') ->assertSeeIn('#error-page .message', 'Access denied'); @@ -64,7 +64,7 @@ // User has no 'meet' entitlement yet $browser->visit('/login') ->on(new Home()) - ->submitLogon('john@kolab.org', 'simple123', true) + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase(), true) ->on(new Dashboard()) ->assertMissing('@links a.link-chat'); diff --git a/src/tests/Browser/PaymentMollieTest.php b/src/tests/Browser/PaymentMollieTest.php --- a/src/tests/Browser/PaymentMollieTest.php +++ b/src/tests/Browser/PaymentMollieTest.php @@ -43,12 +43,12 @@ public function testPayment(): void { $user = $this->getTestUser('payment-test@kolabnow.com', [ - 'password' => 'simple123', + 'password' => \App\Utils::generatePassphrase(), ]); $this->browse(function (Browser $browser) use ($user) { $browser->visit(new Home()) - ->submitLogon('payment-test@kolabnow.com', 'simple123', true, ['paymentProvider' => 'mollie']) + ->submitLogon('payment-test@kolabnow.com', \App\Utils::generatePassphrase(), true, ['paymentProvider' => 'mollie']) ->on(new Dashboard()) ->click('@links .link-wallet') ->on(new WalletPage()) @@ -117,12 +117,12 @@ public function testAutoPaymentSetup(): void { $user = $this->getTestUser('payment-test@kolabnow.com', [ - 'password' => 'simple123', + 'password' => \App\Utils::generatePassphrase(), ]); $this->browse(function (Browser $browser) use ($user) { $browser->visit(new Home()) - ->submitLogon('payment-test@kolabnow.com', 'simple123', true, ['paymentProvider' => 'mollie']) + ->submitLogon('payment-test@kolabnow.com', \App\Utils::generatePassphrase(), true, ['paymentProvider' => 'mollie']) ->on(new Dashboard()) ->click('@links .link-wallet') ->on(new WalletPage()) diff --git a/src/tests/Browser/PaymentStripeTest.php b/src/tests/Browser/PaymentStripeTest.php --- a/src/tests/Browser/PaymentStripeTest.php +++ b/src/tests/Browser/PaymentStripeTest.php @@ -43,12 +43,12 @@ public function testPayment(): void { $user = $this->getTestUser('payment-test@kolabnow.com', [ - 'password' => 'simple123', + 'password' => \App\Utils::generatePassphrase(), ]); $this->browse(function (Browser $browser) use ($user) { $browser->visit(new Home()) - ->submitLogon('payment-test@kolabnow.com', 'simple123', true, ['paymentProvider' => 'stripe']) + ->submitLogon('payment-test@kolabnow.com', \App\Utils::generatePassphrase(), true, ['paymentProvider' => 'stripe']) ->on(new Dashboard()) ->click('@links .link-wallet') ->on(new WalletPage()) @@ -103,13 +103,13 @@ public function testAutoPaymentSetup(): void { $user = $this->getTestUser('payment-test@kolabnow.com', [ - 'password' => 'simple123', + 'password' => \App\Utils::generatePassphrase(), ]); // Test creating auto-payment $this->browse(function (Browser $browser) use ($user) { $browser->visit(new Home()) - ->submitLogon('payment-test@kolabnow.com', 'simple123', true, ['paymentProvider' => 'stripe']) + ->submitLogon('payment-test@kolabnow.com', \App\Utils::generatePassphrase(), true, ['paymentProvider' => 'stripe']) ->on(new Dashboard()) ->click('@links .link-wallet') ->on(new WalletPage()) diff --git a/src/tests/Browser/StatusTest.php b/src/tests/Browser/StatusTest.php --- a/src/tests/Browser/StatusTest.php +++ b/src/tests/Browser/StatusTest.php @@ -72,7 +72,7 @@ $this->browse(function ($browser) use ($john, $domain) { $browser->visit(new Home()) - ->submitLogon('john@kolab.org', 'simple123', true) + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase(), true) ->on(new Dashboard()) ->with(new Status(), function ($browser) use ($john) { $browser->assertSeeIn('@body', 'We are preparing your account') diff --git a/src/tests/Browser/UserProfileTest.php b/src/tests/Browser/UserProfileTest.php --- a/src/tests/Browser/UserProfileTest.php +++ b/src/tests/Browser/UserProfileTest.php @@ -64,7 +64,7 @@ { $this->browse(function (Browser $browser) { $browser->visit(new Home()) - ->submitLogon('john@kolab.org', 'simple123', true) + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase(), true) ->on(new Dashboard()) ->assertSeeIn('@links .link-profile', 'Your profile') ->click('@links .link-profile') @@ -135,7 +135,7 @@ $this->browse(function (Browser $browser) { $browser->visit('/logout') ->visit(new Home()) - ->submitLogon('jack@kolab.org', 'simple123', true) + ->submitLogon('jack@kolab.org', \App\Utils::generatePassphrase(), true) ->on(new Dashboard()) ->assertSeeIn('@links .link-profile', 'Your profile') ->click('@links .link-profile') @@ -157,12 +157,12 @@ */ public function testProfileDelete(): void { - $user = $this->getTestUser('profile-delete@kolabnow.com', ['password' => 'simple123']); + $user = $this->getTestUser('profile-delete@kolabnow.com', ['password' => \App\Utils::generatePassphrase()]); $this->browse(function (Browser $browser) use ($user) { $browser->visit('/logout') ->on(new Home()) - ->submitLogon('profile-delete@kolabnow.com', 'simple123', true) + ->submitLogon('profile-delete@kolabnow.com', \App\Utils::generatePassphrase(), true) ->on(new Dashboard()) ->assertSeeIn('@links .link-profile', 'Your profile') ->click('@links .link-profile') diff --git a/src/tests/Browser/UsersTest.php b/src/tests/Browser/UsersTest.php --- a/src/tests/Browser/UsersTest.php +++ b/src/tests/Browser/UsersTest.php @@ -117,7 +117,7 @@ // Test that the page requires authentication $this->browse(function (Browser $browser) { $browser->visit(new Home()) - ->submitLogon('john@kolab.org', 'simple123', true) + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase(), true) ->on(new Dashboard()) ->assertSeeIn('@links .link-users', 'User accounts') ->click('@links .link-users') @@ -418,7 +418,7 @@ ->type('#email', 'invalid email') ->click('button[type=submit]') ->assertFocused('#password') - ->type('#password', 'simple123') + ->type('#password', \App\Utils::generatePassphrase()) ->click('button[type=submit]') ->assertFocused('#password_confirmation') ->type('#password_confirmation', 'simple') @@ -431,7 +431,7 @@ // Test form error handling (aliases) $browser->with('@general', function (Browser $browser) { $browser->type('#email', 'julia.roberts@kolab.org') - ->type('#password_confirmation', 'simple123') + ->type('#password_confirmation', \App\Utils::generatePassphrase()) ->with(new ListInput('#aliases'), function (Browser $browser) { $browser->addListEntry('invalid address'); }) @@ -552,7 +552,7 @@ $this->browse(function (Browser $browser) { $browser->visit('/logout') ->on(new Home()) - ->submitLogon('jack@kolab.org', 'simple123', true) + ->submitLogon('jack@kolab.org', \App\Utils::generatePassphrase(), true) ->visit('/users') ->assertErrorPage(403); }); @@ -561,7 +561,7 @@ $this->browse(function (Browser $browser) { $browser->visit('/logout') ->on(new Home()) - ->submitLogon('ned@kolab.org', 'simple123', true) + ->submitLogon('ned@kolab.org', \App\Utils::generatePassphrase(), true) ->visit(new UserList()) ->whenAvailable('@table', function (Browser $browser) { $browser->assertElementsCount('tbody tr', 4); @@ -589,7 +589,7 @@ $this->browse(function (Browser $browser) { $browser->visit('/logout') ->on(new Home()) - ->submitLogon('john@kolab.org', 'simple123', true) + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase(), true) ->visit(new UserList()) ->waitFor('@table tr:nth-child(2)') ->click('@table tr:nth-child(2) a') // joe@kolab.org diff --git a/src/tests/Browser/WalletTest.php b/src/tests/Browser/WalletTest.php --- a/src/tests/Browser/WalletTest.php +++ b/src/tests/Browser/WalletTest.php @@ -61,7 +61,7 @@ // Test that the page requires authentication $this->browse(function (Browser $browser) { $browser->visit(new Home()) - ->submitLogon('john@kolab.org', 'simple123', true) + ->submitLogon('john@kolab.org', \App\Utils::generatePassphrase(), true) ->on(new Dashboard()) ->assertSeeIn('@links .link-wallet .name', 'Wallet') ->assertSeeIn('@links .link-wallet .badge', '-12,34 CHF'); @@ -89,7 +89,7 @@ */ public function testReceipts(): void { - $user = $this->getTestUser('wallets-controller@kolabnow.com', ['password' => 'simple123']); + $user = $this->getTestUser('wallets-controller@kolabnow.com', ['password' => \App\Utils::generatePassphrase()]); $wallet = $user->wallets()->first(); $wallet->payments()->delete(); @@ -98,7 +98,7 @@ $browser->visit('/logout') ->waitForLocation('/login') ->on(new Home()) - ->submitLogon('wallets-controller@kolabnow.com', 'simple123', true); + ->submitLogon('wallets-controller@kolabnow.com', \App\Utils::generatePassphrase(), true); }); // Assert Receipts tab content when there's no receipts available @@ -190,14 +190,14 @@ */ public function testHistory(): void { - $user = $this->getTestUser('wallets-controller@kolabnow.com', ['password' => 'simple123']); + $user = $this->getTestUser('wallets-controller@kolabnow.com', ['password' => \App\Utils::generatePassphrase()]); // Log out John and log in the test user $this->browse(function (Browser $browser) { $browser->visit('/logout') ->waitForLocation('/login') ->on(new Home()) - ->submitLogon('wallets-controller@kolabnow.com', 'simple123', true); + ->submitLogon('wallets-controller@kolabnow.com', \App\Utils::generatePassphrase(), true); }); $package_kolab = \App\Package::where('title', 'kolab')->first(); @@ -266,7 +266,7 @@ $this->browse(function (Browser $browser) { $browser->visit('/logout') ->on(new Home()) - ->submitLogon('jack@kolab.org', 'simple123', true) + ->submitLogon('jack@kolab.org', \App\Utils::generatePassphrase(), true) ->on(new Dashboard()) ->assertMissing('@links .link-wallet') ->visit('/wallet') diff --git a/src/tests/Feature/Controller/AuthTest.php b/src/tests/Feature/Controller/AuthTest.php --- a/src/tests/Feature/Controller/AuthTest.php +++ b/src/tests/Feature/Controller/AuthTest.php @@ -58,7 +58,7 @@ // Test token refresh via the info request // First we log in as we need the token (actingAs() will not work) - $post = ['email' => 'john@kolab.org', 'password' => 'simple123']; + $post = ['email' => 'john@kolab.org', 'password' => \App\Utils::generatePassphrase()]; $response = $this->post("api/auth/login", $post); $json = $response->json(); $response = $this->withHeaders(['Authorization' => 'Bearer ' . $json['access_token']]) @@ -102,7 +102,7 @@ // Valid user+password $user = $this->getTestUser('john@kolab.org'); - $post = ['email' => 'john@kolab.org', 'password' => 'simple123']; + $post = ['email' => 'john@kolab.org', 'password' => \App\Utils::generatePassphrase()]; $response = $this->post("api/auth/login", $post); $json = $response->json(); @@ -117,7 +117,7 @@ $this->assertTrue(is_array($json['aliases'])); // Valid user+password (upper-case) - $post = ['email' => 'John@Kolab.org', 'password' => 'simple123']; + $post = ['email' => 'John@Kolab.org', 'password' => \App\Utils::generatePassphrase()]; $response = $this->post("api/auth/login", $post); $json = $response->json(); @@ -174,7 +174,7 @@ $response->assertStatus(401); // Login the user to get a valid token - $post = ['email' => 'john@kolab.org', 'password' => 'simple123']; + $post = ['email' => 'john@kolab.org', 'password' => \App\Utils::generatePassphrase()]; $response = $this->post("api/auth/login", $post); $response->assertStatus(200); $json = $response->json(); diff --git a/src/tests/Feature/Controller/UsersTest.php b/src/tests/Feature/Controller/UsersTest.php --- a/src/tests/Feature/Controller/UsersTest.php +++ b/src/tests/Feature/Controller/UsersTest.php @@ -800,7 +800,7 @@ // Test error on some invalid aliases missing password confirmation $post = [ - 'password' => 'simple123', + 'password' => \App\Utils::generatePassphrase(), 'aliases' => [ 'useralias2@' . \config('app.domain'), 'useralias1@kolab.org', diff --git a/src/tests/Feature/DomainOwnerTest.php b/src/tests/Feature/DomainOwnerTest.php --- a/src/tests/Feature/DomainOwnerTest.php +++ b/src/tests/Feature/DomainOwnerTest.php @@ -36,7 +36,7 @@ [ 'name' => 'Jane Doe', 'email' => 'jane@kolab.org', - 'password' => 'simple123', + 'password' => \App\Utils::generatePassphrase(), 'email_verified_at' => now() ] );