fix(front50_test): Don't inspect config if host platform is 'native'. (…

…#1541)
spinnaker · Apr 6, 2017 · d57d7c9 · d57d7c9
1 parent 9712fba
commit d57d7c9
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/testing/citest/tests/google_front50_test.py b/testing/citest/tests/google_front50_test.py
@@ -63,12 +63,16 @@ def _do_init_bindings(self):
     """Hook to initialize custom test bindings so journaling is scoped."""
     context = ExecutionContext()
     config = self.agent.deployed_config
-    enabled = config.get('spinnaker.gcs.enabled', False)
-    if not enabled:
+    enabled = config.get('spinnaker.gcs.enabled', None)
+    if enabled == False:
       raise ValueError('spinnaker.gcs.enabled is not True')
+    if not self.bindings['GCS_BUCKET'] and not config['spinnaker.gcs.bucket']:
+      raise ValueError('The GCS bucket name is not specified')
+    if not self.bindings['GCS_BASE_PATH'] and not config['spinnaker.gcs.rootFolder']:
+      raise ValueError('The GCS rootFolder is not specified')
 
     self.BUCKET = self.bindings['GCS_BUCKET'] or config['spinnaker.gcs.bucket']
-    self.BASE_PATH = config['spinnaker.gcs.rootFolder'] or self.bindings['GCS_BASE_PATH']
+    self.BASE_PATH = self.bindings['GCS_BASE_PATH'] or config['spinnaker.gcs.rootFolder']
     self.TEST_APP = self.bindings['TEST_APP']
     self.TEST_PIPELINE_NAME = 'My {app} Pipeline'.format(app=self.TEST_APP)
     self.TEST_PIPELINE_ID = '{app}-pipeline-id'.format(app=self.TEST_APP)