spring-projects / spring-restdocs

Test-driven documentation for RESTful services
https://spring.io/projects/spring-restdocs
Apache License 2.0
1.16k stars 735 forks source link

Support for passing custom values/services into Snippet classes #308

Open jmisur opened 8 years ago

jmisur commented 8 years ago

In our auto-documentation extension https://github.com/ScaCap/spring-auto-restdocs we are trying to automate documentation as much as possible by introspecting code. For this we created extensions of Snippets which are dependent on some common services, such as jackson ObjectMapper, custom constraints & javadoc reader, having HandlerMethod available etc.

For passing these common components around, we are doing little hack by putting custom attributes inside MockHttpServletRequest, then inside the Snippet class getting them from Operation attributes:

    public static ObjectMapper getObjectMapper(Operation operation) {
        return (ObjectMapper) operation.getAttributes().get(ObjectMapper.class.getName());
    }

    public static void setObjectMapper(MockHttpServletRequest request, ObjectMapper objectMapper) {
        ((Map) request.getAttribute(ATTRIBUTE_NAME_CONFIGURATION))
                .put(ObjectMapper.class.getName(), objectMapper);
    }

https://github.com/ScaCap/spring-auto-restdocs/blob/master/spring-auto-restdocs-core/src/main/java/capital/scalable/restdocs/OperationAttributeHelper.java

We are setting it for every request using custom ResultHandler and alwaysDo mechanism.

        public void handle(MvcResult result) throws Exception {
            setHandlerMethod(result.getRequest(), (HandlerMethod) result.getHandler());
            setObjectMapper(result.getRequest(), objectMapper);
            ...

https://github.com/ScaCap/spring-auto-restdocs/blob/master/spring-auto-restdocs-core/src/main/java/capital/scalable/restdocs/jackson/JacksonResultHandlers.java

Because we need custom data for every request, such as request pattern and HandlerMethod, we cannot configure these things statically during Snippet class creation.

As this mechanism is basically a hack to inner workings of spring-restdocs (discovered how these attributes are passed around by digging in source code) and it's prone to change as it's internal api, it would be great if this framework offers customization of such things out of the box. Could be still using these attributes internally but having some convenient public api.

wilkinsona commented 8 years ago

I'm not sure about a public API for adding arbitrary attributes. I'm not too keen on the way that things are passed around internally at the moment, so building something that's public on top of that doesn't feel right. Perhaps we can find another way to meet your goal.

What, specifically, do you need to access in your snippet?

wilkinsona commented 7 years ago

Closing due to the lack of the requested information. @jmisur If you can provide the required information we can take another look at this.

jmisur commented 7 years ago

Hi @wilkinsona, sorry for not responding for such a long time. Our project stalled a bit, but we are back contributing to it.

So for us specifically, we need to access these objects in our Snippet classes:

So that's about it. Half of the stuff are custom beans, which we need to access inside Snippets and some things are hacked out of existing org.springframework.restdocs.configuration attribute of the Operation.

Any ideas how to access these things without hacking the Operation's attributes are welcome!