Merge pull request #1979 from fishtown-analytics/feature/configurable…

…-test-warehouses make tests honor environment variables (#1939)
dbt-labs · Dec 5, 2019 · 549168b · 549168b
2 parents ace777e + 5797be9
commit 549168b
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 4 deletions.
diff --git a/test.env.sample b/test.env.sample
@@ -6,6 +6,7 @@ SNOWFLAKE_TEST_DATABASE=
 SNOWFLAKE_TEST_ALT_DATABASE=
 SNOWFLAKE_TEST_QUOTED_DATABASE=
 SNOWFLAKE_TEST_WAREHOUSE=
+SNOWFLAKE_TEST_ALT_WAREHOUSE=
 
 BIGQUERY_TYPE=
 BIGQUERY_PROJECT_ID=

diff --git a/test/integration/050_warehouse_test/models/expected_warehouse.sql b/test/integration/050_warehouse_test/models/expected_warehouse.sql
@@ -1,2 +1,2 @@
 {{ config(materialized='table') }}
-select 'DBT_TEST_ALT' as warehouse
+select '{{ env_var("SNOWFLAKE_TEST_ALT_WAREHOUSE", "DBT_TEST_ALT") }}' as warehouse
diff --git a/test/integration/050_warehouse_test/models/override_warehouse.sql b/test/integration/050_warehouse_test/models/override_warehouse.sql
@@ -1,2 +1,2 @@
-{{ config(snowflake_warehouse='DBT_TEST_ALT', materialized='table') }}
+{{ config(snowflake_warehouse=env_var('SNOWFLAKE_TEST_ALT_WAREHOUSE', 'DBT_TEST_ALT'), materialized='table') }}
 select current_warehouse() as warehouse
diff --git a/test/integration/050_warehouse_test/project-config-models/expected_warehouse.sql b/test/integration/050_warehouse_test/project-config-models/expected_warehouse.sql
@@ -1,2 +1,2 @@
 {{ config(materialized='table') }}
-select 'DBT_TEST_ALT' as warehouse
+select '{{ env_var("SNOWFLAKE_TEST_ALT_WAREHOUSE", "DBT_TEST_ALT") }}' as warehouse
diff --git a/test/integration/050_warehouse_test/test_warehouses.py b/test/integration/050_warehouse_test/test_warehouses.py
@@ -39,7 +39,7 @@ def project_config(self):
             'source-paths': ['project-config-models'],
             'models': {
                 'test': {
-                    'snowflake_warehouse': 'DBT_TEST_ALT',
+                    'snowflake_warehouse': os.getenv('SNOWFLAKE_TEST_ALT_WAREHOUSE', 'DBT_TEST_ALT'),
                 },
             },
         }