Remove unused ExpectedExceptions

Closes gh-10101
pull/10101/merge
Johnny Lim 7 years ago committed by Stephane Nicoll
parent 3ef3b40783
commit 118f65556f

@ -17,9 +17,7 @@
package org.springframework.boot.actuate.autoconfigure.endpoint; package org.springframework.boot.actuate.autoconfigure.endpoint;
import org.junit.Before; import org.junit.Before;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.springframework.boot.actuate.autoconfigure.web.ManagementServerProperties; import org.springframework.boot.actuate.autoconfigure.web.ManagementServerProperties;
@ -34,9 +32,6 @@ public class ManagementEndpointPathResolverTests {
private ManagementEndpointPathResolver resolver; private ManagementEndpointPathResolver resolver;
@Rule
public ExpectedException thrown = ExpectedException.none();
@Before @Before
public void setUp() throws Exception { public void setUp() throws Exception {
ManagementServerProperties properties = new ManagementServerProperties(); ManagementServerProperties properties = new ManagementServerProperties();

@ -44,9 +44,7 @@ import org.ehcache.jsr107.EhcacheCachingProvider;
import org.infinispan.configuration.cache.ConfigurationBuilder; import org.infinispan.configuration.cache.ConfigurationBuilder;
import org.infinispan.jcache.embedded.JCachingProvider; import org.infinispan.jcache.embedded.JCachingProvider;
import org.infinispan.spring.provider.SpringEmbeddedCacheManager; import org.infinispan.spring.provider.SpringEmbeddedCacheManager;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.junit.runner.RunWith; import org.junit.runner.RunWith;
import org.springframework.beans.DirectFieldAccessor; import org.springframework.beans.DirectFieldAccessor;
@ -97,9 +95,6 @@ import static org.mockito.Mockito.verify;
@ClassPathExclusions("hazelcast-client-*.jar") @ClassPathExclusions("hazelcast-client-*.jar")
public class CacheAutoConfigurationTests { public class CacheAutoConfigurationTests {
@Rule
public final ExpectedException thrown = ExpectedException.none();
private final ApplicationContextRunner contextRunner = new ApplicationContextRunner() private final ApplicationContextRunner contextRunner = new ApplicationContextRunner()
.withConfiguration(AutoConfigurations.of(CacheAutoConfiguration.class)); .withConfiguration(AutoConfigurations.of(CacheAutoConfiguration.class));

@ -22,9 +22,7 @@ import com.couchbase.client.java.CouchbaseBucket;
import com.couchbase.client.java.cluster.ClusterInfo; import com.couchbase.client.java.cluster.ClusterInfo;
import com.couchbase.client.java.env.CouchbaseEnvironment; import com.couchbase.client.java.env.CouchbaseEnvironment;
import com.couchbase.client.java.env.DefaultCouchbaseEnvironment; import com.couchbase.client.java.env.DefaultCouchbaseEnvironment;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.springframework.boot.autoconfigure.couchbase.CouchbaseAutoConfiguration.CouchbaseConfiguration; import org.springframework.boot.autoconfigure.couchbase.CouchbaseAutoConfiguration.CouchbaseConfiguration;
import org.springframework.boot.autoconfigure.data.couchbase.CouchbaseDataAutoConfiguration; import org.springframework.boot.autoconfigure.data.couchbase.CouchbaseDataAutoConfiguration;
@ -35,7 +33,7 @@ import static org.assertj.core.api.Assertions.assertThat;
import static org.mockito.Mockito.mock; import static org.mockito.Mockito.mock;
/** /**
* Tests for {@link CouchbaseAutoConfiguration} * Tests for {@link CouchbaseAutoConfiguration}.
* *
* @author Eddú Meléndez * @author Eddú Meléndez
* @author Stephane Nicoll * @author Stephane Nicoll
@ -43,9 +41,6 @@ import static org.mockito.Mockito.mock;
public class CouchbaseAutoConfigurationTests public class CouchbaseAutoConfigurationTests
extends AbstractCouchbaseAutoConfigurationTests { extends AbstractCouchbaseAutoConfigurationTests {
@Rule
public ExpectedException thrown = ExpectedException.none();
@Test @Test
public void bootstrapHostsIsRequired() { public void bootstrapHostsIsRequired() {
load(null); load(null);

@ -22,9 +22,7 @@ import org.elasticsearch.client.Client;
import org.elasticsearch.client.transport.TransportClient; import org.elasticsearch.client.transport.TransportClient;
import org.elasticsearch.cluster.node.DiscoveryNode; import org.elasticsearch.cluster.node.DiscoveryNode;
import org.junit.After; import org.junit.After;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.springframework.boot.autoconfigure.context.PropertyPlaceholderAutoConfiguration; import org.springframework.boot.autoconfigure.context.PropertyPlaceholderAutoConfiguration;
import org.springframework.boot.test.util.TestPropertyValues; import org.springframework.boot.test.util.TestPropertyValues;
@ -43,9 +41,6 @@ import static org.mockito.Mockito.mock;
*/ */
public class ElasticsearchAutoConfigurationTests { public class ElasticsearchAutoConfigurationTests {
@Rule
public ExpectedException thrown = ExpectedException.none();
private AnnotationConfigApplicationContext context; private AnnotationConfigApplicationContext context;
@After @After

@ -22,9 +22,7 @@ import java.util.Set;
import com.mongodb.Mongo; import com.mongodb.Mongo;
import org.junit.After; import org.junit.After;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.springframework.beans.factory.BeanCreationException; import org.springframework.beans.factory.BeanCreationException;
import org.springframework.boot.autoconfigure.AutoConfigurationPackages; import org.springframework.boot.autoconfigure.AutoConfigurationPackages;
@ -60,9 +58,6 @@ import static org.junit.Assert.fail;
*/ */
public class MongoDataAutoConfigurationTests { public class MongoDataAutoConfigurationTests {
@Rule
public final ExpectedException thrown = ExpectedException.none();
private AnnotationConfigApplicationContext context; private AnnotationConfigApplicationContext context;
@After @After

@ -21,9 +21,7 @@ import java.util.Collection;
import java.util.Iterator; import java.util.Iterator;
import java.util.List; import java.util.List;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.springframework.http.converter.ByteArrayHttpMessageConverter; import org.springframework.http.converter.ByteArrayHttpMessageConverter;
import org.springframework.http.converter.HttpMessageConverter; import org.springframework.http.converter.HttpMessageConverter;
@ -48,9 +46,6 @@ import static org.mockito.Mockito.mock;
*/ */
public class HttpMessageConvertersTests { public class HttpMessageConvertersTests {
@Rule
public ExpectedException thrown = ExpectedException.none();
@Test @Test
public void containsDefaults() throws Exception { public void containsDefaults() throws Exception {
HttpMessageConverters converters = new HttpMessageConverters(); HttpMessageConverters converters = new HttpMessageConverters();

@ -17,9 +17,7 @@
package org.springframework.boot.autoconfigure.jmx; package org.springframework.boot.autoconfigure.jmx;
import org.junit.After; import org.junit.After;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.springframework.beans.DirectFieldAccessor; import org.springframework.beans.DirectFieldAccessor;
import org.springframework.beans.factory.NoSuchBeanDefinitionException; import org.springframework.beans.factory.NoSuchBeanDefinitionException;
@ -41,15 +39,12 @@ import org.springframework.test.util.ReflectionTestUtils;
import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThat;
/** /**
* Tests for {@link JmxAutoConfiguration} * Tests for {@link JmxAutoConfiguration}.
* *
* @author Christian Dupuis * @author Christian Dupuis
*/ */
public class JmxAutoConfigurationTests { public class JmxAutoConfigurationTests {
@Rule
public ExpectedException thrown = ExpectedException.none();
private AnnotationConfigApplicationContext context; private AnnotationConfigApplicationContext context;
@After @After

@ -18,9 +18,7 @@ package org.springframework.boot.autoconfigure.mongo;
import com.mongodb.MongoClient; import com.mongodb.MongoClient;
import com.mongodb.MongoClientOptions; import com.mongodb.MongoClientOptions;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.springframework.boot.context.properties.EnableConfigurationProperties; import org.springframework.boot.context.properties.EnableConfigurationProperties;
import org.springframework.boot.test.util.TestPropertyValues; import org.springframework.boot.test.util.TestPropertyValues;
@ -39,9 +37,6 @@ import static org.assertj.core.api.Assertions.assertThat;
*/ */
public class MongoPropertiesTests { public class MongoPropertiesTests {
@Rule
public ExpectedException thrown = ExpectedException.none();
@Test @Test
public void canBindCharArrayPassword() { public void canBindCharArrayPassword() {
// gh-1572 // gh-1572

@ -22,9 +22,7 @@ import java.util.List;
import javax.validation.ValidatorFactory; import javax.validation.ValidatorFactory;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.springframework.beans.DirectFieldAccessor; import org.springframework.beans.DirectFieldAccessor;
import org.springframework.boot.autoconfigure.validation.ValidationAutoConfiguration; import org.springframework.boot.autoconfigure.validation.ValidationAutoConfiguration;
@ -74,9 +72,6 @@ import static org.mockito.Mockito.verify;
*/ */
public class WebFluxAutoConfigurationTests { public class WebFluxAutoConfigurationTests {
@Rule
public ExpectedException thrown = ExpectedException.none();
private GenericReactiveWebApplicationContext context; private GenericReactiveWebApplicationContext context;
@Test @Test

@ -21,9 +21,7 @@ import java.net.URI;
import javax.servlet.MultipartConfigElement; import javax.servlet.MultipartConfigElement;
import org.junit.After; import org.junit.After;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.springframework.boot.autoconfigure.web.ServerProperties; import org.springframework.boot.autoconfigure.web.ServerProperties;
import org.springframework.boot.context.properties.EnableConfigurationProperties; import org.springframework.boot.context.properties.EnableConfigurationProperties;
@ -67,9 +65,6 @@ public class MultipartAutoConfigurationTests {
private AnnotationConfigServletWebServerApplicationContext context; private AnnotationConfigServletWebServerApplicationContext context;
@Rule
public ExpectedException thrown = ExpectedException.none();
@After @After
public void close() { public void close() {
if (this.context != null) { if (this.context != null) {

@ -18,9 +18,7 @@ package org.springframework.boot.autoconfigure.webservices;
import java.util.Collection; import java.util.Collection;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.springframework.beans.factory.BeanCreationException; import org.springframework.beans.factory.BeanCreationException;
import org.springframework.boot.autoconfigure.AutoConfigurations; import org.springframework.boot.autoconfigure.AutoConfigurations;
@ -43,9 +41,6 @@ public class WebServicesAutoConfigurationTests {
private final WebApplicationContextRunner contextRunner = new WebApplicationContextRunner() private final WebApplicationContextRunner contextRunner = new WebApplicationContextRunner()
.withConfiguration(AutoConfigurations.of(WebServicesAutoConfiguration.class)); .withConfiguration(AutoConfigurations.of(WebServicesAutoConfiguration.class));
@Rule
public ExpectedException thrown = ExpectedException.none();
@Test @Test
public void defaultConfiguration() { public void defaultConfiguration() {
this.contextRunner.run((context) -> assertThat(context) this.contextRunner.run((context) -> assertThat(context)

@ -29,9 +29,7 @@ import org.eclipse.aether.repository.LocalRepository;
import org.eclipse.aether.repository.Proxy; import org.eclipse.aether.repository.Proxy;
import org.eclipse.aether.repository.RemoteRepository; import org.eclipse.aether.repository.RemoteRepository;
import org.junit.Before; import org.junit.Before;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.mockito.Mock; import org.mockito.Mock;
import org.mockito.MockitoAnnotations; import org.mockito.MockitoAnnotations;
@ -49,9 +47,6 @@ import static org.mockito.BDDMockito.given;
*/ */
public class SettingsXmlRepositorySystemSessionAutoConfigurationTests { public class SettingsXmlRepositorySystemSessionAutoConfigurationTests {
@Rule
public ExpectedException thrown = ExpectedException.none();
@Mock @Mock
private RepositorySystem repositorySystem; private RepositorySystem repositorySystem;

@ -16,9 +16,7 @@
package org.springframework.boot.test.autoconfigure.data.mongo; package org.springframework.boot.test.autoconfigure.data.mongo;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.junit.runner.RunWith; import org.junit.runner.RunWith;
import org.springframework.beans.factory.annotation.Autowired; import org.springframework.beans.factory.annotation.Autowired;
@ -36,9 +34,6 @@ import static org.assertj.core.api.Assertions.assertThat;
@DataMongoTest @DataMongoTest
public class DataMongoTestReactiveIntegrationTests { public class DataMongoTestReactiveIntegrationTests {
@Rule
public ExpectedException thrown = ExpectedException.none();
@Autowired @Autowired
private ReactiveMongoTemplate mongoTemplate; private ReactiveMongoTemplate mongoTemplate;

@ -24,7 +24,6 @@ import java.util.Properties;
import org.junit.Before; import org.junit.Before;
import org.junit.Rule; import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.junit.rules.TemporaryFolder; import org.junit.rules.TemporaryFolder;
import org.springframework.boot.testsupport.compiler.TestCompiler; import org.springframework.boot.testsupport.compiler.TestCompiler;
@ -41,9 +40,6 @@ public class AutoConfigureAnnotationProcessorTests {
@Rule @Rule
public TemporaryFolder temporaryFolder = new TemporaryFolder(); public TemporaryFolder temporaryFolder = new TemporaryFolder();
@Rule
public ExpectedException thrown = ExpectedException.none();
private TestCompiler compiler; private TestCompiler compiler;
@Before @Before

@ -23,7 +23,6 @@ import java.net.URL;
import org.junit.Before; import org.junit.Before;
import org.junit.Rule; import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.junit.rules.TemporaryFolder; import org.junit.rules.TemporaryFolder;
import org.springframework.boot.loader.TestJarCreator; import org.springframework.boot.loader.TestJarCreator;
@ -42,9 +41,6 @@ public class JarURLConnectionTests {
@Rule @Rule
public TemporaryFolder temporaryFolder = new TemporaryFolder(new File("target")); public TemporaryFolder temporaryFolder = new TemporaryFolder(new File("target"));
@Rule
public ExpectedException thrown = ExpectedException.none();
private File rootJarFile; private File rootJarFile;
private JarFile jarFile; private JarFile jarFile;

@ -17,9 +17,7 @@
package org.springframework.boot.context.config; package org.springframework.boot.context.config;
import org.junit.After; import org.junit.After;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.springframework.boot.SpringApplication; import org.springframework.boot.SpringApplication;
import org.springframework.boot.context.event.ApplicationEnvironmentPreparedEvent; import org.springframework.boot.context.event.ApplicationEnvironmentPreparedEvent;
@ -40,9 +38,6 @@ import static org.assertj.core.api.Assertions.assertThat;
*/ */
public class DelegatingApplicationListenerTests { public class DelegatingApplicationListenerTests {
@Rule
public ExpectedException thrown = ExpectedException.none();
private final DelegatingApplicationListener listener = new DelegatingApplicationListener(); private final DelegatingApplicationListener listener = new DelegatingApplicationListener();
private final StaticApplicationContext context = new StaticApplicationContext(); private final StaticApplicationContext context = new StaticApplicationContext();

@ -18,9 +18,7 @@ package org.springframework.boot.context.properties.bind.convert;
import java.net.InetAddress; import java.net.InetAddress;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThat;
@ -31,9 +29,6 @@ import static org.assertj.core.api.Assertions.assertThat;
*/ */
public class InetAddressToStringConverterTests extends AbstractInetAddressTests { public class InetAddressToStringConverterTests extends AbstractInetAddressTests {
@Rule
public ExpectedException thrown = ExpectedException.none();
private InetAddressToStringConverter converter = new InetAddressToStringConverter(); private InetAddressToStringConverter converter = new InetAddressToStringConverter();
@Test @Test

@ -19,9 +19,7 @@ package org.springframework.boot.context.properties.bind.handler;
import java.util.ArrayList; import java.util.ArrayList;
import java.util.List; import java.util.List;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.springframework.boot.context.properties.bind.BindException; import org.springframework.boot.context.properties.bind.BindException;
import org.springframework.boot.context.properties.bind.BindHandler; import org.springframework.boot.context.properties.bind.BindHandler;
@ -42,9 +40,6 @@ import static org.assertj.core.api.Assertions.fail;
*/ */
public class NoUnboundElementsBindHandlerTests { public class NoUnboundElementsBindHandlerTests {
@Rule
public ExpectedException thrown = ExpectedException.none();
private List<ConfigurationPropertySource> sources = new ArrayList<>(); private List<ConfigurationPropertySource> sources = new ArrayList<>();
private Binder binder; private Binder binder;

@ -20,9 +20,7 @@ import java.util.ArrayList;
import java.util.List; import java.util.List;
import org.junit.Before; import org.junit.Before;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.springframework.boot.context.properties.bind.Bindable; import org.springframework.boot.context.properties.bind.Bindable;
import org.springframework.boot.context.properties.bind.Binder; import org.springframework.boot.context.properties.bind.Binder;
@ -39,9 +37,6 @@ import static org.assertj.core.api.Assertions.assertThat;
*/ */
public class PackagePrivateBeanBindingTests { public class PackagePrivateBeanBindingTests {
@Rule
public ExpectedException thrown = ExpectedException.none();
private List<ConfigurationPropertySource> sources = new ArrayList<>(); private List<ConfigurationPropertySource> sources = new ArrayList<>();
private Binder binder; private Binder binder;

@ -20,9 +20,7 @@ import java.util.Collections;
import java.util.LinkedHashMap; import java.util.LinkedHashMap;
import java.util.Map; import java.util.Map;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.springframework.core.env.ConfigurableEnvironment; import org.springframework.core.env.ConfigurableEnvironment;
import org.springframework.core.env.Environment; import org.springframework.core.env.Environment;
@ -43,9 +41,6 @@ import static org.assertj.core.api.Assertions.assertThat;
*/ */
public class ConfigurationPropertySourcesTests { public class ConfigurationPropertySourcesTests {
@Rule
public ExpectedException thrown = ExpectedException.none();
@Test @Test
public void attachShouldAddAdapterAtBeginning() throws Exception { public void attachShouldAddAdapterAtBeginning() throws Exception {
ConfigurableEnvironment environment = new StandardEnvironment(); ConfigurableEnvironment environment = new StandardEnvironment();

@ -21,9 +21,7 @@ import java.util.HashMap;
import java.util.List; import java.util.List;
import java.util.Map; import java.util.Map;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThat;
@ -34,9 +32,6 @@ import static org.assertj.core.api.Assertions.assertThat;
*/ */
public class MimeMappingsTests { public class MimeMappingsTests {
@Rule
public ExpectedException thrown = ExpectedException.none();
@Test(expected = UnsupportedOperationException.class) @Test(expected = UnsupportedOperationException.class)
public void defaultsCannotBeModified() throws Exception { public void defaultsCannotBeModified() throws Exception {
MimeMappings.DEFAULT.add("foo", "foo/bar"); MimeMappings.DEFAULT.add("foo", "foo/bar");

Loading…
Cancel
Save