Removed a logical error in the checking of the existing of the enviro…

…nment variables

CHANGELOG.rst

@@ -3,6 +3,10 @@ Change Log
 
 pykechain changelog
 
+1.12.2 (15SEP17)
+----------------
+ * Removed a logical error in the checking of the existing of the environment variables
+
 1.12.1 (15SEP17)
 ----------------
  * Added the ability to enforce the use of environment variables when the KECHAIN_FORCE_ENV_USE is set to a true value in the environment. Altered documentation and altered tests for that (#193)

pykechain/__about__.py

@@ -2,7 +2,7 @@
 name = 'pykechain'
 description = 'KE-chain Python SDK'
 
-version = '1.12.1'
+version = '1.12.2'
 
 author = 'KE-works BV'
 email = 'support+pykechain@ke-works.com'

pykechain/helpers.py

@@ -1,3 +1,4 @@
+import os
 from envparse import env
 
 from pykechain.client import Client
@@ -72,14 +73,14 @@ def get_project(url=None, username=None, password=None, token=None, scope=None,
     Bike Project
     """
     if env.bool(kecenv.KECHAIN_FORCE_ENV_USE, default=False):
-        if not env(kecenv.KECHAIN_URL):
+        if not os.getenv(kecenv.KECHAIN_URL):
             raise ClientError(
                 "Error: KECHAIN_URL should be provided as environment variable (use of env vars is enforced)")
-        if not env(kecenv.KECHAIN_TOKEN, None) or \
-                not (env(kecenv.KECHAIN_PASSWORD, None) and (env(kecenv.KECHAIN_PASSWORD, None))):
+        if not (os.getenv(kecenv.KECHAIN_TOKEN) or
+                (os.getenv(kecenv.KECHAIN_PASSWORD) and os.getenv(kecenv.KECHAIN_PASSWORD))):
             raise ClientError("Error: KECHAIN_TOKEN or KECHAIN_USERNAME and KECHAIN_PASSWORD should be provided as "
                               "environment variable(s) (use of env vars is enforced)")
-        if not env(kecenv.KECHAIN_SCOPE, None) or not env(kecenv.KECHAIN_SCOPE_ID, None):
+        if not (os.getenv(kecenv.KECHAIN_SCOPE) or os.getenv(kecenv.KECHAIN_SCOPE_ID)):
             raise ClientError("Error: KECHAIN_SCOPE or KECHAIN_SCOPE_ID should be provided as environment variable "
                               "(use of env vars is enforced)")