Integration test with pedestal-api example

[oliyh]

As many examples are based on pedestal-api we need to make sure it always works!

[sjolicoeur]

@oliyh using: https://pedestal-swagger.herokuapp.com/swagger.json as an endpoint martian doesn't seem to work nor can I get it to work with examples made from Django-rest openAPI

I tried the following in a REPL:

(def m (martian-http/bootstrap-swagger "https://pedestal-swagger.herokuapp.com/swagger.json" 
    {:interceptors (concat martian/default-interceptors [add-json-header])}))
#'user/m
user=> (martian/explore m)
[]

• Is there a good way to debug this?
• is there a good way to not make it dependent on route-name or operationId?

[oliyh]

Hi,

The first thing that stands out is your JSON header interceptor. Firstly, that shouldn't be required (Martian supports it already, and autodetects usage from the consumes/provides directive on Swagger). Secondly, you wouldn't concat it, as then it will run after the interceptor that actually performs the request. You would want to have it before the request interceptor.

Also you should be using the default interceptors from martian-http rather than martian.core otherwise you won't have an interceptor that performs the request at all! I suggest you look at the interceptors inside martian-http/default-interceptors to see how they build the request up and then process the response.

[oliyh]

Sorry I didn't mean to close this issue. Also I notice that what I said above might not apply to the bootstrap phase (but will apply once you get this issue sorted!).

I'll have a closer look at this later unless you manage to solve it in the meantime.