Соглашение с обработкой ошибок Джерси

Я создал эту потрясающую точку контроллера:

@Named
@Path("/awsome")
@Api(value = "An Awsome api")
public class AwsomeEndpoint extends BaseEndpoint {

  @GET
  @Path("{isThisAwsome}")
  @Produces(MediaType.APPLICATION_JSON)
  @ApiOperation(value = "Get organisation by isthisAwsome",
                notes = "Get AwsomeAnswer matching the given isThisAwsome.",
                responseContainer = "single result",
                response = JsonOrganisation.class)
  @ApiResponses(value = {@ApiResponse(code = 200, message = "Awsome found"),
                         @ApiResponse(code = 404, message = "Awsome not found"),
                         @ApiResponse(code = 500, message = "Internal Error")})
  public Response getAwsomeResponse(@PathParam("isThisAwsome") String isThisAwsome) {
    Response response = handleErrors(() -> Response.ok(AwsomeResponseTransformer.transform(awsomeService.getAwsomeByisThisAwsome(isThisAwsome))).build());
    return response;
  }
}

Этот код использует Lamda для обработки потока ошибок:

public class BaseEndpoint {

  protected Response handleErrors(Supplier<Response> responseSupplier) {
    Response response;
    try {
      response = responseSupplier.get();
    } catch (AwsomeRuntimeException e) {
      response = createExceptionResponse(e);
    }
    return response;
  }

  private Response createExceptionResponse(AwsomeRuntimeException e) {
    Response.ResponseBuilder response;

    if (e.getExceptionStatus() == ExceptionStatus.NOT_FOUND) {
      response = Response.status(Status.NOT_FOUND).entity(e.getMessage());
    } else if (e.getExceptionStatus() == ExceptionStatus.ILLEGAL_ARGUMENT) {
      response = Response.status(Status.BAD_REQUEST).entity(e.getMessage());
    } else {
      response = Response.status(Status.INTERNAL_SERVER_ERROR).entity(e.getMessage());
    }

    return response.header("","").header("Pragma", "no-cache, no-store").header("Cache-Control", "no-cache, no-store").header("Expires", "0").build();
  }}

Для этого я создал несколько тестов Pact, все они работают, кроме тех случаев, когда мы имитируем ошибку, которую вызывает служба, которую мы вызываем.

Пример тестового класса pact:

RunWith(PactRunner.class)
@Provider("awsome")
@PactBroker(authentication=@PactBrokerAuth(username = "${pact.broker.username:x}", password = "${pact.broker.password:x}"),
            protocol="${pact.broker.protocol:https}", host="${pact.broker.host:hosted.pact.dius.com.au}", port="${pact.broker.port:443}",
            failIfNoPactsFound=false)
public class AccMgtPactIntegrationTest extends JerseyTest {

  @TestTarget public final HttpTarget target = new HttpTarget(getPort());
  private AwsomeService awsomeService;
  private UserService         userService;

  @Override
  protected ResourceConfig configure() {
    awsomeService = mock(AwsomeService.class);
    return new ResourceConfig().register(new OrganisationsEndpoint(awsomeService)).register(new AwsomeEndpoint())
        //                               .property(ServerProperties.RESPONSE_SET_STATUS_OVER_SEND_ERROR, "true")
        ;
  }

  @State("Notn found")
  public void whenOrganisationWithShortNameDoesNotExist() {
    when(awsomeService.getOrganisationByShortName("WME")).thenThrow(new AwsomeRuntimeException(ExceptionStatus.NOT_FOUND));
  } }

Теперь я получаю некоторые изношенные ошибки от Пакта, и я думаю, что проблема не в пакте, а в том, как girlzy работает в пакте. Беку, я получаю следующую ошибку:

 Verifying a pact between consumer and aswsome   Given when
 organisation with shortName does not exist   Get awsome by isItAwsome
     returns a response which
       has status code 404 (OK)
       includes headers
         "Content-Type" with value "application/json" (FAILED)
       has a matching body (FAILED)

