Skip to content

Conversation

@sfc-gh-pfus
Copy link
Collaborator

Description

NO-SNOW Added test for spread operator.

Checklist

  • Created tests which fail without the change (if possible)
  • Extended the README / documentation, if necessary

@sfc-gh-pfus sfc-gh-pfus requested a review from a team as a code owner March 24, 2025 10:23
@sfc-gh-snowflakedb-snyk-sa
Copy link

sfc-gh-snowflakedb-snyk-sa commented Mar 24, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

defer dbt.mustExec("DROP TABLE IF EXISTS spread_operator_test")
dbt.mustExec("INSERT INTO spread_operator_test VALUES (?)", 1)
dbt.mustExec("INSERT INTO spread_operator_test VALUES (?)", 2)
rows := dbt.mustQuery("SELECT * FROM spread_operator_test WHERE id IN (**?)", []int{1, 2, 3})

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a suggestion - using "SELECT COUNT(*)" instead might might cut the unit test by a couple of lines.

@sfc-gh-pfus sfc-gh-pfus marked this pull request as draft March 31, 2025 05:45
@sfc-gh-mmishchenko sfc-gh-mmishchenko force-pushed the NO-SNOW-spread-operator-test branch from 21ad3df to 8825a37 Compare June 26, 2025 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants