adobe / adobe/aem-testing-clients

Wrong delay in TemporaryContentAuthorGroup class

Open
#82 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
57
Forks
39
PR merge metrics
No merged PRs in 30d

Description

The timeout is 1000000 milliseconds instead of 1000.
p.poll(DEFAULT_TIMEOUT, SECONDS.toMillis(DEFAULT_RETRY_DELAY));
DEFAULT_RETRY_DELAY equals to 1000 and the result is million milliseconds (~16 minutes), which does not seem logical. The code can be replaced with:
p.poll(DEFAULT_TIMEOUT, DEFAULT_RETRY_DELAY);
or
p.poll(DEFAULT_TIMEOUT, SECONDS.toMillis(1));

Contributor guide

Open the contributing guide

Research direction

Find the TemporaryContentAuthorGroup class and inspect the p.poll call that uses DEFAULT_TIMEOUT and DEFAULT_RETRY_DELAY. Correct the millisecond conversion so the retry delay is 1000 milliseconds rather than 1000000, then run the relevant project tests to confirm the timeout behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
testing-qa
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.