Неудачи:

0) Получить организацию по короткому имени возвращает ответ, который включает заголовки «Content-Type» со значением «application / json». Ожидаемый заголовок «Content-Type» должен иметь значение «application / json», но было «text / html; charset = ISO. -8859-1 '

1) Получить организацию по короткому имени возвращает ответ, который имеет соответствующее сравнение тела -> Ожидается тип ответа 'application / json', но фактический тип был 'text / html'

Теперь, если я установлю (закомментированное свойство) RESPONSE_SET_STATUS_OVER_SEND_ERROR

Тогда пакт терпит неудачу вот так:

Verifying a pact between consumer and aswsome
  Given when organisation with shortName does not exist
  Get awsome by isItAwsome:
ul 13, 2017 12:54:14 PM org.glassfish.grizzly.servlet.ServletHandler doServletService
SEVERE: service exception:
javax.servlet.ServletException: javax.servlet.ServletException: java.lang.StringIndexOutOfBoundsException: String index out of range: 0
    at org.glassfish.grizzly.servlet.FilterChainImpl.doFilter(FilterChainImpl.java:151)
    at org.glassfish.grizzly.servlet.FilterChainImpl.invokeFilterChain(FilterChainImpl.java:106)
    at org.glassfish.grizzly.servlet.ServletHandler.doServletService(ServletHandler.java:224)
    at org.glassfish.grizzly.servlet.ServletHandler.service(ServletHandler.java:173)
    at org.glassfish.grizzly.http.server.HttpHandler$1.run(HttpHandler.java:224)
    at org.glassfish.grizzly.threadpool.AbstractThreadPool$Worker.doWork(AbstractThreadPool.java:593)
    at org.glassfish.grizzly.threadpool.AbstractThreadPool$Worker.run(AbstractThreadPool.java:573)
    at java.lang.Thread.run(Thread.java:745)
Caused by: javax.servlet.ServletException: java.lang.StringIndexOutOfBoundsException: String index out of range: 0
    at org.glassfish.jersey.servlet.WebComponent.serviceImpl(WebComponent.java:489)
    at org.glassfish.jersey.servlet.WebComponent.service(WebComponent.java:427)
    at org.glassfish.jersey.servlet.ServletContainer.service(ServletContainer.java:388)
    at org.glassfish.jersey.servlet.ServletContainer.service(ServletContainer.java:341)
    at org.glassfish.jersey.servlet.ServletContainer.service(ServletContainer.java:228)
    at org.glassfish.grizzly.servlet.FilterChainImpl.doFilter(FilterChainImpl.java:147)
    ... 7 more
Caused by: java.lang.StringIndexOutOfBoundsException: String index out of range: 0
    at java.lang.String.charAt(String.java:658)
    at org.glassfish.grizzly.http.HttpHeader.isSpecialHeader(HttpHeader.java:925)
    at org.glassfish.grizzly.http.HttpHeader.handleGetSpecialHeader(HttpHeader.java:901)
    at org.glassfish.grizzly.http.HttpHeader.containsHeader(HttpHeader.java:762)
    at org.glassfish.grizzly.http.server.Response.containsHeader(Response.java:1268)
    at org.glassfish.grizzly.servlet.HttpServletResponseImpl.containsHeader(HttpServletResponseImpl.java:472)
    at org.glassfish.jersey.servlet.internal.ResponseWriter.writeResponseStatusAndHeaders(ResponseWriter.java:159)
    at org.glassfish.jersey.server.ServerRuntime$Responder.writeResponse(ServerRuntime.java:683)
    at org.glassfish.jersey.server.ServerRuntime$Responder.processResponse(ServerRuntime.java:444)
    at org.glassfish.jersey.server.ServerRuntime$Responder.process(ServerRuntime.java:434)
    at org.glassfish.jersey.server.ServerRuntime$2.run(ServerRuntime.java:329)
    at org.glassfish.jersey.internal.Errors$1.call(Errors.java:271)
    at org.glassfish.jersey.internal.Errors$1.call(Errors.java:267)
    at org.glassfish.jersey.internal.Errors.process(Errors.java:315)
    at org.glassfish.jersey.internal.Errors.process(Errors.java:297)
    at org.glassfish.jersey.internal.Errors.process(Errors.java:267)
    at org.glassfish.jersey.process.internal.RequestScope.runInScope(RequestScope.java:317)
    at org.glassfish.jersey.server.ServerRuntime.process(ServerRuntime.java:305)
    at org.glassfish.jersey.server.ApplicationHandler.handle(ApplicationHandler.java:1154)
    at org.glassfish.jersey.servlet.WebComponent.serviceImpl(WebComponent.java:473)
    ... 12 more

    returns a response which
      has status code 404 (FAILED)
      includes headers
        "Content-Type" with value "application/json" (FAILED)
      has a matching body (FAILED)

