Current coverage is 23.68%

Merging #533 into master will decrease coverage by -0.03% as of e4bb095

@@            master    #533   diff @@
======================================
  Files          295     295       
  Stmts         6102    6109     +7
  Branches       530     532     +2
  Methods          0       0       
======================================
  Hit           1447    1447       
  Partial         85      85       
- Missed        4570    4577     +7

Review entire Coverage Diff as of e4bb095

Powered by Codecov. Updated on successful CI builds.

• Please provide information when this field appeared (you should compare API versions https://docs.docker.com/engine/reference/api/docker_remote_api/ )
• Please read https://github.com/docker-java/docker-java/blob/master/docs/devel.adoc
• Please ensure that option is added to jaxrs and netty implementation.

@KostyaSha ,
The shmsize option appeared in the latest version (v1.22) of the remote api.
I've adapted code to the project code style/design.
Thanks!

LGTM @KostyaSha ?

but there is no test :(

Integration test will be difficult... How this could be tested?

@marcuslinke for non cmd

@KostyaSha This field is not serialized to json, so serder test will not work.

You have the field and it sends to docker daemon as request. Url parameter is only untestable without integration.

Sorry, not sure what you mean...

@marcuslinke all *Cmd classes corresponds to request objects. For docker daemon result of *cmd execution is just json (request) that sent to daemon. You can dump it and i want serialise/deserialise request data to class directly. Atm i can't do this because all *Cmd classes has constructor, because cmd contains request data and execution logic (code that do request). As i understand initially in this library *Cmd was only the execution method that allowed set request data.
Tied data to request code allows make fluent and shorter library calls.

LGTM @KostyaSha ?

LGTM