Skip to content

Commit

Permalink
chore: improve test structure
Browse files Browse the repository at this point in the history
Signed-off-by: Ivo Yankov <[email protected]>
  • Loading branch information
Ivo-Yankov committed Sep 30, 2024
1 parent 5bc5f71 commit 4df05b9
Showing 1 changed file with 12 additions and 4 deletions.
16 changes: 12 additions & 4 deletions test/e2e/commands/node_upgrade.test.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -58,19 +58,27 @@ describe('Node upgrade', () => {
expect(status).toBeTruthy()
}, 450000)

it('should upgrade all nodes on the network successfully', async () => {
it('should prepare network upgrade successfully', async () => {
await nodeCmd.prepareUpgrade(upgradeArgv)
}, 300000)

it('should download generated files successfully', async () => {
await nodeCmd.downloadGeneratedFiles(upgradeArgv)
}, 300000)

it('should upgrade all nodes on the network successfully', async () => {
await nodeCmd.freezeUpgrade(upgradeArgv)
}, 300000)


Check failure on line 73 in test/e2e/commands/node_upgrade.test.mjs

View workflow job for this annotation

GitHub Actions / Code Style / Standard

More than 1 blank line not allowed
it('should have accesses only the expected config properties', async () => {
await nodeCmd.accountManager.close()
expect(nodeCmd.getUnusedConfigs(NodeCommand.DELETE_CONFIGS_NAME)).toEqual([
flags.app.constName,
flags.devMode.constName,
flags.endpointType.constName
])

await nodeCmd.accountManager.close()
}, 600000)
})

balanceQueryShouldSucceed(nodeCmd.accountManager, nodeCmd, namespace)

Expand Down

0 comments on commit 4df05b9

Please sign in to comment.