Support multiple matchers in MockMvc Kotlin DSL

[devtribe]

Affects: 5.2.1.RELEASE

I would expect that in a DSL everything in a lambda block will be executed.
But if i try to assert multiple matchers in the model block only the last one will be executed.
I didn't get any feedback about this behaviour, so i get the false impression that my code is executed, if it isn't.

Doesn't work like expected:

mockMvc.get(url)
  .andExpect {
    model {
      attribute("foo", "foo")
      attribute("bar", "bar")
    }
}

Works, but is redundant code:

mockMvc.get(url)
  .andExpect {
    model { attribute("foo", "foo") }
    model { attribute("bar", "bar") }
}

[sdeleuze]

I can understand how it could be confusing, but here we are reusing the ModelResultMatchers Java API:

fun model(matcher: ModelResultMatchers.() -> ResultMatcher) {
	actions.andExpect(MockMvcResultMatchers.model().matcher())
}

One way of supporting your use case would be to create a ModelResultMatchersDsl that would support multiple matchers.

Any thoughts @checketts @jnizet?

[checketts]

I absolutely want a ModelResultMatchersDsl (and would even be interested in implementing it).

I'll be converting my tests over from the kd4smt to this new DSL and I suspect I'll run into these sorts of issues.

[sdeleuze]

@checketts Do you want to provide a PR?
@jnizet Any thoughts?

[jnizet]

@sdeleuze I haven't been using the DSL in a while, and I've never been hit by this issue myself but I agree that the current situation is confusing, and that a ModelResultMatchersDsl would be nice.