search

omnivery / mautic-omnivery-plugin

Email transport using Omnivery API
https://omnivery.com/
GNU General Public License v3.0
4 stars 0 forks source link

Better integration test #26

Open axelocz opened 2 years ago

axelocz commented 2 years ago

test connection in email settings doesn't really do much of a test. i suggest the test actually tests the connection and authentication.

this could be easily done by using the parse email endpoint https://docs.omnivery.com/docs/mailgun-api-v3/517872689535a-parse-email-address

mzagmajster commented 2 years ago

Right beside Test connection you have send test email button which I would guess at some point must authenticate to successfully send message.

While I have not checked if we can manipulate/override test connection button behavior.

Its a lot easier to create new button on Omnivery Settings tab (it exists now in configuration) and call this endpoint. Let me know what you think.