Skip to content

client: add action wait time#7322

Open
psiinon wants to merge 1 commit intozaproxy:mainfrom
psiinon:client/action-wait
Open

client: add action wait time#7322
psiinon wants to merge 1 commit intozaproxy:mainfrom
psiinon:client/action-wait

Conversation

@psiinon
Copy link
Copy Markdown
Member

@psiinon psiinon commented Apr 30, 2026

Will help with both PTK and slow running sites.

Signed-off-by: Simon Bennetts <psiinon@gmail.com>
@psiinon
Copy link
Copy Markdown
Member Author

psiinon commented Apr 30, 2026

Logo
Checkmarx One – Scan Summary & Details0d42522f-aa17-49cf-890b-20347b05660a

Great job! No new security vulnerabilities introduced in this pull request


Use @Checkmarx to interact with Checkmarx PR Assistant.
Examples:
@Checkmarx how are you able to help me?
@Checkmarx rescan this PR

action.run(wd);
if (actionWaitTimeInSecs > 0) {
try {
Thread.sleep(actionWaitTimeInSecs * 1000L);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

TimeUnit.SECONDS.toMillis


// Then
assertThat(task.getStatus(), is(Status.FINISHED));
assertThat(timestamps.size(), is(2));
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hasSize

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants