From 00985037f44a19726805c483a3bd28bafa071876 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?C=C3=A9dric=20Moreau?= <cem.moreau@gmail.com> Date: Sat, 9 Dec 2023 17:25:42 +0100 Subject: [PATCH] Resolve "Tests E2E fails but CI pass" (nodes/rust/duniter-v2s!210) * fix clippy * update again metadata after rebase on master * panic if failed steps * fix(#154): E2E: parsing errors must make the test fail --- end2end-tests/tests/cucumber_tests.rs | 12 ++++++++++++ resources/metadata.scale | Bin 133152 -> 132945 bytes 2 files changed, 12 insertions(+) diff --git a/end2end-tests/tests/cucumber_tests.rs b/end2end-tests/tests/cucumber_tests.rs index b7c0804a6..92f1f36ee 100644 --- a/end2end-tests/tests/cucumber_tests.rs +++ b/end2end-tests/tests/cucumber_tests.rs @@ -558,6 +558,9 @@ async fn should_have_distance_ok(world: &mut DuniterWorld, who: String) -> Resul .await? { Some((_, gdev::runtime_types::pallet_distance::types::DistanceStatus::Valid)) => Ok(()), + Some((_, gdev::runtime_types::pallet_distance::types::DistanceStatus::Invalid)) => { + Err(anyhow::anyhow!("invalid distance status").into()) + } Some((_, gdev::runtime_types::pallet_distance::types::DistanceStatus::Pending)) => { Err(anyhow::anyhow!("pending distance status").into()) } @@ -669,9 +672,18 @@ async fn main() { .run(features_path) .await; + if summarize.failed_steps() > 0 { + panic!("Could not run tests correctly (failed steps)"); + } if summarize.hook_errors() > 0 { panic!("Could not run tests correctly (hook errors)"); } + if summarize.parsing_errors() > 0 { + panic!("Could not run tests correctly (parsing errors)"); + } + if summarize.execution_has_failed() { + panic!("Could not run tests correctly (execution has failed)"); + } } fn genesis_conf_name(feature_tags: &[String], scenario_tags: &[String]) -> String { diff --git a/resources/metadata.scale b/resources/metadata.scale index a4239406fb5544df7692aff4f117176809213953..fb9903cdcd8ab7a3f169487e4becee25afdc7d8c 100644 GIT binary patch delta 792 zcmZ25f#YHy#|CBxMw`iu9<EvpEHRD>xv9BHsYS&ZnFR{viNy+uDJiKbdM>_j(V*1a z{Iba(93*Wdic%Ac^YauL8Dr3-6LW&VQVfhNDHtXefehGu!@-qN2vgr=V<$f@73b8V z5=W55lQ%v$o_yL<i3cPCwmCJ0XR`b~qsi%BrrZ$0ywq~2!dqTln<u*XGcoc^e&{B` zC*zq{mY9>7!obA9V97Y$K9f;q@(vRoCV_$QtpVd?HBUWqz1DooYx^xP#%vKrlj$dl z7-M5xTr!JG67!N%J&R$!WMm0|2`Xe3D`Y}EuLl?O%P(;ShDT{)Nh&Z%STc|m=jE3u zq{1Yprxi1*Ffvc?E@lj5WSM@qn9-Y&b-HE=qc0=d^zstMAV&7-r$7|PblFlyM@G)+ s@uiIBj9k;FmjVU3r@ty?Ok?Dk9$3cMzdbIQF_VAjIC?wK(dQ=u0CSKK*8l(j delta 1051 zcmccE$FX1n#|CBxK@Z>5+@#c^;*87!$K=A&%%aJH4pw>@jtaSOafR~4VueJoWNM0D z0764ZYEf=xUSbJQi{#`U2QJGTOu5Ybyr9&?;`}@YMwS8$Q%j&GY(C-O$|#0i`($}1 zKTa9H)N<$4q7s407d#}Td@_rROH+&D!Bk3oa(-!Ei2x%{sj<;y!@JV*j4?&2$*GxT zKow91sv1U5WuBna-2AfC6p)!B)G#yA(~LVPH7~UsV&>%Y9>$w%oo_NpaL5Ft=A~rj zr86)vh=e8PWTr4MGB8*&PX6#nYVtM{J|TfY69op0lc#&=4au;cT<hG`9PhO~-iz^} zFr&eA$2i7VM-!LK;*!L?<W#@>68~&q0z3c{P{;r#>Ad_B1z7q}NK`1uFU~9hCK`p* zvc#Oy#1df2o&GD1QH7Cdx_mriAS3hi!gxk+W)_w?(>JCuN>0BK&uGNRI-NU#F^G|E zdK`#ipT04H(UFm3`h*fjb4JeT+KE6xuIY7&jA@MA+n**f{@@>+ZkrBtTmN>!GDe4q E06U{<)Bpeg -- GitLab