Skip to content

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

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

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

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.2.


Note

Separates unit-tests into distinct C/C++ and Rust steps and updates artifact upload and failure conditions accordingly.

  • CI (GitHub Actions):
    • Workflow ./.github/workflows/task-test.yml:
      • Split unit testing into two steps: "C/C++ tests" (id: c_unit_tests, runs make unit-tests) and Rust tests (id: rust_unit_tests, runs make rust-tests).
      • Updated artifact upload conditions to check rust_unit_tests and c_unit_tests outcomes instead of unit_tests.
      • Updated failure gate to echo both test outcomes and fail if either c_unit_tests or rust_unit_tests (or flow tests) fail.
      • Adjusted step IDs/names accordingly and removed combined make unit-tests rust-tests run.

Written by Cursor Bugbot for commit 844fd24. 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 89.47%. Comparing base (e2b405d) to head (844fd24).
⚠️ Report is 1 commits behind head on 8.2.

Additional details and impacted files
@@            Coverage Diff             @@
##              8.2    #7591      +/-   ##
==========================================
- Coverage   89.54%   89.47%   -0.08%     
==========================================
  Files         253      253              
  Lines       41197    41197              
  Branches     3725     3725              
==========================================
- Hits        36889    36859      -30     
- Misses       4259     4289      +30     
  Partials       49       49              
Flag Coverage Δ
flow 82.55% <ø> (-0.31%) ⬇️
unit 47.46% <ø> (+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.2 with commit 5cc09cd Dec 1, 2025
25 checks passed
@redisearch-backport-pull-request redisearch-backport-pull-request bot deleted the backport-7587-to-8.2 branch December 1, 2025 13:19
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