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.

Sign up for GitHub

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

Jump to bottom

test(connector-iroha2): fix flaky generate-and-send-signed-transaction #2370

Closed
petermetz opened this issue Apr 4, 2023 · 0 comments · Fixed by #2381
Closed

test(connector-iroha2): fix flaky generate-and-send-signed-transaction #2370

petermetz opened this issue Apr 4, 2023 · 0 comments · Fixed by #2381
Assignees
@petermetz
Labels
bug Something isn't working Flaky-Test-Automation Issues related to test stability (which is a long running issue that can never fully be solved) Hacktoberfest Hacktoberfest participants are welcome to take a stab at issues marked with this label. Iroha Bugs/features related to the Iroha ledger connector plugin P3 Priority 3: Medium Tests Anything related to tests be that automatic or manual, integration or unit, etc.

Comments

@petermetz
Copy link
Contributor

Describe the bug

yarn jest packages/cactus-plugin-ledger-connector-iroha2/src/test/typescript/integration/iroha2-generate-and-send-signed-transaction.test.ts
succeeds on my local host, but occasionally fails on the CI.

To Reproduce

Run it enough times that the flake magically reappears.

Expected behavior

Test case should be consistently passing in different environments.

Logs/Stack traces

2023-04-04T19-23-07-connector-iroha2-ci-test-crash.log

Cloud provider or hardware configuration:

BuildJet runners

Operating system name, version, build:

Ubuntu-20.04

Hyperledger Cactus release version or commit (git rev-parse --short HEAD):

main

Hyperledger Cactus Plugins/Connectors Used

Iroha 2
@petermetz petermetz added bug Something isn't working Hacktoberfest Hacktoberfest participants are welcome to take a stab at issues marked with this label. Iroha Bugs/features related to the Iroha ledger connector plugin Flaky-Test-Automation Issues related to test stability (which is a long running issue that can never fully be solved) Tests Anything related to tests be that automatic or manual, integration or unit, etc. P3 Priority 3: Medium labels Apr 4, 2023
@petermetz petermetz self-assigned this Apr 4, 2023
outSH added a commit to outSH/cactus that referenced this issue Apr 7, 2023
@outSH
fix(iroha2-connector): fix flaky tests
ce04851 
- Use waitForCommit for all transactions used in test suites to prevent spurious CI test failures.

Closes: hyperledger-cacti#2370
Closes: hyperledger-cacti#2373

Signed-off-by: Michal Bajer <michal.bajer@fujitsu.com>
@outSH outSH mentioned this issue Apr 7, 2023
Merged
outSH added a commit to outSH/cactus that referenced this issue Apr 11, 2023
@outSH
fix(iroha2-connector): fix flaky tests to solve hyperledger-cacti#2370 …
f433ad9 
…and hyperledger-cacti#2373

- Use waitForCommit for all transactions used in test suites to prevent spurious CI test failures.

Closes: hyperledger-cacti#2370
Closes: hyperledger-cacti#2373

Signed-off-by: Michal Bajer <michal.bajer@fujitsu.com>
ryjones pushed a commit to outSH/cactus that referenced this issue Apr 11, 2023
@outSH @ryjones
fix(iroha2-connector): fix flaky tests to solve hyperledger-cacti#2370 …
d774951 
…and hyperledger-cacti#2373

- Use waitForCommit for all transactions used in test suites to prevent spurious CI test failures.

Closes: hyperledger-cacti#2370
Closes: hyperledger-cacti#2373

Signed-off-by: Michal Bajer <michal.bajer@fujitsu.com>
petermetz pushed a commit to outSH/cactus that referenced this issue Apr 11, 2023
@outSH @petermetz
fix(iroha2-connector): fix flaky tests to solve hyperledger-cacti#2370 …
4e0a574 
…and hyperledger-cacti#2373

- Use waitForCommit for all transactions used in test suites to prevent spurious CI test failures.

Closes: hyperledger-cacti#2370
Closes: hyperledger-cacti#2373

Signed-off-by: Michal Bajer <michal.bajer@fujitsu.com>
sandeepnRES pushed a commit to outSH/cactus that referenced this issue Apr 26, 2023
@outSH @sandeepnRES
fix(iroha2-connector): fix flaky tests to solve hyperledger-cacti#2370 …
9aa8c83 
…and hyperledger-cacti#2373

- Use waitForCommit for all transactions used in test suites to prevent spurious CI test failures.

Closes: hyperledger-cacti#2370
Closes: hyperledger-cacti#2373

Signed-off-by: Michal Bajer <michal.bajer@fujitsu.com>
petermetz pushed a commit to outSH/cactus that referenced this issue May 3, 2023
@outSH @petermetz
fix(iroha2-connector): fix flaky tests to solve hyperledger-cacti#2370 …
0e16601 
…and hyperledger-cacti#2373

- Use waitForCommit for all transactions used in test suites to prevent spurious CI test failures.

Closes: hyperledger-cacti#2370
Closes: hyperledger-cacti#2373

Signed-off-by: Michal Bajer <michal.bajer@fujitsu.com>
petermetz pushed a commit that referenced this issue May 4, 2023
@outSH @petermetz
fix(iroha2-connector): fix flaky tests to solve #2370 and #2373
669b51f 
- Use waitForCommit for all transactions used in test suites to prevent spurious CI test failures.

Closes: #2370
Closes: #2373

Signed-off-by: Michal Bajer <michal.bajer@fujitsu.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@petermetz petermetz

Labels
bug Something isn't working Flaky-Test-Automation Issues related to test stability (which is a long running issue that can never fully be solved) Hacktoberfest Hacktoberfest participants are welcome to take a stab at issues marked with this label. Iroha Bugs/features related to the Iroha ledger connector plugin P3 Priority 3: Medium Tests Anything related to tests be that automatic or manual, integration or unit, etc.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(iroha2-connector): fix flaky tests to solve #2370 and #2373 outSH/cactus
1 participant
@petermetz