Skip to content

Commit

Permalink
Increase time in IntervalThrottlerTests (#120344)
Browse files Browse the repository at this point in the history
We suspect that something in the test environment is causing more than
10ms to pass between some of the calls so increasing to 100 for now to
verify

Closes #120023
  • Loading branch information
limotova authored Jan 18, 2025
1 parent 1e19596 commit 2c3a0f5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
3 changes: 0 additions & 3 deletions muted-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -220,9 +220,6 @@ tests:
- class: org.elasticsearch.xpack.test.rest.XPackRestIT
method: test {p0=transform/transforms_unattended/Test unattended put and start}
issue: https://github.com/elastic/elasticsearch/issues/120019
- class: org.elasticsearch.index.mapper.IntervalThrottlerTests
method: testThrottling
issue: https://github.com/elastic/elasticsearch/issues/120023
- class: org.elasticsearch.xpack.ilm.actions.SearchableSnapshotActionIT
method: testUpdatePolicyToAddPhasesYieldsInvalidActionsToBeSkipped
issue: https://github.com/elastic/elasticsearch/issues/118406
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,12 +14,12 @@
public class IntervalThrottlerTests extends ESTestCase {

public void testThrottling() throws Exception {
var throttler = new IntervalThrottler.Acceptor(10);
var throttler = new IntervalThrottler.Acceptor(100);
assertTrue(throttler.accept());
assertFalse(throttler.accept());
assertFalse(throttler.accept());

Thread.sleep(20);
Thread.sleep(110);
assertTrue(throttler.accept());
assertFalse(throttler.accept());
assertFalse(throttler.accept());
Expand Down

0 comments on commit 2c3a0f5

Please sign in to comment.