Standardize test block syntax for consistency with existing patterns

[Copilot]

Two spec files used multi-line expect do...end blocks while the codebase standard (established in AffiliateBorrowRequestFormSpec) uses inline expect { } syntax.

Changes

• spec/models/doemoff_patron_email_form_spec.rb: Converted expect do...end to expect { }
• spec/models/efees_invoice_spec.rb: Converted expect do...end to expect { }

Example

# Before
expect do
  @form.submit!
end.to have_enqueued_job(ActionMailer::MailDeliveryJob).with(...)

# After
expect { @form.submit! }.to have_enqueued_job(ActionMailer::MailDeliveryJob).with(...)

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[Copilot]

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.