Timshel

Results 60 comments of Timshel

Added `docker-compose` control in `Playwright` so to run all tests: ```bash export DOCKER_BUILDKIT=1 docker-compose --env-file test.env run Playwright ```

Added integration with https://github.com/maildev/maildev in the tests to be able to check the mail sent. This will allow to test things like 2FA mail authentication and organization invitation.

Added some Organization invitation tests and switched to the `compose` plugin instead of `docker-compose`. This now depends on : - https://github.com/dani-garcia/vaultwarden/pull/4827 - https://github.com/dani-garcia/vaultwarden/pull/4856

Rebased on main since both previously PR are merged :). Added some email 2FA tests :).

Made the change differently with the following logic: - I believe the root error should be logged so no change here. - I don't think it makes sense to log...

Cf https://github.com/dani-garcia/vaultwarden/pull/3899#issuecomment-2349876938

Hey, Not a maintainer but have you tested adding your patch on an existing PG database ? Because unless I'm mistaken from the point of view of existing deployment you...

From a quick search Diesel run each migration in a transaction. PG and cockroachdb appears to support nested transaction, but it will probably still trigger the bug (Edit: maybe in...

> if it works in cockroachdb would you be willing to test in postgres? Not very hard to spin-up PG :sweat_smile:, but yes should have some time to test it

I'll mention that all the `sso.roles` and `sso.organizations` config/features are not part of the merged PR.