stalwartlabs / smtp-server

Stalwart SMTP Server
https://stalw.art/smtp
GNU Affero General Public License v3.0
473 stars 24 forks source link

Tests failed on local #18

Open Dennis-Zhang-SH opened 1 year ago

Dennis-Zhang-SH commented 1 year ago

I pulled the repo and run cargo test on local environment, it failed 2 tests.

failures:

---- tests::outbound::lmtp::lmtp_delivery stdout ---- thread 'tests::outbound::lmtp::lmtp_delivery' panicked at 'assertion failed: (left == right) left: 0, right: 4', src/tests/outbound/lmtp.rs:148:5 note: run with RUST_BACKTRACE=1 environment variable to display a backtrace

---- tests::outbound::smtp::smtp_delivery stdout ---- thread 'tests::outbound::smtp::smtp_delivery' panicked at 'retries [0, 0, 0]', src/tests/outbound/smtp.rs:173:5

failures: tests::outbound::lmtp::lmtp_delivery tests::outbound::smtp::smtp_delivery

mdecimus commented 1 year ago

Have you tried running the tests more than once? This is most likely caused by a timeout issue, some of the tests are time sensitive and might fail if your environment is faster/slower than the one where the tests were written. That being said, this is something that needs to be fixed in the tests.

Dennis-Zhang-SH commented 1 year ago

Have you tried running the tests more than once? This is most likely caused by a timeout issue, some of the tests are time sensitive and might fail if your environment is faster/slower than the one where the tests were written. That being said, this is something that needs to be fixed in the tests.

Hi

After serveral times retrying cargo clean && cargo test, it still failed these two tests.