centrifuge / centrifuge-chain

Centrifuge Chain: the layer-1 blockchain for real-world assets, built on Substrate.
https://centrifuge.io
GNU Lesser General Public License v3.0
182 stars 78 forks source link

IT: Support `#ignore = reason` for `test_runtimes` macro #1908

Closed lemunozm closed 2 months ago

lemunozm commented 2 months ago

Description

Gives support to ignore tests in integration tests with the #[test_runtimes()] macro.

Using ignore instead of commenting the test helps to not forget and leave the test commented when it could be fixed.

Now:

#[test_runtimes([development, altair, centrifuge], ignore)]
// or
#[test_runtimes([development, altair, centrifuge], ignore = "reason")]

NOTE: useful to work smoothly in the lpv2 branch when solidity/centrifuge-chain don't match correctly.

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.

Project coverage is 46.31%. Comparing base (2c1be11) to head (fb1f8de).

Files Patch % Lines
runtime/integration-tests/procedural/src/lib.rs 0.00% 7 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1908 +/- ## ======================================= Coverage 46.31% 46.31% ======================================= Files 179 179 Lines 13076 13081 +5 ======================================= + Hits 6056 6059 +3 - Misses 7020 7022 +2 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

lemunozm commented 2 months ago

Fixed clippy, can I have a reapproval? @wischli