Skip to content

[8.4] [MOD-12701] Split the execution of Rust and C/C++ unit tests across two different CI steps.#7592

Merged
redisearch-backport-pull-request[bot] merged 1 commit into8.4from
backport-7587-to-8.4
Dec 1, 2025
Merged

[8.4] [MOD-12701] Split the execution of Rust and C/C++ unit tests across two different CI steps.#7592
redisearch-backport-pull-request[bot] merged 1 commit into8.4from
backport-7587-to-8.4

Conversation

@redisearch-backport-pull-request
Copy link
Contributor

@redisearch-backport-pull-request redisearch-backport-pull-request bot commented Dec 1, 2025

Description

Backport of #7587 to 8.4.


Note

Separates unit tests into distinct C/C++ and Rust steps and updates artifact upload and fail conditions to track each independently.

  • CI Workflow (.github/workflows/task-test.yml)
    • Tests:
      • Split unit tests into separate steps: "C/C++ tests" (id: c_unit_tests, make unit-tests) and Rust tests (id: rust_unit_tests, make rust-tests).
    • Artifacts & Failures:
      • Update artifact upload conditions to trigger on failures of rust_unit_tests or c_unit_tests (alongside flow tests).
      • Update final failure gate to consider and print outcomes for both rust_unit_tests and c_unit_tests.

Written by Cursor Bugbot for commit 5bac834. This will update automatically on new commits. Configure here.

…wo different CI steps. (#7587)

Split the execution of Rust and C/C++ unit tests across two different CI steps.

(cherry picked from commit e9bc53b)
@codecov
Copy link

codecov bot commented Dec 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.91%. Comparing base (fbccafd) to head (5bac834).
⚠️ Report is 1 commits behind head on 8.4.

Additional details and impacted files
@@            Coverage Diff             @@
##              8.4    #7592      +/-   ##
==========================================
- Coverage   85.94%   85.91%   -0.04%     
==========================================
  Files         331      331              
  Lines       52701    52701              
  Branches    12004    12004              
==========================================
- Hits        45295    45276      -19     
- Misses       7239     7258      +19     
  Partials      167      167              
Flag Coverage Δ
flow 84.48% <ø> (-0.15%) ⬇️
unit 52.37% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@redisearch-backport-pull-request redisearch-backport-pull-request bot added this pull request to the merge queue Dec 1, 2025
Merged via the queue into 8.4 with commit 506a7b7 Dec 1, 2025
28 checks passed
@redisearch-backport-pull-request redisearch-backport-pull-request bot deleted the backport-7587-to-8.4 branch December 1, 2025 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant