Merge branch '1.5.x'
This commit is contained in:
commit
b113ba5e70
|
|
@ -1,5 +1,5 @@
|
||||||
/*
|
/*
|
||||||
* Copyright 2012-2016 the original author or authors.
|
* Copyright 2012-2017 the original author or authors.
|
||||||
*
|
*
|
||||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||||
* you may not use this file except in compliance with the License.
|
* you may not use this file except in compliance with the License.
|
||||||
|
|
@ -334,7 +334,7 @@ public class ModifiedClassPathRunner extends BlockJUnit4ClassRunner {
|
||||||
|
|
||||||
@Override
|
@Override
|
||||||
public Class<?> loadClass(String name) throws ClassNotFoundException {
|
public Class<?> loadClass(String name) throws ClassNotFoundException {
|
||||||
if (name.startsWith("org.junit")) {
|
if (name.startsWith("org.junit") || name.startsWith("org.hamcrest")) {
|
||||||
return this.junitLoader.loadClass(name);
|
return this.junitLoader.loadClass(name);
|
||||||
}
|
}
|
||||||
return super.loadClass(name);
|
return super.loadClass(name);
|
||||||
|
|
|
||||||
|
|
@ -1,5 +1,5 @@
|
||||||
/*
|
/*
|
||||||
* Copyright 2012-2016 the original author or authors.
|
* Copyright 2012-2017 the original author or authors.
|
||||||
*
|
*
|
||||||
* Licensed under the Apache License, Version 2.0 (the "License");
|
* Licensed under the Apache License, Version 2.0 (the "License");
|
||||||
* you may not use this file except in compliance with the License.
|
* you may not use this file except in compliance with the License.
|
||||||
|
|
@ -16,10 +16,13 @@
|
||||||
|
|
||||||
package org.springframework.boot.junit.runner.classpath;
|
package org.springframework.boot.junit.runner.classpath;
|
||||||
|
|
||||||
|
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 static org.assertj.core.api.Assertions.assertThat;
|
import static org.assertj.core.api.Assertions.assertThat;
|
||||||
|
import static org.hamcrest.CoreMatchers.isA;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Tests for {@link ModifiedClassPathRunner} excluding entries from the class path.
|
* Tests for {@link ModifiedClassPathRunner} excluding entries from the class path.
|
||||||
|
|
@ -33,6 +36,9 @@ public class ModifiedClassPathRunnerExclusionsTests {
|
||||||
private static final String EXCLUDED_RESOURCE = "META-INF/services/"
|
private static final String EXCLUDED_RESOURCE = "META-INF/services/"
|
||||||
+ "javax.validation.spi.ValidationProvider";
|
+ "javax.validation.spi.ValidationProvider";
|
||||||
|
|
||||||
|
@Rule
|
||||||
|
public ExpectedException thrown = ExpectedException.none();
|
||||||
|
|
||||||
@Test
|
@Test
|
||||||
public void entriesAreFilteredFromTestClassClassLoader() {
|
public void entriesAreFilteredFromTestClassClassLoader() {
|
||||||
assertThat(getClass().getClassLoader().getResource(EXCLUDED_RESOURCE)).isNull();
|
assertThat(getClass().getClassLoader().getResource(EXCLUDED_RESOURCE)).isNull();
|
||||||
|
|
@ -44,4 +50,10 @@ public class ModifiedClassPathRunnerExclusionsTests {
|
||||||
.getResource(EXCLUDED_RESOURCE)).isNull();
|
.getResource(EXCLUDED_RESOURCE)).isNull();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
public void testsThatUseHamcrestWorkCorrectly() {
|
||||||
|
this.thrown.expect(isA(IllegalStateException.class));
|
||||||
|
throw new IllegalStateException();
|
||||||
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue