Checked map length to prevent errors

[lvarley]

resolves #

Not sure if there is an existing logged issue for this, however:

Description

This is a fix to an error I get with my manifest.json using dbt-core 1.6.6 and snowflake-adapter 1.6.4. Possibly due to dynamic tables or quoted columns I couldn't quite work it out

  File "/dbterd/adapters/algos/test_relationship.py", line 170, in get_table_map
    if f'("{map[1].split(".")[-1]}")'.lower() in to_model.replace("'", '"').lower():
IndexError: list index out of range

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
• I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR

[codecov]

Codecov Report

Merging #58 (f810919) into main (81ed668) will decrease coverage by 0.09%.
Report is 5 commits behind head on main.
The diff coverage is 95.65%.

❗ Current head f810919 differs from pull request most recent head 89a2640. Consider uploading reports for the commit 89a2640 to get more accurate results

@@             Coverage Diff             @@
##              main      #58      +/-   ##
===========================================
- Coverage   100.00%   99.91%   -0.09%     
===========================================
  Files           43       43              
  Lines         1130     1137       +7     
===========================================
+ Hits          1130     1136       +6     
- Misses           0        1       +1     

Files	Coverage Δ
dbterd/adapters/algos/base.py	100.00% <100.00%> (ø)
dbterd/adapters/filter.py	100.00% <100.00%> (ø)
dbterd/adapters/meta.py	100.00% <100.00%> (ø)
dbterd/cli/params.py	100.00% <100.00%> (ø)
...ests/unit/adapters/algos/test_test_relationship.py	100.00% <100.00%> (ø)
...t/adapters/targets/d2/test_d2_test_relationship.py	100.00% <ø> (ø)
...apters/targets/dbml/test_dbml_test_relationship.py	100.00% <ø> (ø)
...argets/graphviz/test_graphviz_test_relationship.py	100.00% <ø> (ø)
.../targets/mermaid/test_mermaid_test_relationship.py	100.00% <ø> (ø)
...argets/plantuml/test_plantuml_test_relationship.py	100.00% <ø> (ø)
... and 2 more

[datnguye]

Hi @lvarley thanks for proactively flag it out and having the PR 💯

Could you help to give a sample for reproducing? Or may be a whole manifest.json file? Thanks a lot

[datnguye]

Thanks @lvarley I got your email of the attached manifest.json
I see the issue which is the root caused by the column quote (") in snowflake.

I have a comment on the change, could you help to make a change? I am happy to make it otherwise

• Revert your current change
• Change Line 87 and Line 96: remove double quote

).lower(),

to:

).replace("\"","").lower(),

• For more context, in DBML syntax, it doesn't support to escape double quote, we can only ignore it though

Thanks