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

test: broke websocket acceptancetest CI task into smaller separate batches (#2382) #2401

Merged

Conversation

quiet-node
Copy link
Member

Description:
Currently there are 300+ total acceptance test cases in the WS server, and they all run within one CI task with one instance of localnode. Due to the enormous size of the CI task, the log wouldn't be able to show all the records, and without seeing the full log it's impossible to trace where the task fails.

This PR attempts to break the one CI task into 3 smaller separate batches.

Related issue(s):

Fixes #2382

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@quiet-node quiet-node self-assigned this Apr 23, 2024
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
10.5% Duplication on New Code

See analysis details on SonarCloud

Copy link

Tests

    2 files  147 suites   13s ⏱️
815 tests 814 ✔️ 1 💤 0
827 runs  826 ✔️ 1 💤 0

Results for commit 753f42a.

Copy link

Acceptance Tests

  14 files  191 suites   16m 18s ⏱️
580 tests 577 ✔️ 3 💤 0
615 runs  612 ✔️ 3 💤 0

Results for commit 753f42a.

@quiet-node quiet-node merged commit 09007c5 into main Apr 23, 2024
32 checks passed
@quiet-node quiet-node deleted the 2382-divide-ws-acceptance-ci-test-into-separate-batches branch April 23, 2024 23:26
quiet-node added a commit that referenced this pull request Apr 24, 2024
…tches (#2382) (#2401)

test: divided ws-server total acceptancetest into batches (#2382)

Signed-off-by: Logan Nguyen <[email protected]>
quiet-node added a commit that referenced this pull request Apr 24, 2024
quiet-node added a commit that referenced this pull request Apr 25, 2024
…tches (#2382) (#2401)

test: divided ws-server total acceptancetest into batches (#2382)

Signed-off-by: Logan Nguyen <[email protected]>
Nana-EC pushed a commit that referenced this pull request Apr 25, 2024
* test: broke websocket acceptancetest CI task into smaller separate batches (#2382) (#2401)

test: divided ws-server total acceptancetest into batches (#2382)

Signed-off-by: Logan Nguyen <[email protected]>

* feat: support deterministic proxy contract deployment transaction (#2287) (#2393)

* feat: supported foundry deterministic deployment proxy contract transaction

Signed-off-by: Logan Nguyen <[email protected]>

* fix: removed Foundry reference

Signed-off-by: Logan Nguyen <[email protected]>

---------

Signed-off-by: Logan Nguyen <[email protected]>

* pkg: bumped hedera-local to 2.23.0

Signed-off-by: Logan Nguyen <[email protected]>

---------

Signed-off-by: Logan Nguyen <[email protected]>
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.

Divide WS acceptance CI test into separate batches
2 participants