Skip to content

Commit dfa76ec

Browse files
committed
raise and catch DbtSelectorsError instead of DbtRuntimeError
1 parent 743145b commit dfa76ec

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

core/dbt/graph/selector_methods.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@
3333
)
3434
from dbt.contracts.graph.unparsed import UnparsedVersion
3535
from dbt.contracts.state import PreviousState
36+
from dbt.exceptions import DbtSelectorsError
3637
from dbt.node_types import NodeType
3738
from dbt_common.dataclass_schema import StrEnum
3839
from dbt_common.events.contextvars import get_project_root
@@ -975,7 +976,7 @@ def _search_for_matched_selector(
975976

976977
def search(self, included_nodes: Set[UniqueId], selector: str) -> Iterator[UniqueId]:
977978
if self._selectors is None or self._get_selected_callback is None:
978-
raise DbtRuntimeError("Cannot use selector: method if selectors.yml is not provided")
979+
raise DbtSelectorsError("Cannot use selector: method if selectors.yml is not provided")
979980
matched_selector_dfns = []
980981

981982
# ensure that selectors are sorted to make the execution deterministic
@@ -988,7 +989,7 @@ def search(self, included_nodes: Set[UniqueId], selector: str) -> Iterator[Uniqu
988989
matched_selector_dfns.append(s_value["definition"])
989990

990991
if not matched_selector_dfns:
991-
raise DbtRuntimeError(
992+
raise DbtSelectorsError(
992993
f"Selector '{selector}' did not match any selector in selectors.yml"
993994
)
994995

tests/functional/selectors/test_selector_selector_method.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@
22

33
import pytest
44

5+
from dbt.exceptions import DbtSelectorsError
56
from dbt.tests.util import run_dbt
6-
from dbt_common.exceptions import DbtRuntimeError
77
from dbt_common.exceptions import RecursionError as DbtRecursionError
88

99
models__model_a_sql = """
@@ -239,5 +239,5 @@ def test_selector_with_explicit_selector_method(self, project):
239239
assert_result_set(result, {"test.model_a"})
240240

241241
def test_raises_if_no_selector_matches(self, project):
242-
with pytest.raises(DbtRuntimeError):
242+
with pytest.raises(DbtSelectorsError):
243243
run_dbt(["ls", "--select", "selector:nonexistent_selector"])

0 commit comments

Comments
 (0)