fix: Do not require alias to avoid duplicate aliasing with --empty

[anaghshineh]

resolves dbt-labs/dbt-adapters#474
docs N/A

Problem

When running a dry run via --empty, dbt would duplicate any model aliases, causing the query to fail due to invalid syntax.

Solution

This PR adopts the approaches taken in other adapters that do not require aliasing. Namely, it overrides the default True value for require_alias, setting it to False. As a result, when dbt creates dynamic SQL for dry runs, it does not inject additional aliasing. Thus, if a reference uses an alias, it keeps it as is. If not, it does not inject anything.

I also imported the BaseTestEmptyInlineSourceRef from dbt-adapters to run those additional tests. Please let me know if you'd like to see additional testing done for this PR!

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

[VersusFacit]

Thanks for your contribution. I ported your PR here and preserved your contribution entry on the changelog. Always appreciate when SMEs point out gaps in our runtime: dbt-labs/dbt-adapters#474