gaspersic/java-design-patterns
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #752 from christophercolumbusdog/master

Browse Source 
#643 Fix flaky tests for throttling pattern
This commit is contained in:
Ilkka Seppälä 2018-05-27 10:59:52 +03:00 committed by GitHub
commit c713dbcbf4
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
throttling/src/test/java/com/iluwatar/throttling/B2BServiceTest.java
View File

@ -22,29 +22,27 @@
*/ */
package com.iluwatar.throttling; package com.iluwatar.throttling;
import com.iluwatar.throttling.timer.ThrottleTimerImpl;
import com.iluwatar.throttling.timer.Throttler; import com.iluwatar.throttling.timer.Throttler;
import org.junit.jupiter.api.Disabled;
import org.junit.jupiter.api.Test; import org.junit.jupiter.api.Test;
import static org.junit.jupiter.api.Assertions.assertTrue; import static org.junit.jupiter.api.Assertions.assertEquals;
/** /**
* B2BServiceTest class to test the B2BService * B2BServiceTest class to test the B2BService
*/ */
public class B2BServiceTest { public class B2BServiceTest {
@Disabled
@Test @Test
public void dummyCustomerApiTest() { public void dummyCustomerApiTest() {
Tenant tenant = new Tenant("testTenant", 2); Tenant tenant = new Tenant("testTenant", 2);
Throttler timer = new ThrottleTimerImpl(100); // In order to assure that throttling limits will not be reset, we use an empty throttling implementation
Throttler timer = () -> { };
B2BService service = new B2BService(timer); B2BService service = new B2BService(timer);
for (int i = 0; i < 5; i++) { for (int i = 0; i < 5; i++) {
service.dummyCustomerApi(tenant); service.dummyCustomerApi(tenant);
} }
long counter = CallsCount.getCount(tenant.getName()); long counter = CallsCount.getCount(tenant.getName());
assertTrue(counter == 2, "Counter limit must be reached"); assertEquals(2, counter, "Counter limit must be reached");
} }
} }