support update state when handling patch requests in CrudDispatcher

fabric8io / mockwebserver

An extension of okhttp's mockwebserver, that provides a DSL and is easier to use

Apache License 2.0

113 stars 38 forks source link

support update state when handling patch requests in CrudDispatcher #32

Closed psFried closed 5 years ago

psFried commented 5 years ago

The CrudDispatcher does not currently update its stored resource when handling a patch request. I'm currently subclassing KubernetesCrudDispatcher to handle PATCH requests as expected, but it seemed like that addition would be generally useful. This PR adds that behavior, so that you can execute a PATCH and then a subsequent GET will return the modified resource. Please let me know if there's any issues with this.

jkebinger commented 5 years ago

@oscerd Do you have an ETA when you'll be releasing a new build of mockwebserver with this functionality? Spent a while chasing my tail today expecting patch to work and it would be great to count on going forward. Thank you!