Skip to content

Add spec for integration testing infrastructure - #4584

Merged
uglide merged 3 commits into
masterfrom
im/testing-spec
Jun 30, 2026
Merged

Add spec for integration testing infrastructure#4584
uglide merged 3 commits into
masterfrom
im/testing-spec

Conversation

@uglide

@uglide uglide commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Note

Low Risk
Documentation and site navigation only; no runtime or test code changes.

Overview
Adds docs/integration-testing.md, a full spec for how Jedis integration tests work: the redislabs/client-libs-test Docker setup, docker-compose / Makefile workflows, endpoints.json and Endpoints.getRedisEndpoint, TLS via TlsUtil, unit vs integration naming (*IT for new tests), and how CI (test-on-docker vs integration) maps to TEST_ENV_PROVIDER.

CONTRIBUTING.md is shortened: community support points to Redis Discord instead of Google Groups; the long inline Docker/Makefile testing section is replaced by a link to the new guide plus explicit *IT / unit-test rules. The “adding commands” subsection is removed from contributing.

mkdocs.yml adds a Development guide nav section linking Contributing, Integration Testing, and benchmarks.

Reviewed by Cursor Bugbot for commit ce3042b. Bugbot is set up for automated code reviews on this repo. Configure here.

@jit-ci

jit-ci Bot commented Jun 30, 2026

Copy link
Copy Markdown

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

@github-actions

github-actions Bot commented Jun 30, 2026

Copy link
Copy Markdown

Test Results

  205 files  ± 0    205 suites  ±0   10m 20s ⏱️ +39s
7 801 tests +12  7 735 ✅ +939  66 💤  - 927  0 ❌ ±0 
7 821 runs  +24  7 755 ✅ +959  66 💤  - 935  0 ❌ ±0 

Results for commit ce3042b. ± Comparison against base commit b405db6.

This pull request removes 2 and adds 14 tests. Note that renamed tests count towards both.
redis.clients.jedis.commands.unified.search.FTHybridCommandsTestBase$SupportedScorersTest ‑ testScorer(Scorer, double, double)[1]
redis.clients.jedis.commands.unified.search.FTHybridCommandsTestBase$SupportedScorersTest ‑ testScorer(Scorer, double, double)[2]
redis.clients.jedis.commands.unified.search.FTHybridCommandsTestBase$SupportedScorersTest[1] ‑ testScorer(Scorer, double, double)[1][1]
redis.clients.jedis.commands.unified.search.FTHybridCommandsTestBase$SupportedScorersTest[1] ‑ testScorer(Scorer, double, double)[1][2]
redis.clients.jedis.commands.unified.search.FTHybridCommandsTestBase$SupportedScorersTest[1] ‑ testScorer(Scorer, double, double)[1][3]
redis.clients.jedis.commands.unified.search.FTHybridCommandsTestBase$SupportedScorersTest[1] ‑ testScorer(Scorer, double, double)[1][4]
redis.clients.jedis.commands.unified.search.FTHybridCommandsTestBase$SupportedScorersTest[1] ‑ testScorer(Scorer, double, double)[1][5]
redis.clients.jedis.commands.unified.search.FTHybridCommandsTestBase$SupportedScorersTest[1] ‑ testScorer(Scorer, double, double)[1][6]
redis.clients.jedis.commands.unified.search.FTHybridCommandsTestBase$SupportedScorersTest[1] ‑ testScorer(Scorer, double, double)[1][7]
redis.clients.jedis.commands.unified.search.FTHybridCommandsTestBase$SupportedScorersTest[2] ‑ testScorer(Scorer, double, double)[2][1]
redis.clients.jedis.commands.unified.search.FTHybridCommandsTestBase$SupportedScorersTest[2] ‑ testScorer(Scorer, double, double)[2][2]
redis.clients.jedis.commands.unified.search.FTHybridCommandsTestBase$SupportedScorersTest[2] ‑ testScorer(Scorer, double, double)[2][3]
…

♻️ This comment has been updated with latest results.

@uglide
uglide requested a review from ggivo June 30, 2026 10:58
Comment thread .github/CONTRIBUTING.md Outdated
Comment thread docs/integration-testing.md Outdated
@uglide
uglide merged commit fae6da2 into master Jun 30, 2026
18 checks passed
@uglide
uglide deleted the im/testing-spec branch June 30, 2026 12:23
@ggivo ggivo added maintenance churn Chore: Formatting and comment cleanup (non-functional change) labels Aug 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

churn Chore: Formatting and comment cleanup (non-functional change) maintenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants