Bugfix: Update available/supported implementation.

Closes bug #3032.

4 files changed, 9 insertions, 10 deletions

diff --git a/changes/bug-3032_update-supported-provided-services b/changes/bug-3032_update-supported-provided-services
new file mode 100644
index 00000000..6cd231ae
--- /dev/null
+++ b/changes/bug-3032_update-supported-provided-services
@@ -0,0 +1 @@
+  o Update implementation and semantics of the supported and available services by a provider. Closes bug #3032.
diff --git a/src/leap/config/providerconfig.py b/src/leap/config/providerconfig.py
index 94568c27..8b72153a 100644
--- a/src/leap/config/providerconfig.py
+++ b/src/leap/config/providerconfig.py
@@ -24,7 +24,6 @@ import os
 from leap.common.check import leap_assert
 from leap.common.config.baseconfig import BaseConfig, LocalizedKey
 from leap.config.provider_spec import leap_provider_spec
-from leap.services import get_available
 
 logger = logging.getLogger(__name__)
 
@@ -81,17 +80,16 @@ class ProviderConfig(BaseConfig):
 
     def get_services(self):
         """
-        Returns a list with the services supported by the
-        current provider.
+        Returns a list with the available services in the current provider.
 
         :rtype: list
         """
-        services = get_available(self._safe_get_value("services"))
+        services = self._safe_get_value("services")
         return services
 
     def get_services_string(self):
         """
-        Returns a string with the services supported by the current
+        Returns a string with the available services in the current
         provider, ready to be shown to the user.
         """
         services_str = ", ".join(self.get_services())
diff --git a/src/leap/gui/wizard.py b/src/leap/gui/wizard.py
index 34eba84c..b29250c8 100644
--- a/src/leap/gui/wizard.py
+++ b/src/leap/gui/wizard.py
@@ -33,7 +33,7 @@ from leap.util.privilege_policies import is_missing_policy_permissions
 from leap.util.request_helpers import get_content
 from leap.util.keyring_helpers import has_keyring
 from leap.services.eip.providerbootstrapper import ProviderBootstrapper
-from leap.services import get_available
+from leap.services import get_supported
 
 logger = logging.getLogger(__name__)
 
@@ -523,7 +523,7 @@ class Wizard(QtGui.QWizard):
             self.tr("Services by %s") %
             (self._provider_config.get_name(),))
 
-        services = get_available(
+        services = get_supported(
             self._provider_config.get_services())
 
         for service in services:
diff --git a/src/leap/services/__init__.py b/src/leap/services/__init__.py
index 70a5dcf2..fc4aa416 100644
--- a/src/leap/services/__init__.py
+++ b/src/leap/services/__init__.py
@@ -17,10 +17,10 @@
 """
 Services module.
 """
-NOT_YET_DEPLOYED = ["mx"]  # for 0.2.2 release
+DEPLOYED = ["openvpn"]  # for 0.2.2 release
 
 
-def get_available(services):
+def get_supported(services):
     """
     Returns a list of the available services.
 
@@ -30,4 +30,4 @@ def get_available(services):
     :returns: a list of the available services
     :rtype: list of str
     """
-    return filter(lambda s: s not in NOT_YET_DEPLOYED, services)
+    return filter(lambda s: s in DEPLOYED, services)