From c642e6d5505091a9aa9d81647aca70fbff23815a Mon Sep 17 00:00:00 2001 From: Piotr Dybowski Date: Fri, 8 Oct 2021 10:41:40 +0200 Subject: [PATCH] Fix example in fixture documentation (`sending_user`, `receiving_user`) (#9179) --- AUTHORS | 1 + doc/en/fixture.rst | 8 ++++---- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/AUTHORS b/AUTHORS index 2c690c5d2..2c83c84e9 100644 --- a/AUTHORS +++ b/AUTHORS @@ -240,6 +240,7 @@ Petter Strandmark Philipp Loose Pieter Mulder Piotr Banaszkiewicz +Piotr Dybowski Piotr Helm Prakhar Gurunani Prashant Anand diff --git a/doc/en/fixture.rst b/doc/en/fixture.rst index d8046fb7f..959a99dd5 100644 --- a/doc/en/fixture.rst +++ b/doc/en/fixture.rst @@ -869,19 +869,19 @@ Here's what that might look like: def sending_user(mail_admin): user = mail_admin.create_user() yield user - admin_client.delete_user(user) + mail_admin.delete_user(user) @pytest.fixture def receiving_user(mail_admin): user = mail_admin.create_user() yield user - admin_client.delete_user(user) + mail_admin.delete_user(user) - def test_email_received(receiving_user, email): + def test_email_received(receiving_user, sending_user): email = Email(subject="Hey!", body="How's it going?") - sending_user.send_email(_email, receiving_user) + sending_user.send_email(email, receiving_user) assert email in receiving_user.inbox Because ``receiving_user`` is the last fixture to run during setup, it's the first to run