Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: Node test "can send transactions" is flaky #3066

Open
ikolomi opened this issue Feb 4, 2025 · 0 comments
Open

CI: Node test "can send transactions" is flaky #3066

ikolomi opened this issue Feb 4, 2025 · 0 comments
Assignees
Labels
bug Something isn't working CI/CD CI/CD related Flaky-tests Flaky-tests node Node.js wrapper testing Everything about testing

Comments

@ikolomi
Copy link
Collaborator

ikolomi commented Feb 4, 2025

Describe the bug

Test failed on one configuration during the full matrix tests:
https://github.com/valkey-io/valkey-glide/actions/runs/13127527442/job/36638493419#step:5:502

● GlideClient › Protocol is RESP2 = 0 › Decoder String = 0 › can send transactions

    Checks failed in transaction response:
    xpendingWithOptions(key9, groupName1, -, +, 10) failed, expected <[["0-2","8c46b672-6422-472e-b284-2c7807a59e4a",0,1]]>, actual <[["0-2","8c46b672-6422-472e-b284-2c7807a59e4a",1,1]]>

      582 |
      583 |     if (failedChecks.length > 0) {
    > 584 |         throw new Error(
          |               ^
      585 |             "Checks failed in transaction response:\n" +
      586 |                 failedChecks.join("\n"),
      587 |         );

      at validateTransactionResponse (tests/TestUtilities.ts:584:15)
      at tests/GlideClient.test.ts:299:56
      at fulfilled (tests/GlideClient.test.ts:8:58)

Expected Behavior

test passes reliably

Current Behavior

test failed on one configuration during the full matrix test

Reproduction Steps

full matrix tests

Possible Solution

No response

Additional Information/Context

No response

Client version used

.

Engine type and version

.

OS

ubuntu

Language

TypeScript

Language Version

.

Cluster information

No response

Logs

No response

Other information

No response

@ikolomi ikolomi added bug Something isn't working CI/CD CI/CD related node Node.js wrapper testing Everything about testing labels Feb 4, 2025
@Yury-Fridlyand Yury-Fridlyand added the Flaky-tests Flaky-tests label Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working CI/CD CI/CD related Flaky-tests Flaky-tests node Node.js wrapper testing Everything about testing
Projects
Status: No status
Development

No branches or pull requests

3 participants