Skip to content

Align tests with RE#447

Merged
mgravell merged 22 commits into
masterfrom
im/align-tests-with-re
Dec 8, 2025
Merged

Align tests with RE#447
mgravell merged 22 commits into
masterfrom
im/align-tests-with-re

Conversation

@uglide

@uglide uglide commented Sep 30, 2025

Copy link
Copy Markdown
Contributor

Improve assertions to make tests compatible with RE

@jit-ci

jit-ci Bot commented Sep 30, 2025

Copy link
Copy Markdown

Hi, I’m Jit, a friendly security platform designed to help developers build secure applications from day zero with an MVS (Minimal viable security) mindset.

In case there are security findings, they will be communicated to you as a comment inside the PR.

Hope you’ll enjoy using Jit.

Questions? Comments? Want to learn more? Get in touch with us.

@mgravell

Copy link
Copy Markdown
Collaborator

Note that the failing size assertions are pretty new - if they're going to cause problems, we can probably just... not do that.

@uglide

uglide commented Sep 30, 2025

Copy link
Copy Markdown
Contributor Author

@mgravell, they are actually quite helpful for the case when the tested endpoint is not local but a remote RE cluster with some extra latency. I just need to fix some mistakes to make it work with both OSS and RE

* Add 8.4 to CI matrix integration.yml

* Update integration.yml

* Update integration.yml

* Update integration.yml
@mgravell

Copy link
Copy Markdown
Collaborator

@uglide what's the status of this one; still in progress? do you want me to take it over? or...?

@uglide

uglide commented Dec 5, 2025

Copy link
Copy Markdown
Contributor Author

@mgravell This PR is blocked until my PR to RediSearch will be backported to v8.0 and released as Redis v8.0.x and Redis v8.2.x

- 8.0 is for security fixes now, so the "current" version should be switched to 8.2
- 8.4 has stable release now
@uglide uglide marked this pull request as ready for review December 5, 2025 15:28
@uglide

uglide commented Dec 5, 2025

Copy link
Copy Markdown
Contributor Author

run integration tests

@uglide

uglide commented Dec 5, 2025

Copy link
Copy Markdown
Contributor Author
Testcase Errors Failures Skipped Total
NRedisStack.Tests.dll 0 10 277 821

---- Details for maintainers

@uglide

uglide commented Dec 5, 2025

Copy link
Copy Markdown
Contributor Author

run integration tests

@uglide

uglide commented Dec 5, 2025

Copy link
Copy Markdown
Contributor Author
Testcase Errors Failures Skipped Total
NRedisStack.Tests.dll 0 1 277 821

---- Details for maintainers

@uglide

uglide commented Dec 5, 2025

Copy link
Copy Markdown
Contributor Author
Testcase Errors Failures Skipped Total
NRedisStack.Tests.dll 0 0 277 821

---- Details for maintainers

@uglide

uglide commented Dec 8, 2025

Copy link
Copy Markdown
Contributor Author

run integration tests

@mgravell

mgravell commented Dec 8, 2025

Copy link
Copy Markdown
Collaborator

Re-running, but: one failure - looks unreliable / transient (I've seen this flip between pass/fail)

[xUnit.net 00:00:38.71] NRedisStack.Tests.Search.SearchTests.TestDocumentLoadWithDB_Issue352(endpointId: "standalone") [FAIL]
[xUnit.net 00:00:38.71] Assert.Equal() Failure: Values differ
[xUnit.net 00:00:38.72] Expected: 3
[xUnit.net 00:00:38.72] Actual: 1
[xUnit.net 00:00:38.72] Stack Trace:

@uglide

uglide commented Dec 8, 2025

Copy link
Copy Markdown
Contributor Author
Testcase Errors Failures Skipped Total
NRedisStack.Tests.dll 0 1 205 821

---- Details for maintainers

@uglide

uglide commented Dec 8, 2025

Copy link
Copy Markdown
Contributor Author
Testcase Errors Failures Skipped Total
NRedisStack.Tests.dll 0 0 205 821

---- Details for maintainers

@mgravell mgravell merged commit 1ca7020 into master Dec 8, 2025
32 of 33 checks passed
@mgravell mgravell deleted the im/align-tests-with-re branch December 8, 2025 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants