DefGuard / DefGuard/defguard

Propose a better way of testing email notifications

Open
#2,621 0 comments 0 reactions 0 assignees View on GitHub
optional for release
Dominant language
Rust
Stars
2.8k
Forks
115
Avg merge
1d 4h
Merged PRs (30d)
51

Description

### Description

Since we've migrated away from a shared email channel and now just use `send_and_forget()` it's not possible to test in our integration tests if correct emails are being sent in specific API scenarios.

Right now we do have some tests marked with:
```
#[ignore = "requires SMTP server"]
#[sqlx::test]
```
But this is not reasonable to run in CI.

Example test scenario I'd like to achieve:
- enable LDAP remote enrollment + automatic invites
- sync user from LDAP
- verify that the enrollment invite is sent

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.