https://blogs.gentoo.org/ago/2020/07/04/gentoo-tinderbox/ Issue: app-shells/atuin-18.0.1 fails tests. Discovered on: amd64 (internal ref: ci) Info about the issue: https://wiki.gentoo.org/wiki/Project:Tinderbox/Common_Issues_Helper#CF0015
Created attachment 885601 [details] build.log build log and emerge --info
Error(s) that match a know pattern in addition to what has been reported in the summary: thread 'change_password' panicked at atuin/tests/common/mod.rs:57:13: thread 'change_password' panicked at atuin/tests/common/mod.rs:77:10: thread 'multi_user_test' panicked at atuin/tests/common/mod.rs:57:13: thread 'multi_user_test' panicked at atuin/tests/common/mod.rs:77:10: thread 'registration' panicked at atuin/tests/common/mod.rs:57:13: thread 'registration' panicked at atuin/tests/common/mod.rs:77:10: called `Result::unwrap()` on an `Err` value: error sending request for url (http://127.0.0.1:43437/018dcb4cc63a7b5885753f968e7ccfaf/user/018dcb4cc70b7669904af251bc0c9f99): connection error: Connection reset by peer (os error 104)
@Jonas Would you mind taking a stab at this? It seems atuin grew some tests that require network. We should probably add a #[ignore] annotation to them.
Sure thing, I'll take a look later today
The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=6663153d5011f64051edc3e6b58b929fe5b05232 commit 6663153d5011f64051edc3e6b58b929fe5b05232 Author: Florian Schmaus <flow@gentoo.org> AuthorDate: 2024-02-23 11:28:34 +0000 Commit: Florian Schmaus <flow@gentoo.org> CommitDate: 2024-02-23 12:17:09 +0000 app-shells/atuin: enable server tests Closes: https://bugs.gentoo.org/925163 Closes: https://github.com/gentoo/gentoo/pull/35495 Signed-off-by: Florian Schmaus <flow@gentoo.org> app-shells/atuin/atuin-18.0.1.ebuild | 42 ++++++++++++++++++++++++++++++------ 1 file changed, 36 insertions(+), 6 deletions(-)
I found that the tests did not fail because they required network access, they failed because they required as postgres instance. This is now fixed and all atuin tests are enabled. \o/
