Merge pull request #1425 from fishtown-analytics/fix/propagate-undefi…

…ned-to-calls

Propagate ParserMacroCapture undefineds into the calls (#1424)

core/dbt/clients/jinja.py

@@ -219,7 +219,7 @@ def __getattr__(self, name):
  return self
 
  def __call__(self, *args, **kwargs):
- return True
+ return self
 
  return ParserMacroCapture
 

test/integration/006_simple_dependency_test/sad_iteration_models/iterate.sql

@@ -0,0 +1,2 @@
+{% for x in no_such_dependency.no_such_method() %}
+{% endfor %}

test/integration/006_simple_dependency_test/test_local_dependency.py

@@ -5,6 +5,7 @@
 import dbt.config
 import dbt.exceptions
 
+
 class TestSimpleDependency(DBTIntegrationTest):
 
  @property
@@ -45,6 +46,25 @@ def test_postgres_local_dependency(self):
  )
 
 
+class TestMissingDependency(DBTIntegrationTest):
+ @property
+ def schema(self):
+ return "local_dependency_006"
+
+ @property
+ def models(self):
+ return "test/integration/006_simple_dependency_test/sad_iteration_models"
+
+ @use_profile('postgres')
+ def test_postgres_missing_dependency(self):
+ # dbt should raise a dbt exception, not raise a parse-time TypeError.
+ with self.assertRaises(dbt.exceptions.Exception) as exc:
+ self.run_dbt(['compile'])
+ message = str(exc.exception)
+ self.assertIn('no_such_dependency', message)
+ self.assertIn('is undefined', message)
+
+
 class TestSimpleDependencyWithSchema(TestSimpleDependency):
  @property
  def project_config(self):