Prefer root namespace by default (#31)

dbt-labs · Oct 25, 2021 · f1deecc · f1deecc
1 parent 8437887
commit f1deecc
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/tests/unit/test_context.py b/tests/unit/test_context.py
@@ -452,14 +452,19 @@ def test_resolve_specific(config, manifest_extended, redshift_adapter, get_inclu
     with pytest.raises(dbt.exceptions.CompilationException):
         ctx['adapter'].dispatch('macro_a').macro
 
+    # root namespace is always preferred, unless search order is explicitly defined in 'dispatch' config
     assert ctx['adapter'].dispatch('some_macro').macro is package_rs_macro
-    assert ctx['adapter'].dispatch('some_macro', 'dbt').macro is rs_macro
+    assert ctx['adapter'].dispatch('some_macro', 'dbt').macro is package_rs_macro
     assert ctx['adapter'].dispatch('some_macro', 'root').macro is package_rs_macro
 
-    # override 'dbt' namespace, dispatch to 'root' first
+    # override 'dbt' namespace search order, dispatch to 'root' first
     ctx['adapter'].config.dispatch = [{'macro_namespace': 'dbt', 'search_order': ['root', 'dbt']}]
     assert ctx['adapter'].dispatch('some_macro', macro_namespace = 'dbt').macro is package_rs_macro
 
-    # override 'root' namespace, dispatch to 'dbt' first
+    # override 'dbt' namespace search order, dispatch to 'dbt' only
+    ctx['adapter'].config.dispatch = [{'macro_namespace': 'dbt', 'search_order': ['dbt']}]
+    assert ctx['adapter'].dispatch('some_macro', macro_namespace = 'dbt').macro is rs_macro
+
+    # override 'root' namespace search order, dispatch to 'dbt' first
     ctx['adapter'].config.dispatch = [{'macro_namespace': 'root', 'search_order': ['dbt', 'root']}]
-    assert ctx['adapter'].dispatch('some_macro', macro_namespace='root').macro is rs_macro
+    assert ctx['adapter'].dispatch('some_macro', macro_namespace = 'root').macro is rs_macro