Commit e2a19ce0 by Ryan Baxter

Merge remote-tracking branch 'origin/master' into 2.0.x

parents 3b4ed6c0 ab9b5ba3
...@@ -64,6 +64,7 @@ import org.springframework.web.multipart.MultipartFile; ...@@ -64,6 +64,7 @@ import org.springframework.web.multipart.MultipartFile;
import static java.nio.charset.Charset.defaultCharset; import static java.nio.charset.Charset.defaultCharset;
import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertEquals;
import static org.springframework.cloud.netflix.zuul.filters.support.FilterConstants.PRE_TYPE;
import static org.springframework.util.StreamUtils.copyToString; import static org.springframework.util.StreamUtils.copyToString;
import lombok.extern.slf4j.Slf4j; import lombok.extern.slf4j.Slf4j;
...@@ -272,7 +273,7 @@ class FormZuulProxyApplication { ...@@ -272,7 +273,7 @@ class FormZuulProxyApplication {
@Override @Override
public String filterType() { public String filterType() {
return "pre"; return PRE_TYPE;
} }
@Override @Override
......
...@@ -62,6 +62,7 @@ import org.springframework.web.bind.annotation.RestController; ...@@ -62,6 +62,7 @@ import org.springframework.web.bind.annotation.RestController;
import org.springframework.web.multipart.MultipartFile; import org.springframework.web.multipart.MultipartFile;
import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertEquals;
import static org.springframework.cloud.netflix.zuul.filters.support.FilterConstants.PRE_TYPE;
import lombok.extern.slf4j.Slf4j; import lombok.extern.slf4j.Slf4j;
...@@ -182,7 +183,7 @@ class FormZuulServletProxyApplication { ...@@ -182,7 +183,7 @@ class FormZuulServletProxyApplication {
@Override @Override
public String filterType() { public String filterType() {
return "pre"; return PRE_TYPE;
} }
@Override @Override
......
...@@ -54,6 +54,7 @@ import org.springframework.web.bind.annotation.RequestMethod; ...@@ -54,6 +54,7 @@ import org.springframework.web.bind.annotation.RequestMethod;
import org.springframework.web.bind.annotation.RestController; import org.springframework.web.bind.annotation.RestController;
import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertEquals;
import static org.springframework.cloud.netflix.zuul.filters.support.FilterConstants.PRE_TYPE;
@RunWith(SpringJUnit4ClassRunner.class) @RunWith(SpringJUnit4ClassRunner.class)
@SpringBootTest(classes = RetryableZuulProxyApplication.class, webEnvironment = WebEnvironment.RANDOM_PORT, value = { @SpringBootTest(classes = RetryableZuulProxyApplication.class, webEnvironment = WebEnvironment.RANDOM_PORT, value = {
...@@ -117,7 +118,7 @@ class RetryableZuulProxyApplication { ...@@ -117,7 +118,7 @@ class RetryableZuulProxyApplication {
return new ZuulFilter() { return new ZuulFilter() {
@Override @Override
public String filterType() { public String filterType() {
return "pre"; return PRE_TYPE;
} }
@Override @Override
......
...@@ -46,6 +46,7 @@ import org.springframework.web.bind.annotation.RestController; ...@@ -46,6 +46,7 @@ import org.springframework.web.bind.annotation.RestController;
import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNotNull; import static org.junit.Assert.assertNotNull;
import static org.springframework.cloud.netflix.zuul.filters.support.FilterConstants.PRE_TYPE;
@RunWith(SpringJUnit4ClassRunner.class) @RunWith(SpringJUnit4ClassRunner.class)
@SpringBootTest(classes = SimpleZuulServerApplication.class, webEnvironment = WebEnvironment.RANDOM_PORT) @SpringBootTest(classes = SimpleZuulServerApplication.class, webEnvironment = WebEnvironment.RANDOM_PORT)
...@@ -120,7 +121,7 @@ class SimpleZuulServerApplication { ...@@ -120,7 +121,7 @@ class SimpleZuulServerApplication {
return new ZuulFilter() { return new ZuulFilter() {
@Override @Override
public String filterType() { public String filterType() {
return "pre"; return PRE_TYPE;
} }
@Override @Override
......
...@@ -47,6 +47,9 @@ import org.springframework.web.bind.annotation.RestController; ...@@ -47,6 +47,9 @@ import org.springframework.web.bind.annotation.RestController;
import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThat;
import static org.springframework.boot.test.context.SpringBootTest.WebEnvironment.RANDOM_PORT; import static org.springframework.boot.test.context.SpringBootTest.WebEnvironment.RANDOM_PORT;
import static org.springframework.cloud.netflix.zuul.filters.support.FilterConstants.POST_TYPE;
import static org.springframework.cloud.netflix.zuul.filters.support.FilterConstants.PRE_TYPE;
import static org.springframework.cloud.netflix.zuul.filters.support.FilterConstants.ROUTE_TYPE;
/** /**
* @author Spencer Gibb * @author Spencer Gibb
...@@ -111,7 +114,7 @@ public class SendErrorFilterIntegrationTests { ...@@ -111,7 +114,7 @@ public class SendErrorFilterIntegrationTests {
return new FailureFilter() { return new FailureFilter() {
@Override @Override
public String filterType() { public String filterType() {
return "pre"; return PRE_TYPE;
} }
}; };
} }
...@@ -121,7 +124,7 @@ public class SendErrorFilterIntegrationTests { ...@@ -121,7 +124,7 @@ public class SendErrorFilterIntegrationTests {
return new FailureFilter() { return new FailureFilter() {
@Override @Override
public String filterType() { public String filterType() {
return "route"; return ROUTE_TYPE;
} }
}; };
} }
...@@ -131,7 +134,7 @@ public class SendErrorFilterIntegrationTests { ...@@ -131,7 +134,7 @@ public class SendErrorFilterIntegrationTests {
return new FailureFilter() { return new FailureFilter() {
@Override @Override
public String filterType() { public String filterType() {
return "post"; return POST_TYPE;
} }
}; };
} }
......
...@@ -42,6 +42,7 @@ import static org.junit.Assert.assertFalse; ...@@ -42,6 +42,7 @@ import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertTrue; import static org.junit.Assert.assertTrue;
import static org.mockito.MockitoAnnotations.initMocks; import static org.mockito.MockitoAnnotations.initMocks;
import static org.springframework.cloud.netflix.zuul.filters.support.FilterConstants.FORWARD_TO_KEY; import static org.springframework.cloud.netflix.zuul.filters.support.FilterConstants.FORWARD_TO_KEY;
import static org.springframework.cloud.netflix.zuul.filters.support.FilterConstants.PRE_TYPE;
import static org.springframework.cloud.netflix.zuul.filters.support.FilterConstants.REQUEST_URI_KEY; import static org.springframework.cloud.netflix.zuul.filters.support.FilterConstants.REQUEST_URI_KEY;
import static org.springframework.cloud.netflix.zuul.filters.support.FilterConstants.SERVICE_ID_KEY; import static org.springframework.cloud.netflix.zuul.filters.support.FilterConstants.SERVICE_ID_KEY;
...@@ -85,7 +86,7 @@ public class PreDecorationFilterTests { ...@@ -85,7 +86,7 @@ public class PreDecorationFilterTests {
public void basicProperties() throws Exception { public void basicProperties() throws Exception {
assertEquals(5, this.filter.filterOrder()); assertEquals(5, this.filter.filterOrder());
assertEquals(true, this.filter.shouldFilter()); assertEquals(true, this.filter.shouldFilter());
assertEquals("pre", this.filter.filterType()); assertEquals(PRE_TYPE, this.filter.filterType());
} }
@Test @Test
......
...@@ -82,6 +82,7 @@ import static org.hamcrest.Matchers.is; ...@@ -82,6 +82,7 @@ import static org.hamcrest.Matchers.is;
import static org.junit.Assert.assertEquals; import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse; import static org.junit.Assert.assertFalse;
import static org.junit.Assume.assumeThat; import static org.junit.Assume.assumeThat;
import static org.springframework.cloud.netflix.zuul.filters.support.FilterConstants.PRE_TYPE;
/** /**
* @author Spencer Gibb * @author Spencer Gibb
...@@ -406,7 +407,7 @@ public abstract class ZuulProxyTestBase { ...@@ -406,7 +407,7 @@ public abstract class ZuulProxyTestBase {
return new ZuulFilter() { return new ZuulFilter() {
@Override @Override
public String filterType() { public String filterType() {
return "pre"; return PRE_TYPE;
} }
@Override @Override
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment