From 1aef8d68e3bc261702fbba2d6502fa709bc6e209 Mon Sep 17 00:00:00 2001
From: nik <Nikolay.Chashnikov@jetbrains.com>
Date: Thu, 15 Aug 2013 17:12:20 +0400
Subject: [PATCH] remote servers: 'test connection' button fixed

---
 .../impl/configuration/RemoteServerConfigurable.java        | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/platform/remote-servers/impl/src/com/intellij/remoteServer/impl/configuration/RemoteServerConfigurable.java b/platform/remote-servers/impl/src/com/intellij/remoteServer/impl/configuration/RemoteServerConfigurable.java
index 1a65ddd4c06b..0f460ace3309 100644
--- a/platform/remote-servers/impl/src/com/intellij/remoteServer/impl/configuration/RemoteServerConfigurable.java
+++ b/platform/remote-servers/impl/src/com/intellij/remoteServer/impl/configuration/RemoteServerConfigurable.java
@@ -61,7 +61,7 @@ public class RemoteServerConfigurable extends NamedConfigurable<RemoteServer<?>>
       @Override
       public void actionPerformed(ActionEvent e) {
         try {
-          apply();
+          myConfigurable.apply();
         }
         catch (ConfigurationException exc) {
           Messages.showErrorDialog(myMainPanel, "Cannot test connection: " + exc.getMessage(), exc.getTitle());
@@ -100,7 +100,9 @@ public class RemoteServerConfigurable extends NamedConfigurable<RemoteServer<?>>
           }
         });
         while (!indicator.isCanceled()) {
-          semaphore.waitFor(500);
+          if (semaphore.waitFor(500)) {
+            break;
+          }
         }
         Runnable showResult = showResultRef.get();
         if (showResult != null) {