Dc/8546 semantic models in graph selection

[dave-connors-3]

resolves #8546

• adds SemanticModelSelectionMethod to allow for dbt ls -s "semantic_model:*"
• adds ability to dbt ls --resource-type semantic_model

as of 9/7

SemanticModelSelectionMethod works

❯ dbt ls -s +semantic_model:orders                      
15:10:25  Running with dbt=1.7.0-b2
15:10:25  Registered adapter: snowflake=1.7.0-b1
15:10:25  Found 5 models, 1 analysis, 1 source, 0 exposures, 0 metrics, 514 macros, 0 groups, 1 semantic model
davebt.numbers # model
semantic_model:davebt.orders # semantic model

• dbt ls does not include semantic models
• dbt ls --resource-type semantic_model yields no nodes

Problem

Checklist

• I have read the contributing guide and understand what's expected of me
• I have run this code in development and it appears to resolve the stated issue
• This PR includes tests, or tests are not required/relevant for this PR
• This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

[github-actions]

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide.

[codecov]

Codecov Report

Attention: 18 lines in your changes are missing coverage. Please review.

Comparison is base (aa86fdf) 86.61% compared to head (89d6aad) 86.59%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8589      +/-   ##
==========================================
- Coverage   86.61%   86.59%   -0.02%     
==========================================
  Files         176      176              
  Lines       25678    25731      +53     
==========================================
+ Hits        22240    22283      +43     
- Misses       3438     3448      +10     

Flag	Coverage Δ
integration	83.35% <68.85%> (-0.01%)	⬇️
unit	65.14% <55.73%> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
core/dbt/cli/params.py	100.00% <ø> (ø)
core/dbt/compilation.py	95.84% <100.00%> (-0.35%)	⬇️
core/dbt/graph/cli.py	94.35% <100.00%> (ø)
core/dbt/task/list.py	93.38% <100.00%> (+4.69%)	⬆️
core/dbt/graph/selector_methods.py	91.03% <74.07%> (-0.70%)	⬇️
core/dbt/contracts/graph/nodes.py	95.09% <54.16%> (-1.06%)	⬇️

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jtcohen6]

TODO: finish ability to dbt ls --resource-type semantic_model

Found it:

dbt-core/core/dbt/graph/cli.py

Line 24 in b39eeb3

DEFAULT_INCLUDES: List[str] = ["fqn:*", "source:*", "exposure:*", "metric:*"]

Should be:

DEFAULT_INCLUDES: List[str] = ["fqn:*", "source:*", "exposure:*", "metric:*", "semantic_model:*"]

Then semantic models will be included when --select is not passed.

It does feel like there's a real opportunity for refactoring here. It feels odd that sources/exposures/semantic_models/metrics are "pointer" node types, as opposed to the "logical" node types (models/seeds/snapshots/tests/analyses), and only those are included by the fqn:* selection.

But I think that's all out of scope for something we want to backport to v1.6!

[jtcohen6]

Thanks for the solid start @dave-connors-3 !!

@MichelleArk is going to take this one over the finish line, so we can get it merged & backported for the next v1.6.x patch

[FishtownBuildBot]

Opened a new issue in dbt-labs/docs.getdbt.com: dbt-labs/docs.getdbt.com#4133

[FishtownBuildBot]

Opened a new issue in dbt-labs/docs.getdbt.com: dbt-labs/docs.getdbt.com#4134