Browse Source

test: Improve exception checking in SentinelAnnotationQuarkusAdapterTest (#2116)

master
Guilherme Amaral GitHub 3 years ago
parent
commit
00a560bd00
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 5 additions and 7 deletions
  1. +5
    -7
      sentinel-adapter/sentinel-quarkus-adapter/sentinel-annotation-quarkus-adapter-deployment/src/test/java/com/alibaba/csp/sentinel/adapter/quarkus/annotation/deployment/SentinelAnnotationQuarkusAdapterTest.java

+ 5
- 7
sentinel-adapter/sentinel-quarkus-adapter/sentinel-annotation-quarkus-adapter-deployment/src/test/java/com/alibaba/csp/sentinel/adapter/quarkus/annotation/deployment/SentinelAnnotationQuarkusAdapterTest.java View File

@@ -36,6 +36,7 @@ import java.util.Collections;


import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThat;
import static org.assertj.core.api.Assertions.fail; import static org.assertj.core.api.Assertions.fail;
import static org.junit.jupiter.api.Assertions.assertThrows;


/** /**
* @author sea * @author sea
@@ -92,7 +93,7 @@ public class SentinelAnnotationQuarkusAdapterTest {
new FlowRule(resourceName).setCount(0) new FlowRule(resourceName).setCount(0)
)); ));


Assertions.assertThrows(ArcUndeclaredThrowableException.class, () -> {
assertThrows(ArcUndeclaredThrowableException.class, () -> {
fooService.baz("Sentinel"); fooService.baz("Sentinel");
}); });
} }
@@ -104,13 +105,10 @@ public class SentinelAnnotationQuarkusAdapterTest {
ClusterNode cn = ClusterBuilderSlot.getClusterNode(resourceName); ClusterNode cn = ClusterBuilderSlot.getClusterNode(resourceName);
assertThat(cn).isNotNull(); assertThat(cn).isNotNull();
assertThat(cn.passQps()).isPositive(); assertThat(cn.passQps()).isPositive();

try {
assertThrows(IllegalMonitorStateException.class, () -> {
fooService.baz("fail"); fooService.baz("fail");
fail("should not reach here");
} catch (IllegalMonitorStateException ex) {
assertThat(cn.exceptionQps()).isZero();
}
});
assertThat(cn.exceptionQps()).isZero();
} }


@Test @Test


Loading…
Cancel
Save