Неудачи:

0) Get organisation by short name returns a response which includes headers "Content-Type" with value "application/json"
      Expected header 'Content-Type' to have value 'application/json' but was 'text/html;charset=ISO-8859-1'

1) Get organisation by short name returns a response which has a matching body
      comparison -> Expected a response type of 'application/json' but the actual type was 'text/html'

2) Get organisation by short name returns a response which has status code 

    404
          assert expectedStatus == actualStatus
                 |              |  |
                 404            |  500
                                false

У меня такое чувство, что grizly не нравится Lamda, которую я сделал для обработки ошибок. Но я не хочу менять свой код только для того, чтобы заставить своего провайдера работать из-за контейнера сервера. Любые идеи, которые я пытался отладить, код течет правильно, пока не передаст его контейнеру. А затем он, кажется, полностью игнорирует аннотации Джерси


java junit jersey pact grizzly
person Thadir    schedule 13.07.2017    source источник

Ответы (2)


arrow_upward
1
arrow_downward

Я думаю, что ваш макет настроен неправильно, и поэтому вы не обрабатываете правильное исключение. Вместо того

when(awsomeService.getOrganisationByShortName("WME")).thenThrow(new AwsomeRuntimeException(ExceptionStatus.NOT_FOUND));

Ты можешь попробовать

when(awsomeService.getOrganisationByShortName(anyString()).thenThrow(new AwsomeRuntimeException(ExceptionStatus.NOT_FOUND));

Строка выше должна исправить ваше насмешливое поведение.

person damelines    schedule 14.07.2017
comment
Allass даже с уловом anyString () не работает. - person Thadir; 14.07.2017
comment
После небольшой дополнительной работы кажется, что jersy не полностью загружен, поэтому сообщения об ошибках не работают, поскольку весь bc jersy не полностью загружен в контейнер. Я собираюсь попробовать использовать другой контейнер. - person Thadir; 14.07.2017

arrow_upward
0
arrow_downward

В конце концов, исправление кажется грязным, но оно сработало. Кажется, что контейнер Grizley действительно загружает не целиком. Итак, в конце концов я поместил ответ JSON в свой макет следующим образом:

when(awsomeService.getOrganisationByShortName("WME")).thenThrow(new AwsomeRuntimeException(ExceptionStatus.NOT_FOUND,"{\n"
        + "    \"timestamp\": 1500040460362,\n" + "    \"status\": 404,\n" + "    \"error\": \"Not Found\",\n" + "    \"message\": \"Not Found\",\n"
        + "    \"path\": \"/awsome/isThisAwsome/ABCDE\"\n" + "}"));

Pact кажется счастливым, что он становится похожим на ответ JSON, которому он может соответствовать. И мне не нужно загружать больше Джерси, чем я хочу в тестовый контейнер. (Ведьма, я могу догадаться).

Таким образом, в целом, это работает, это не кажется полностью богатым способом, но это для нестандартного потока, поэтому я в порядке с этим на данный момент.

person Thadir    schedule 17.07.2017