Merge branch '__rultor'

Author: rultor

src/main/java/com/amihaiemil/docker/Plugin.java

@@ -34,10 +34,6 @@
  * A docker plugin.
  * @author Boris Kuzmic ([email protected])
  * @see <a href="https://docs.docker.com/engine/api/v1.35/#tag/Plugin">Docker Plugin API</a>
- * @todo #266:30min Implement Plugin#configure method. The tests are already
- *  coded, so after the implementation just remove the ignore annotation from
- *  these tests. More information about Configure API method can be found at:
- *  https://docs.docker.com/engine/api/v1.35/#operation/PluginSet
  * @since 0.0.7
  */
 public interface Plugin extends JsonObject {

src/main/java/com/amihaiemil/docker/RtPlugin.java

@@ -28,7 +28,9 @@
 import java.io.IOException;
 import java.net.URI;
 import java.util.Map;
+import javax.json.Json;
 import javax.json.JsonArray;
+import javax.json.JsonArrayBuilder;
 import javax.json.JsonObject;
 import org.apache.http.HttpStatus;
 import org.apache.http.client.HttpClient;
@@ -170,12 +172,31 @@ public void push() throws IOException, UnexpectedResponseException {
     @Override
     public void configure(final Map<String, String> options)
         throws IOException, UnexpectedResponseException {
-        throw new UnsupportedOperationException(
-            String.join(" ",
-                "RtPlugin.configure() is not yet implemented.",
-                "If you can contribute please",
-                "do it here: https://www.github.com/amihaiemil/docker-java-api"
-            )
-        );
+        final JsonArrayBuilder json = Json.createArrayBuilder();
+        if (options != null) {
+            options.forEach(
+                (key, value) -> json.add(String.format("%s=%s", key, value))
+            );
+        }
+        final HttpPost upgrade =
+            new HttpPost(
+                String.format("%s/%s", this.uri.toString(), "set")
+            );
+        try {
+            upgrade.setEntity(
+                new StringEntity(
+                    json.build().toString(), ContentType.APPLICATION_JSON
+                )
+            );
+            this.client.execute(
+                upgrade,
+                new MatchStatus(
+                    upgrade.getURI(),
+                    HttpStatus.SC_NO_CONTENT
+                )
+            );
+        } finally {
+            upgrade.releaseConnection();
+        }
     }
 }

src/test/java/com/amihaiemil/docker/RtPluginTestCase.java

@@ -40,7 +40,6 @@
 import org.hamcrest.MatcherAssert;
 import org.hamcrest.collection.IsCollectionWithSize;
 import org.hamcrest.core.IsEqual;
-import org.junit.Ignore;
 import org.junit.Test;
 import org.mockito.Mockito;
 
@@ -437,7 +436,6 @@ public void pushFailsPluginNotInstalled() throws Exception {
      * RtPlugin configure plugin.
      * @throws Exception If something goes wrong.
      */
-    @Ignore
     @Test
     public void configureOk() throws Exception {
         new ListedPlugins(
@@ -493,7 +491,6 @@ public void configureOk() throws Exception {
      * responds with 404.
      * @throws Exception If something goes wrong.
      */
-    @Ignore
     @Test(expected = UnexpectedResponseException.class)
     public void configureFailsPluginNotInstalled() throws Exception {
         final Plugin plugin = new RtPlugin(