Skip to content

Commit

Permalink
chore(test): wait for server in upgrade test (matter-labs#1784)
Browse files Browse the repository at this point in the history
## What ❔

- Waits more for server to start in upgrade test. 
- Fixes typo in `core/lib/zksync_core/Cargo.toml`

## Why ❔

Test is flaky in its current form

## Checklist

<!-- Check your PR fulfills the following items. -->
<!-- For draft PRs check the boxes as you complete them. -->

- [ ] PR title corresponds to the body of PR (we generate changelog
entries from PRs).
- [ ] Tests for the changes have been added / updated.
- [ ] Documentation comments have been added / updated.
- [ ] Code has been formatted via `zk fmt` and `zk lint`.
- [ ] Spellcheck has been run via `zk spellcheck`.
- [ ] Linkcheck has been run via `zk linkcheck`.
  • Loading branch information
perekopskiy authored Apr 24, 2024
1 parent 1a34c8b commit df54516
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion core/lib/zksync_core/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ jsonrpsee.workspace = true
tempfile.workspace = true
test-casing.workspace = true
test-log.workspace = true
backon.workspsace = true
backon.workspace = true

[build-dependencies]
zksync_protobuf_build.workspace = true
4 changes: 2 additions & 2 deletions core/tests/upgrade-test/tests/upgrade.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -277,14 +277,14 @@ describe('Upgrade test', function () {
step('Execute transactions after simple restart', async () => {
// Stop server.
await utils.exec('pkill zksync_server');
await utils.sleep(5);
await utils.sleep(10);

// Run again.
utils.background(
'cd $ZKSYNC_HOME && zk f cargo run --bin zksync_server --release -- --components=api,tree,eth,state_keeper,commitment_generator &> upgrade.log',
[null, logs, logs]
);
await utils.sleep(5);
await utils.sleep(10);

// Trying to send a transaction from the same address again
await checkedRandomTransfer(alice, BigNumber.from(1));
Expand Down

0 comments on commit df54516

Please sign in to comment.